Greg Stein wrote:
> On Mon, Dec 15, 2008 at 10:52, Hyrum K. Wright
> <hyrum_wright_at_mail.utexas.edu> wrote:
>> ...
>>>> This patch seems to be pretty straightforward, but since it touches code used to
>>>> interact with older WC formats, and I don't have a very good way of testing that
>>>> code, I'm posting it here for review before committing.
>
> It looks good. I have one nit, and then would suggest posting a copy
> of the patch with whitespace-ignored. Then the changes will be even
> easier to review.
>
> Tho, really: I'd say go ahead and commit it, too. Like I say, it looks
> good... the patch-post would just make another double-verification a
> little bit easier.
>
>> ...
>> + *modified_p = !(local_propchanges->nelts == 0);
>
> Making life difficult for yourself?
>
> *modified_p = local_propchanges->nelts > 0;
Updated to your suggestion, though I suppose != 0 would work just as well.
r34767.
-Hyrum
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=985830
Received on 2008-12-17 17:03:38 CET