Subject: Re: CVS commit: src
To: Tom Spindler <dogcow@babymeat.com>
From: Jason Thorpe <thorpej@shagadelic.org>
List: source-changes
Date: 10/12/2006 11:39:15
On Oct 12, 2006, at 11:31 AM, Tom Spindler wrote:

> On Thu, Oct 12, 2006 at 04:46:57AM +0000, Jason R Thorpe wrote:
>> 	src/common/lib/libprop: prop_number.3 prop_number.c
>> cvs rdiff -r1.7 -r1.8 src/common/lib/libprop/prop_number.c
>
> Note that this causes the build to break while linting:

I'll fix this today / tonight.

>
> #   compile  libprop/prop_number.ln
> CC=/home/dogcow/work/nbobj/tools/bin/i386--netbsdelf-gcc /home/ 
> dogcow/work/nbobj/tools/bin/i386--netbsdelf-lint -chapbxzFS -d /home/ 
> dogcow/work/nbobj/dd/usr/include  -D_LIBPROP -D_REENTRANT -I/src/lib/ 
> libc/include    -i /src/common/lib/libprop/prop_number.c
> /src/common/lib/libprop/prop_number.c(307): syntax error [249]
> /src/common/lib/libprop/prop_number.c(308): undefined struct/union  
> member: pnv_is_unsigned [101]
> /src/common/lib/libprop/prop_number.c(304): warning: argument val  
> unused in function prop_number_create_integer [231]
> /src/common/lib/libprop/prop_number.c(323): syntax error [249]
> /src/common/lib/libprop/prop_number.c(324): undefined struct/union  
> member: pnv_is_unsigned [101]
> /src/common/lib/libprop/prop_number.c(320): warning: argument
>  val unused in function prop_number_create_unsigned_integer [231]
> /src/common/lib/libprop/prop_number.c(484): warning: constant in  
> conditional context [161]
> /src/common/lib/libprop/prop_number.c(484): warning: expression has  
> null effect [129]
> /src/common/lib/libprop/prop_number.c(506): warning: constant in  
> conditional context [161]
> /src/common/lib/libprop/prop_number.c(506): warning: expression has  
> null effect [129]
> /src/common/lib/libprop/prop_number.c(532): syntax error [249]
> /src/common/lib/libprop/prop_number.c(533): undefined struct/union  
> member: pnv_is_unsigned [101]
> [deletia]
> *** Failed target:  prop_number.ln
> nbmake: stopped in /src/lib/libprop
>
> The "obvious" workaround is to add MKLINT=no to
> src/common/lib/libprop/Makefile.inc, but having lint understand
> the newfangled c9x constructs would probably be better.
>
> In addition - if any of the libprop stuff ever gets pulled into
> anything in tools/, that might cause difficulty for crosscompilation
> on machines that only have ANSI C compilers.

-- thorpej