pkgsrc-WIP-review archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: pkg_select upgrade



Build failed on NetBSD/amd64.

--------------------
#   compile  pkg_select-20050315/ftpio.o
cc -O2 -g -g -I. -DPKGSRCBASE=\"/home/yui/pkgsrc/pkgsrc\" 
-DPKGDB=\"/var/db/pkg\" -DMAKE=\"/usr/bin/make\" 
-DCONFPATH=\"/usr/pkg/etc/pkg_select.conf\" 
-DMIRRORS=\"/usr/pkg/share/pkg_select\"  -Wall -Wstrict-prototypes 
-Wmissing-prototypes -Wpointer-arith -Wno-sign-compare -Wno-traditional 
-Wno-uninitialized -Wreturn-type -Wswitch -Wshadow -Wcast-qual -Wwrite-strings  
-Werror       -c    ftpio.c
ftpio.c: In function `expect':
ftpio.c:353: warning: subscript has type `char'
*** Error code 1

Stop.
-------------------

I think we have two options.
Casting a variable to int in pkgselect source code or in ctype.h

Option #1 would be like this:
In ftpio.c file,
-                               if (ftprc && isdigit(buf[match.rm_so+1]))
+                               if (ftprc && isdigit((int)buf[match.rm_so+1]))

And here is option #2:
In /usr/src/include/ctype.h file,
-#define isdigit(c)      ((int)((_ctype_ + 1)[(c)] & _N))
+#define isdigit(c)      ((int)((_ctype_ + 1)[(int)(c)] & _N))


IMHO, testing if it is digit or not should be working on a character type.
Am I thinking wrong?


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
pkgsrc-wip-review mailing list
pkgsrc-wip-review%lists.sourceforge.net@localhost
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-review



Home | Main Index | Thread Index | Old Index