http://bugzilla.novell.com/show_bug.cgi?id=496307 Summary: koffice-1.6.3-214.15: 2 * broken case values Classification: openSUSE Product: openSUSE 11.2 Version: Factory Platform: Other OS/Version: openSUSE 11.1 Status: NEW Severity: Normal Priority: P5 - None Component: Other AssignedTo: bnc-team-screening@forge.provo.novell.com ReportedBy: dcb314@hotmail.com QAContact: qa@suse.de Found By: --- I just tried to build Factory package koffice-1.6.3-214.15 and the build said textelement.cc:414: warning: case label value exceeds maximum value for type textelement.cc:415: warning: case label value exceeds maximum value for type The source code is case 0x1D574: character = 0x2111; break; case 0x1D579: character = 0x211C; break; I agree with the compiler. This code will never be executed because the return value of the unicode() function is only two bytes in size. While it is certain this code is dead code, perhaps it would benefit from some rework to ensure the original programmer hasn't committed a coding error. I also notice that there is no default case in the switch. This may be viewed as a basic coding error by many students of software engineering. -- Configure bugmail: http://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.