Commit: Re: RTF importer mods (PATCH)

CVS: ----------------------------------------------------------------------
CVS: Enter Log. Lines beginning with `CVS:' are removed automatically
CVS:
CVS: Committing in .
CVS:
CVS: Modified Files:
CVS: src/wp/impexp/xp/ie_imp_RTF.cpp src/wp/impexp/xp/ie_imp_RTF.h
CVS: ----------------------------------------------------------------------
Roland Kay's patches to help 8001 and 8813

Cheers

Martin

>
>
>
> Hi guys,
>
> #8813:
> I've just submitted some more modifications to the RTF
> importer on bugzilla[1]. These add support for the \deff
> keyword and fix a problem with the importers handling of
> multiple characters sets in a single document (something
> that MSWord exported documents in Asia contain a lot of).
>
> #8001:
> I've also posted the patch[2] from my last email which
> addresses the clipboard issue.
>
> While doing this I noticed what looks suspiciously like a
> missing break in a select construct. See line 4124 in the
> patched version of abi/src/wp/impexp/xp/ie_imp_RTF.cpp.
> I've commented it in the patch, would someone mind having a
> look? Before the patch the case for the "clcbpat" command
> was falling through to the handler for the "dn". With the
> patch it falls through to the "deff" handler. I assume both
> are incorrect?
>
> [1] http://bugzilla.abisource.com/show_bug.cgi?id=8813
> [2] http://bugzilla.abisource.com/show_bug.cgi?id=8001
>
> Best wishes,
>
> R.
>
>
>
Received on Thu Jun 2 10:52:05 2005