To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=79411
                 Issue #|79411
                 Summary|rsc: coverity warnings
               Component|gsl
                 Version|OOo 2.2.1
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|PATCH
                Priority|P3
            Subcomponent|code
             Assigned to|pl
             Reported by|cmc





------- Additional comments from [EMAIL PROTECTED] Mon Jul  9 14:39:41 +0000 
2007 -------
I decided to bundle the coverity warnings for rsc into one patch.

In effect, 
a) use of rc as unit, easy fix, though likely false positive
b) deadcode in RscString::IsValueDefault where we have
if( !bStrEmpty || !bDefStrEmpty ){
...
}
else if( !(bStrEmpty || bDefStrEmpty) ){
...
}

we're never going into that else if leg
c) and uses of pTC where we previously tested it against NULL for using it, but
afterwards pass it down to MakeName which makes no such test.

---------------------------------------------------------------------
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]

Reply via email to