Web lists-archives.com

Re: [PATCH 34/34] wt-status: release strbuf after use in wt_longstatus_print_tracking()




René Scharfe <l.s.r@xxxxxx> writes:

> Am 06.09.2017 um 21:51 schrieb Junio C Hamano:
>> Rene Scharfe <l.s.r@xxxxxx> writes:
>> 
>>> If format_tracking_info() returns 0 only if it didn't touch its strbuf
>>> parameter, so it's OK to exit early in that case.  Clean up sb in the
>>> other case.
>> 
>> These two "if"s confuse me; perhaps the first one is not needed?
>
> Yes, removing it looks like the best way to make that sentence clearer.
> Another would be to replace "only if" with "then".

Either makes it understandable. Thanks.