2013/6/28 Robert Millan <r...@debian.org>:
>> -               if (!*p || *p == '#')
>> +               if (!*p || *p == '#') {

Silly me, looks like I can't read C anymore. Please ignore what I said
about null dereference. Anyway, I guess it can be simplified this way?

> if (!*p)
>  continue;
>
> if (*p == '#')
>  {
>     ...
>  }

--
Robert Millan


-- 
To UNSUBSCRIBE, email to debian-bsd-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/caofdtxna7cyx3oia6yjscg8j8nu7dtp9okm13qcqx-rt2jv...@mail.gmail.com

Reply via email to