On Thursday 10 July 2008 10:44:51 NotFound wrote: > On Wed, Jul 9, 2008 at 9:33 PM, Andrew Johnson <[EMAIL PROTECTED]> wrote: > > The other warnings I get on Solaris are just these: > > > > "src/stm/backend.c", line 969: warning: statement not reached > > "./src/pmc/nci.pmc", line 486: warning: statement not reached > > "src/encodings/utf16.c", line 326: warning: statement not reached > > "src/encodings/utf16.c", line 455: warning: statement not reached > > "src/encodings/utf16.c", line 479: warning: statement not reached > > "src/encodings/utf16.c", line 502: warning: statement not reached > > "src/encodings/utf16.c", line 525: warning: statement not reached > > Please check those with r29234
Those warnings are now gone, thanks. > Except the src/pmc/nci.pmc line 486, there is not such line in the > file, and I'm unable to locate the problem. If you look in the generated src/pmc/nci.c file at line 486, it has a #line directive that purports to point back to line 486 of src/pmc/nci.pmc (there's evidently another bug there in the Pmc2c emit process). A few lines before however, it also has #line 266 "./src/pmc/nci.pmc" and from that and the surrounding comments its reasonably obvious that the issue is actually at line 272 of nci.pmc — the RETURN line is superfluous since neither branch of the if statement returns. - Andrew -- Talk is cheap. Show me the code. -- Linus Torvalds
