It seems like I somehow missed the fact that patch was already accepted on the first try, however I still ended up rebasing the patch against the origin.
I'll resend it anyways, since the commit message needs to be changed.
On Thu, Sep 11, 2008 at 10:34 AM, Dylan Smith dylan.ah.smith@gmail.comwrote:
Previously the control words in skipped groups were being processed by the read hook on the RTF parser. By moving this code into the class callbacks for the parser, the skipped groups actually remain skipped.
An example of a skipped group is a optional destination that isn't recognized by the parser. The RTF specifications say to ignore these groups, and native richedit controls act accordingly.
This patch fixes bug 15227 which was caused by the \itap4 control word in a destination which caused an unterminated table, since this control word changes the nesting level by starting nested tables to reach that nesting level.
Changes for try 2: Added in a destination callback for nesttableprops to avoid it being ignored. Otherwise this patch would cause a regression where the default table properties would be used for nested tables.
dlls/riched20/editor.c | 3 +++ dlls/riched20/reader.c | 10 ++++++++++ dlls/riched20/rtf.h | 1 + 3 files changed, 14 insertions(+), 0 deletions(-)
-
Dylan Smith