First Last Prev Next    No search results available      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
Show dependency graph
Bug 228725 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

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







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) [edit]
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
First Last Prev Next    No search results available      Search page      Enter new bug