Re: RFC: No recursion into language subdirectories

[Philipp Thomas <pthomas at suse dot de>]

* Zack Weinberg (zack@wolery.cumb.org) [20000620 08:17]:

> The appended patch changes the Makefiles so we never recurse into
> language subdirectories.

Marvellous ;-))

> This is either a catastrophe, or no big deal, depending how you look
> at it.  (The 'ANSI C forbids ...' warnings need to be looked at;
> that's not just the usual "works on all sufficiently Unixy platforms"
> issues, but use of GCC-specific extensions.)

As Gaby already said, frontends other then C aren't supposed to build with
anything else than gcc, so there's no big deal.

I'll have a look at the warnings if Kaveh doesn't beat me to it.

>      65 comparison between signed and unsigned
>      61 pointer targets in assignment differ in signedness
>      51 comma at end of enumerator list
>      39 pointer targets in passing arg ??? of `???' differ in signedness
>      17 promoted argument `???' doesn't match prototype

These are rather tedious to fix but no problem.

>     449 function macro ??? must be used with arguments in traditional C
>      52 Use of ANSI string concatenation

These I'd tend to ignore. There's no need to cater K&R.

>     223 ANSI C forbids braced-groups within expressions
>     178 ANSI C forbids constructor expressions
>     141 ANSI C forbids conditional expr with only one void side

Well, and all these cases simply get put into __extension__ blocks.

All and all, this is IMO once a case where the ability to switch on/off
specific warning flags in the source would be *real* handy. Mark, I'm really
going to search the mail archives for your proposal and try to reinitiate
the discussion.

Philipp

-- 
Philipp Thomas <pthomas@suse.de>
Development, SuSE GmbH, Schanzaecker Str. 10, D-90443 Nuremberg, Germany

#define NINODE  50              /* number of in core inodes */
#define NPROC   30              /* max number of processes */
 	-- Version 7 UNIX for PDP 11, /usr/include/sys/param.h