[devel] [SCM] packages/apt: heads/rework-dynamic-mmap

[Aleksei Nikiforov]

29.01.2020 19:32, Ivan Zakharyaschev пишет:
> 
> On Wed, 29 Jan 2020, Ivan Zakharyaschev wrote:
> 
>> On Wed, 29 Jan 2020, Aleksei Nikiforov wrote:
> 
>>> Вот второй фрагмент изменений под вопросом:
>>>
>>>> @@ -544,18 +562,24 @@ bool
>>> pkgCacheGenerator::ListParser::NewProvides(pkgCache::VerIterator &Ver,
>>>>    #endif
>>>>
>>>>       // Get a structure
>>>> -   unsigned long Provides =
>>> Owner->AllocateInMap(sizeof(pkgCache::Provides));
>>>> -   if (Provides == 0)
>>>> +   const auto Provides =
>>> Owner->AllocateInMap(sizeof(pkgCache::Provides));
>>>> +   const auto idxVersion = Version.empty()
>>>> +                         ? std::experimental::optional<unsigned long>()
>>>> +                         : WriteString(Version);
>>>> +   if (!Provides || (!Version.empty() && !idxVersion))
>>>>          return false;
>>>>       Cache.HeaderP->ProvidesCount++;
>>>>
>>>>       // Fill it in
>>>> -   pkgCache::PrvIterator Prv(Cache,Cache.ProvideP +
>>> Provides,Cache.PkgP);
>>>> +   pkgCache::PrvIterator Prv(Cache,Cache.ProvideP +
>>> *Provides,Cache.PkgP);
>>>>       Prv->Version = Ver.Index();
>>>>       Prv->NextPkgProv = Ver->ProvidesList;
>>>>       Ver->ProvidesList = Prv.Index();
>>>> -   if (Version.empty() == false && (Prv->ProvideVersion =
>>> WriteString(Version)) == 0)
>>>> -      return false;
>>>> +
>>>> +   if (Version.empty() == false)
>>>> +   {
>>>> +      Prv->ProvideVersion = *idxVersion;
>>>> +   }
>>>>
>>>>       // Locate the target package
>>>>       pkgCache::PkgIterator Pkg;
> 
> У меня возник вопрос: а в случае, если Version.empty() истинно, нужно ли
> как-то заполнять Prv->ProvideVersion специально?
> 
> Раньше (то, что минусами отмечено) в таком случае Prv->ProvideVersion
> никак не трогали (потому что происходил shortcut в вычислении &&).
> 

Сложно ответить на этот вопрос, но на всякий случай выставлять особое 
значение "ноль" думаю повредить не должно.

С уважением,
Алексей Никифоров