search

oizma / angleproject

Automatically exported from code.google.com/p/angleproject
Other
0 stars 0 forks source link

ret = yyparse(...) is dropped in PaParseStrings #16

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. build http://hg.mozilla.org/mozilla-central/rev/96de199027d7

What is the expected output?
no warnings about unused variable ‘ret’

What do you see instead?
glslang.cpp:
 In function ‘int PaParseStrings(char**, int*, int, TParseContext&)’:
2683: warning: unused variable ‘ret’

What version of the product are you using?

On what operating system?
on OS X 64 bit 10.6.2 with gcc 4.2.1

Please provide any additional information below.

While 'out' is unused, it seems like yakk would appreciate the failure from 
yyparse being propagated

Original issue reported on code.google.com by timel...@gmail.com on 16 Jul 2010 at 12:53

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by alokp@chromium.org on 9 Aug 2010 at 6:06

GoogleCodeExporter commented 9 years ago 
Fixed in r377.

Original comment by alokp@chromium.org on 9 Aug 2010 at 10:30