PERFORCE change 180026 for review

Ivan Voras ivoras at freebsd.org
Tue Jun 22 14:22:04 UTC 2010


On 21 June 2010 03:25, Garrett Cooper <gcooper at freebsd.org> wrote:
> On Sun, Jun 20, 2010 at 6:15 PM, Ivan Voras <ivoras at freebsd.org> wrote:
>> On 21 June 2010 02:58, Julien LAFFAYE <jlaffaye at freebsd.org> wrote:

>>> Why don't you use read_plist(Package *pkg, int fd) from libpkg instead?
>>
>> Should I, in the light of
>> YES_I_KNOW_THE_API_IS_RUBBISH_AND_IS_DOOMED_TO_CHANGE ? I thought the
>> entire lib is to be rightfully scrapped and implemented as something
>> decent.
>
>    Yes, it will be eventually, but there's no real harm in
> implementing this bit with read_plist for the time being though. If
> and when dforsyth's work gets straightened out, a lot of this crud
> will get scrapped and/or properly rewritten.

Just to confirm this: for example, if I need to get the MD5 hash of a
file from this structure, is the accepted practice to scan through the
entire list searching for the file node and then scan the rest of the
list searching for the first comment node that begins with "MD5:"?


More information about the p4-projects mailing list