Re: The warns-a-thon continues...

> :*) Fixed some sprintf() format type mismatches
> 
> Don't change all mechanically.

I looked over these for a few minutes and couldn't see any harm in,
what I thought, was correcting them.

> :-	    sprintf(buf, "$%c", node->nd_nth);
> :+	    sprintf(buf, "$%ld", node->nd_nth);
> 
> %lc

I'm not an expert, or any where close, but if I'm reading node.h
right, nd_nth is a long.  Shouldn't it be %ld?

> :-	    sprintf(buf, "0%o", NUM2INT(v));
> :+	    sprintf(buf, "0%ld", NUM2INT(v));
> 
> %lo

Corrected, thank you.

> :-	    sprintf(buf, "0x%x", NUM2ULONG(v));
> :+	    sprintf(buf, "0x%ld", NUM2ULONG(v));
> 
> %lx

Corrected, thank you.

> :-		if (c >= '0' && c <= '7' || c == '_') {
> :+		if (c >= '0' && (c <= '7' || c == '_')) {
> 
> 		if ((c >= '0' && c <= '7') || c == '_') {

Corrected, thank you.  Updated patch attached.  -sc

-- 
Sean Chittenden