Re: lynx-dev lynx2.8.2dev.18

[Greg Marr]

At 04:33 PM 3/4/99 , gil wrote:
>In a recent note, address@hidden said:
>> I was more concerned about making an array initialized with enums (but we've
>> been doing that under WWW).  Expect some minor compiler problems (i.e.,
fixes
>> that will be a few lines of code).
>>   
>It just broke on OS/390:
>
>    ERROR CBC3275 ../../lynx2-8-2/src/LYKeymap.h:145
>        Unexpected text ',' encountered.
>    FSUM3065 The COMPILE step ended with return code 12.
>    FSUM3017 Could not compile ../../lynx2-8-2/src/LYMap.c.
>        Correct the errors and try again.
>    gmake[1]: *** [LYMap.o] Error 3
>
>Bummer.  It's a comma between the enumerator-list and the "}".
>ANSI seems not to allow this.  Sure makes ending an enumerator-list
>inside an ifdef a pain.  Since the list unconditionally begins with
>LYK_UNKNOWN, the problem could be solved by writing all subsequent
>entries with the comma in front of the enumerator.  Ugly.  But should
>I try a patch?

Another method that I've used is to put something like UNUSED just before the
}.
This requires the least amount of changes, and also doesn't break if the
*first* item in the list gets #ifdef'ed out, or if the list gets sorted.

-- 
Greg Marr
address@hidden
"We thought you were dead." 
"I was, but I'm better now." - Sheridan, "The Summoning"