On Tue, Dec 10, 2019 at 06:23:31PM +0300, Aleksei Nikiforov wrote:
> Update types of other variables as well.

What's the rationale for changing types of *all* these fields,
and for choosing architecture-dependent "unsigned long"
as the new type instead of "unsigned short"?

> ---
>  apt/apt-pkg/pkgcache.h | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/apt/apt-pkg/pkgcache.h b/apt/apt-pkg/pkgcache.h
> index 05a63bc..6b44522 100644
> --- a/apt/apt-pkg/pkgcache.h
> +++ b/apt/apt-pkg/pkgcache.h
> @@ -175,13 +175,13 @@ struct pkgCache::Header
>     unsigned long OptionsHash;
>     
>     // Size of structure values
> -   unsigned short HeaderSz;
> -   unsigned short PackageSz;
> -   unsigned short PackageFileSz;
> -   unsigned short VersionSz;
> -   unsigned short DependencySz;
> -   unsigned short ProvidesSz;
> -   unsigned short VerFileSz;
> +   unsigned long HeaderSz;
> +   unsigned long PackageSz;
> +   unsigned long PackageFileSz;
> +   unsigned long VersionSz;
> +   unsigned long DependencySz;
> +   unsigned long ProvidesSz;
> +   unsigned long VerFileSz;
>     
>     // Structure counts
>     unsigned long PackageCount;

-- 
ldv