Re: [PATCH 2/2] silence comment5 warnings

Hello Reini,

I don't agree with this patch. The code that caused the warning messages
is valid C. This, for example, is a valid C program:

==========================================================================
#include <stdio.h>

#define hello "hello world" /* some nice
                               greeting */
int main () {
  printf (hello "\n");
}
==========================================================================

Or even this:
==========================================================================
#include <stdio.h>

#define hello "hello world" /\
* some nice
                               greeting *\
/
int main () {
  printf (hello "\n");
}
==========================================================================

I prefer much to fix the tools so that they understand the C syntax that
we use, rather than to add a constraint that we would have to remember
in future developments.

Done by modifying varbrace.c.

Sam wrote:
> I would rather eliminate comment5 altogether, rather than mutilate the
> code to pacify it.

Eliminating comment5 would not help here, since the warning comes from
'varbrace', and comment5's processing does not modify the lines that
trigger the warning.

Bruno


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
clisp-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/clisp-devel