To comment on the following update, log in, then open the issue:
http://www.openoffice.org/issues/show_bug.cgi?id=109787
                 Issue #|109787
                 Summary|[sw-rtf] unknown regression on frames in RTF import
               Component|Word processor
                 Version|DEV300m73
                Platform|All
                     URL|
              OS/Version|All
                  Status|NEW
       Status whiteboard|
                Keywords|
              Resolution|
              Issue type|DEFECT
                Priority|P3
            Subcomponent|open-import
             Assigned to|mst
             Reported by|mst





------- Additional comments from [email protected] Wed Mar  3 20:08:42 +0000 
2010 -------
it seems i've introduced some regression in the sw RTF import back in cws
odfmetadata3:

rtffly.cxx:1173:
                                0 != ( pFlyCnt = 
(SwTxtFlyCnt*)pTxtNd->GetTxtAttr(
                                                                                
                0, RES_TXTATR_FLYCNT )) &&

in this conditional, the problem is the call to GetTxtAttr.
i've renamed one of the 2 overloaded methods GetTxtAttr, and the remaining one
will never return a RES_TXTATR_FLYCNT.

this call should have been renamed to GetTxtAttrForCharAt, but wasn't.
unfortunately the compiler didn't tell me to change this call, because thanks to
the magic of the C++ type system "0" implicitly converts to a SwIndex instance.

because the code in rtffly.cxx is not exactly readable, i've no idea what
exactly i've broken here.

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