markcox / snappy

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

Error compiling with enabled debug mode in file snappy.cc #58

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Compiling firefox nightly code (using the snappy) with debug mode enabled 
(ac_add_options --enable-debug on mozconfig)

What is the expected output? What do you see instead?
Finish the compilation process.

Errors:
/mozilla-central/other-licenses/snappy/src/snappy.cc:504: error: extra ‘;’
/mozilla-central/other-licenses/snappy/src/snappy.cc:590: error: extra ‘;’

What version of the product are you using? On what operating system?
Snappy v1.0.4, September 15th 2011.
Mozilla Firefox  12
gcc-4.3.real (Ubuntu 4.3.5-3ubuntu1) 4.3.5
Ubuntu 10.10

Please provide any additional information below.
I reported the bug on bugzilla 
(https://bugzilla.mozilla.org/show_bug.cgi?id=715113).
I submitted a patch file there. Here attached the patch file.

Original issue reported on code.google.com by rogerio.rag on 4 Jan 2012 at 4:40

Attachments:

GoogleCodeExporter commented 9 years ago
Hi,

This is not an error in Snappy; it is not illegal C++, but it probably stops 
Firefox compile anyway due to -Werror. (I haven't checked that they actually 
compile with -Werror.) However, these warnings were already reported in bug 57, 
and the warning was fixed in r56.

Original comment by se...@google.com on 4 Jan 2012 at 4:46