apt/test/libapt/globalerror_test.cc, branch 2.2.1Debians commandline package managerhttps://git.kalnischkies.de/apt/atom?h=2.2.12017-12-13T22:53:41Zavoid some useless casts reported by -Wuseless-cast2017-12-13T22:53:41ZDavid Kalnischkiesdavid@kalnischkies.de2017-12-13T20:39:16Zurn:sha1:1adcf56bec7d2127d83aa423916639740fe8e586
The casts are useless, but the reports show some where we can actually
improve the code by replacing them with better alternatives like
converting whatever int type into a string instead of casting to a
specific one which might in the future be too small.
Reported-By: gcc -Wuseless-cast
Reformat and sort all includes with clang-format2017-07-12T11:57:51ZJulian Andres Klodejak@debian.org2017-07-12T11:40:41Zurn:sha1:87274d0f22e1dfd99b2e5200e2fe75c1b804eac3
This makes it easier to see which headers includes what.
The changes were done by running
git grep -l '#\s*include' \
| grep -E '.(cc|h)$' \
| xargs sed -i -E 's/(^\s*)#(\s*)include/\1#\2 include/'
To modify all include lines by adding a space, and then running
./git-clang-format.sh.
properly format multiline error messages2016-04-25T13:35:52ZDavid Kalnischkiesdavid@kalnischkies.de2016-03-12T14:49:54Zurn:sha1:82153438235144c485a8539b5c035e67bbef96e4use Google C++ Testing Framework for libapt tests2014-04-16T16:36:14ZDavid Kalnischkiesdavid@kalnischkies.de2014-04-16T15:09:37Zurn:sha1:f00832cc273e52a47fb88e49849891b771de4e17
My commit 45df0ad2 from 26. Nov 2009 had a little remark:
"The commit also includes a very very simple testapp."
This was never intended to be permanent, but as usually…
The commit adds the needed make magic to compile gtest statically
as it is required and links it against a small runner. All previous
testcase binaries are reimplemented in gtest and combined in this
runner. While most code is a 1:1 translation some had to be rewritten
like compareversion_test.cc, but the coverage remains the same.
cleanup headers and especially #includes everywhere2014-03-13T12:58:45ZDavid Kalnischkiesdavid@kalnischkies.de2014-03-05T21:11:25Zurn:sha1:453b82a388013e522b3a1b9fcd6ed0810dab1f4f
Beside being a bit cleaner it hopefully also resolves oddball problems
I have with high levels of parallel jobs.
Git-Dch: Ignore
Reported-By: iwyu (include-what-you-use)
warning: unused parameter ‘foo’ [-Wunused-parameter]2014-03-13T12:58:45ZDavid Kalnischkiesdavid@kalnischkies.de2014-03-01T14:11:42Zurn:sha1:655122418d714f342b5d9789f45f8035f3fe8b9a
Reported-By: gcc -Wunused-parameter
Git-Dch: Ignore
Fix typos in documentation (codespell)2014-02-22T17:34:33ZMichael Vogtmvo@debian.org2014-02-22T17:34:33Zurn:sha1:1e3f4083db29bba600b9725e9456b0e140975c99* test/libapt/globalerror_test.cc:2012-01-20T00:34:31ZPino Toscanopino@debian.org2012-01-20T00:34:31Zurn:sha1:18aea9e6c38ec0397e546563b8a54a99b431028b
- errno 0 has a different strerror on hurd, so generate the expected
message dynamically instead of hardcoding 'Success' (Closes: #656530)
write a proper testcase replacing the print-only uri.cc test2011-08-15T16:22:44ZDavid Kalnischkieskalnischkies@gmail.com2011-08-15T16:22:44Zurn:sha1:bf5438c1904c57d5f03c58a2f7d6e972020f91f9* apt-pkg/contrib/error.cc:2011-02-15T13:04:06ZDavid Kalnischkieskalnischkies@gmail.com2011-02-15T13:04:06Zurn:sha1:38f297033faa099a0705c8f66964d8c5d7927c8d
- ensure that va_list is not invalid in second try