Subject: Re: patch to fix bug 1207
From: Matti Picus (matti@picus.org)
Date: Wed Sep 12 2001 - 09:42:55 CDT
Sorry, I meant bug 1211 which relates back to 1207. If you don't like the
patch, feel free to reject it. I was kind of using it as a testbed for
parsing using NextToken() and a hash.
Matti
"Stumbling, bumbling, somehow the job gets done"
At 02:41 PM 12/9/2001 +0200, hub wrote:
>According to Matti Picus <matti@picus.org>:
> > The RTF importer was not ignoring leading spaces properly. This patch
> fixes
> > bug 1207 by:
> > 1. Use NextToken() within the font table parsing
> > 2. Fix NextToken() to ignore leading spaces (spaces between keywords)
> >
> > It also removes parsing of the "panose" keyword, which was parsed but not
> > used. This keyword, or any that includes a non-numerical parameter, will
> > break NextToken (actually, ReadKeyword()) if not enclosed in braces
> > {\*\panose020206080507}.
> >
> > The patched file was tested against the RTF files in the test
> directory, as
> > well as bug1207.rtf
>
>Sorry, but given that the bug 1207 is closed and does not match this, I
>don't really understand the exact problem
>
>
>Hub
This archive was generated by hypermail 2b25 : Wed Sep 12 2001 - 08:52:02 CDT