Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 228725
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gentoo's Team for Core System packages <base-system@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Nico R. <n-roeser@gmx.net>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
lzma-4.32.6-extra-semicolon.patch Patch to remove an extra semicolon from lzma-4.32.6. patch Nico R. 2008-06-21 12:36 0000 432 bytes Details | Diff
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 228725 depends on: Show dependency tree
Bug 228725 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.






View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug

Description:   Opened: 2008-06-21 12:34 0000 
In app-arch/lzma-utils-4.32.5 and app-arch/lzma-utils-4.32.6, there is an extra
semicolon after the closing brace in
src/sdk/7zip/Compress/LZMA/LZMAEncoder.cpp, line 185. This is not valid ISO
C++, as far as I know.

After removing the extra semicolon, the code compiles with ‘-pedantic’ in
CXXFLAGS.

------- Comment #1 From Nico R. 2008-06-21 12:36:46 0000 ------- 
Created an attachment (id=157899) [details]
Patch to remove an extra semicolon from lzma-4.32.6.

Code compiles with gcc-4.3.1 with -pedantic after using this patch.

------- Comment #2 From SpanKY 2008-06-22 01:17:50 0000 ------- 
source code correctness (-pedantic) is not something we care about ... these
kind of things go upstream

upstream has already fixed this though and the patch is trivial, so i'll add it
Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug