]> granicus.if.org Git - nethack/commit
build fix (trunk only)
authornethack.rankin <nethack.rankin>
Sun, 3 Feb 2008 06:20:05 +0000 (06:20 +0000)
committernethack.rankin <nethack.rankin>
Sun, 3 Feb 2008 06:20:05 +0000 (06:20 +0000)
commit6968f45024fcaeb67fac8d229029058f3763ed49
tree1459e1b08bdd99046906e6d1d5878965bd7524eb
parent6f0e178368af9e359e91713a3dc606a6a05ed06f
build fix (trunk only)

     The recent SYSCF patch introduced a build problem even though I
haven't attempt to use that new stuff yet.  My compiler complained that
`out' in build_english_list() was used without being initialized, which
in turn caused make to quit.  The compiler was right; only the words==1
case actually set up the output buffer.  Once that buffer was fixed, the
routine to copy a single word was overwriting it on each call instead of
building up via appending as intended.

     I changed the 3 or more case to yield "A, B, or C" like Keni wrote
in his description rather than the "A, B or C" which was being produced.
I'm pretty sure that both forms are considered acceptible; I've always
used the first one with an extra comma in front of and/or.
src/end.c