convert a few for-loop char finds to proper strchr and memchr

* methods/https.cc:
  - cleanup broken downloads properly

* apt-pkg/deb/dpkgpm.cc:
  - use std::vector instead of fixed size arrays to store args and
    multiarch-packagename strings
  - load the dpkg base arguments only one time and reuse them later
* cmdline/apt-get.cc:
  - follow Provides in the evaluation of saving candidates, too, for
    statisfying garbage package dependencies (Closes: #640590)
* apt-pkg/algorithms.cc:
  - if a package is garbage, don't try to save it with FixByInstall

* apt-pkg/algorithms.cc:
  - if a package is garbage, don't try to save it with FixByInstall

* cmdline/apt-get.cc:
  - follow Provides in the evaluation of saving candidates, too, for
    statisfying garbage package dependencies (Closes: #640590)

load the dpkg base arguments only one time and reuse them later

* apt-pkg/deb/dpkgpm.cc:
  - use std::vector instead of fixed size arrays to store args and
    multiarch-packagename strings

fix foldmarker in algorithms.h

remove old APT_COMPATIBILITY ifdef's

use forward declaration in headers if possible instead of includes

do not pollute namespace in the headers with using (Closes: #500198)

remove an extra argument for the error mesage format

* apt-pkg/acquire-item.h, apt-pkg/deb/debmetaindex.cc:
  - fix fetching language information by adding OptionalSubIndexTarget

update the test to ensure that NotAutomatic and co is used also
if the archive is signed, but the signature can't be checked
as the key is missing (debbugs #597301 - fix in r2105)

releasing version 0.8.16~exp6

* doc/apt-get.8.xml:
  - change wording of autoremove description as suggested
    by Robert Simmons, thanks! (Closes: #641490)

merged from donkult

enable APT in unpack/configure ordering to handle loops as well
as tight dependencies between immediate packages better
enabling also the possibility to mark all packages as immediate
(at least Closes: #353290, #540227, #559733, #621836, #639290)

Changelog entry forspelling fix

merged from donkult

Fix spelling error

update symbols

update symbols file

update package names too for the new abi

* [abi-break] Support large files in the complete toolset. Indexes of this
* bump ABI version

merged from lp:~donkult/apt/experimental

releasing version 0.8.15.8

* apt-pkg/contrib/configuration.cc:
  - fix double delete (LP: #848907)
  - ignore only the invalid regexp instead of all options

* cmdline/apt-get.cc:
  - output list of virtual package providers to c1out in -q=1
    instead of /dev/null to unbreak sbuild (LP: #816155)

merge with debian/sid

merge with debian/experimental

Support large files in the complete toolset. Indexes of this
size are pretty unlikely for now, but we need it for deb
packages which could become bigger than 4GB now (LP: #815895)

* cmdline/apt-get.cc:
  - output list of virtual package providers to c1out in -q=1
    instead of /dev/null to unbreak sbuild (LP: #816155)

* apt-pkg/contrib/configuration.cc:
  - fix double delete (LP: #848907)
  - ignore only the invalid regexp instead of all options

reorder includes: add <config.h> if needed and include it at first

releasing version 0.8.15.7

merged from lp:~donkult/apt/sid

fix changelog typos and bugfix-link

M-A:same lockstep unpack should operate on installed
packages first (LP: #835625)

fix bashism (local outside function) and releasefile creation in testcases

  - add an Acquire::Min-ValidTime option (Closes: #640122)
* doc/apt.conf.5.xml:
  - reword Acquire::Max-ValidTime documentation to make clear
    that it doesn't provide the new Min-ValidTime functionality

* apt-pkg/indexrecords.cc:
  - fix Acquire::Max-ValidTime option by interpreting it really
    as seconds as specified in the manpage and not as days

* apt-pkg/orderlist.cc:
  - prefer visiting packages marked for deletion in VisitProvides
    if we are operating on a negative dependency so that we can
    deal early with the fallout of this remove

print from their the visit came from

* cmdline/apt-key:
  - if command is 'add' do not error out if the specified
    keyring doesn't exist, it will be created by gpg

remove the caches in 'apt-get update', too, as they will be
invalid in most cases anyway

* apt-pkg/acquire.cc:
  - non-existing directories are by definition clean

* apt-pkg/acquire-item.cc:
  - if no Release.gpg file is found try to verify with hashes,
    but do not fail if a hash can't be found

* cmdline/apt-get.cc:
  - remove the binary caches in 'apt-get clean' as it is the first
    thing recommend by many supporters in case of APT segfaults

merged lp:~donkult/apt/sid

set the msglevel to a high dummy value if none is specified

for consistency allow the usage of 'make test' instead of './run-tests'

understand DEB_BUILD_OPTIONS nocheck to disable the testexecution

* debian/apt.symbols:
  - add the newly added symbols since 0.8.15.3

* test/*
  - reorganize the various testcases and helper we have and
    integrate them better into the buildsystem
  - run the test/libapt testcases at package build-time

fix 'comparison between signed and unsigned integer expressions' warning
in the udevcdrom testcase

do not output color in the testcases if stdout/stderr are not a /dev/pts

build test-binaries by default and add a test target to toplevel

fix the compareversion test to work then called from outside of libapt

the experimental branch has a better version of this test in libapt,
so remove it now - we will get it back at merge-time with experimental

remove fake-status leftover from the python-test

move the networkless test to the interactive helpers as it needs
root rights for operation (ifup/down, iptables) so it is better run
under supervision

remove the python testcases as this is covered by integration/

move the interactive helper to a subdirectory of test

add 'dirs' target to the 'all' target to ensure that for e.g. the tests
all (sub)directories are build which are needed (e.g. obj/ and co)

move the list of versions to test with to the libapt subdirectory

merged from lp:~donkult/apt/sid

remove the interactive scratch test, it doesn't test much and what
it tests is covered by your non-interactive testcases,
so this one wasn't used for a looooong time

replaced by the integration/test-compressed-indexes test

merge with the final changelog from debian-sid

* apt-pkg/packagemanager.cc, apt-pkg/pkgcache.cc:
  - ignore "self"-conflicts for all architectures of a package
    instead of just for the architecture of the package locked at
    in the ordering of installations too (Closes: #802901)

merge the tests for configuration into another libapt-test

Fixed a problem where the loop flag would be removed prematurely.

  SmartConfigure xserver-xorg-video-apm (1:1.2.3-0ubuntu1)
    SmartConfigure xserver-xorg-core (2:1.9.0-0ubuntu7.3) <- Loop flag set on xserver-xorg-core
      SmartConfigure xserver-xorg (1:7.5+6ubuntu3)
        SmartConfigure xserver-xorg-core (2:1.9.0-0ubuntu7.3) (Only Correct Dependancies) <- Loop flag removed prematurely
          SmartConfigure libpciaccess0 (0.12.0-1)
          SmartConfigure libpixman-1-0 (0.18.4-1)
        SmartConfigure xserver-xorg-video-all (1:7.5+6ubuntu3)
          SmartConfigure xserver-xorg-video-apm (1:1.2.3-0ubuntu1) (Only Correct Dependancies)
            SmartConfigure xserver-xorg-core (2:1.9.0-0ubuntu7.3) <- Incorrectly detects first run as no loop flag

Also applied this fix to the SmartUnpack method.

Fixed a bug where SmartUnPack would be called with Immediate=true (to resolve dependancies in SmartConfigure) yet Pkg
would not be immediately configured. This was because SmartUnPack still required the immediate flag to be set on Pkg.

Also inproved the debuging adding indented output for SmartUnPack and SmartConfigure and specifying in the output if
SmartConfigure or SmartUnPack was called just to Correct something (PkgLoop = true) or not.

support yet another webserver in the integration tests: gatling

write a proper testcase replacing the print-only uri.cc test

releasing version 0.8.16~exp5

merged from lp:~mvo/apt/mvo

merged from  the debian-sid branch

releasing version 0.8.15.6

merged fixes from lp:~mvo/apt/mvo

Applied DonKult (David)'s excellent fix for inproving the loop management. Now both SmartConfigure and SmartUnPack can be
called mutiple times on the same package, this is to make sure that when loops are broken all packages that are required
are kept in the same dpkg run.

Fix a bug introduced in Rev.2159 on line 398, also fix another potential bug.

Added code to allow SmartConfigure to be called mutiple times on the same package to ensure all dependancies are satisfied.

Inproved debug with versioning

Small fix for loop handeling.

lots of cppcheck fixes

cherry pick test/integration/Packages-hashsum-verification addition from lp:~mvo/apt/mvo

test/integration/Packages-hashsum-verification: commit the forgotten packages file

follow the recommendation of cppcheck to make some method methods (scnr)
const and initial mostly Debug member values in the constructors

rename the parameter name of MarkDelete from Purge to MarkPurge to
fix a cosmetic warning from cppcheck:
[apt-pkg/depcache.h:462] -> [apt-pkg/depcache.h:122]:
 (style) Variable 'Purge' hides enumerator with same name

fix a few more cppcheck performance and scope warnings

cppcheck complains about some possible speed improvements which could be
done on the mirco-optimazation level, so lets fix them:
 (performance) Possible inefficient checking for emptiness.
 (performance) Prefer prefix ++/-- operators for non-primitive types.

fix some cppcheck: (warning) Member variable is not initialized in the constructor.

cppcheck is right that the check for preventing null deference of Query
is redundant in Redirect() as we can't reach the code with null anyway

[apt-pkg/acquire-method.cc:433]: (error) Possible null pointer dereference:
 Queue - otherwise it is redundant to check if Queue is null at line 425

Added a RmFlag function to remvoe the loop flag, this should prevent any errors or wierd behaviour because of the loop flag being used
at mutiple stages in both SmartUnpack and SmartConfigure.

* methods/mirror.cc:
  - include the architecture(s) in the query string as well so
    that the server can make better decisions

More inproved comments about loops.

* apt-pkg/acquire.cc:
  - fix potential divide-by-zero

merged from the debian-experimental2 branch