cabinent: add a comment about parm passing

Jeff L lats at yless4u.com.au
Thu Jun 28 04:17:18 CDT 2007


James Hawkins wrote:
> On 6/27/07, Jeff Latimer <lats at yless4u.com.au> wrote:
>> Coverity CID 43 Forward_Null is false.  This adds a comment explaining
>> the NULL pass is ok.
> Why do we need a comment for that?
Mainly because the logic is a bit difficult.  It depends on the length 
being 0 to terminate the for loop and prevent the null reference.  It is 
not obvious and easily missed the first time through. Wine has a number 
of things like that and more comments would make it quicker and safer to 
works with.



More information about the wine-devel mailing list