On Fri, Oct 08, 2010 at 01:39:46PM -0400, Jerry D. Hedden wrote:
> There's a bug in this patch!  This line:
> 
> > +               if (name_cursor == ':') {
> 
> needs to be:
> 
> > +               if (*name_cursor == ':') {
> 
> Also, the next line:
> 
> > +                   tmpbuf = name;
> 
> gives a build warning:
> 
> gv.c:1058: warning: assignment discards qualifiers from pointer target type

Well spotted. Fixed with 3a5b580ccfc4c8c7d52ecd8ccd1431eb99a05a0f

Nicholas Clark

Reply via email to