rtf ignoring TableBody et al

2005-09-19 Thread Matthew L Daniel
I am using trunk code and with an input FO that produces excellent PDF output, I am seeing the following messages about lots of ignored instances. After checking RTFHandler.java, it seems they are omitted from the very large instance-switch in "invokeDeferredEvent". My question is if this is "in

Re: rtf ignoring TableBody et al

2005-09-19 Thread Andreas L Delmelle
On Sep 20, 2005, at 00:28, Matthew L Daniel wrote: I am using trunk code and with an input FO that produces excellent PDF output, I am seeing the following messages about lots of ignored instances. After checking RTFHandler.java, it seems they are omitted from the very large instance-switch in

Re: rtf ignoring TableBody et al

2005-09-19 Thread Jeremias Maerki
I added the warning because I found a case where a block-container wasn't handled as it should have been: http://svn.apache.org/viewcvs?rev=278720&view=rev The warning is an indicator that there are still a few things in RTF support that need fixing. On 20.09.2005 00:28:56 Matthew L Daniel wrote:

Re: rtf ignoring TableBody et al

2005-09-20 Thread Matthew L Daniel
> The least this would do is avoid a number of unnecessary calls to > instanceof. You're on the right track, and maybe that "switch" impl would be an 80/20 win over the cost of the Right Way. But that huge if-instance case is exactly what the Visitor pattern was created to solve. I haven't spent

Re: rtf ignoring TableBody et al

2005-09-20 Thread Matthew L Daniel
> The warning is an indicator that there are still a few things in RTF > support that need fixing. So is there someone who owns this class? How long has it been out-of-sync - that is, is it in the same condition as TXTRenderer? Also, while this is probably not the correct forum for this question,

Re: rtf ignoring TableBody et al

2005-09-20 Thread Andreas L Delmelle
On Sep 20, 2005, at 17:40, Matthew L Daniel wrote: Hi Matthew, The least this would do is avoid a number of unnecessary calls to instanceof. You're on the right track, and maybe that "switch" impl would be an 80/20 win over the cost of the Right Way. But that huge if-instance case is exactl

AW: rtf ignoring TableBody et al

2005-09-20 Thread Peter Herweg
Jeremias Maerki wrote: > I added the warning because I found a case where a block-container > wasn't handled as it should have been: > http://svn.apache.org/viewcvs?rev=278720&view=rev Good idea, in my opinion. Regards, Peter Herweg

AW: rtf ignoring TableBody et al

2005-09-20 Thread Peter Herweg
TableBody et al > The warning is an indicator that there are still a few things in RTF > support that need fixing. So is there someone who owns this class? How long has it been out-of-sync - that is, is it in the same condition as TXTRenderer? Also, while this is probably not the correct for

AW: rtf ignoring TableBody et al

2005-09-20 Thread Peter Herweg
f: Re: rtf ignoring TableBody et al On Sep 20, 2005, at 00:28, Matthew L Daniel wrote: > I am using trunk code and with an input FO that produces excellent PDF > output, I am seeing the following messages about lots of ignored > instances. > > After checking RTFHandler.java, it seems

Re: rtf ignoring TableBody et al

2005-09-20 Thread Jeremias Maerki
On 20.09.2005 17:50:03 Matthew L Daniel wrote: > > The warning is an indicator that there are still a few things in RTF > > support that need fixing. > > So is there someone who owns this class? No. There are no class owners in Apache projects. > How long has it been > out-of-sync - that is, i