To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=100083 Issue #|100083 Summary|Suspicious semicolon in old-style system dictionaries |lookup code Component|lingucomponent Version|DEV300m42 Platform|Opteron/x86_64 URL| OS/Version|Unix, X11 Status|UNCONFIRMED Status whiteboard| Keywords| Resolution| Issue type|PATCH Priority|P4 Subcomponent|other Assigned to|iss...@lingucomponent Reported by|dtardon
------- Additional comments from [email protected] Tue Mar 10 08:55:59 +0000 2009 ------- i.e. there is the following construct in file lingucomponent/source/lingutil/lingutil.cxx:194 (GetOldStyleDicts()): if (nIndex != -1); sRegion = sChunk.copy( nIndex, sChunk.getLength() - nIndex ); The indentation of the second line suggests the code should be conditional, but, having looked at the code, it seems to me the condition is always true anyway. I'm attaching two patches: one to get the right conditional behaviour and one to simplify the code, given that there should be no condition. --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
