28 Jun
2007
28 Jun
'07
4:17 a.m.
James Hawkins wrote:
On 6/27/07, Jeff Latimer <lats(a)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.