Re: svn commit: r1144351 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPDocumentHandler.java

2011-07-12 Thread mehdi houshmand
Hi Jeremias, Just to satiate your curiousity, in the modca spec, page 96, it says "A medium map remains in effect until another medium map is selected or the end of the document is reached.", which means the previous implementation, storing the lastMediumMap, would be perfectly valid. Mehdi On 8

Re: svn commit: r1144351 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPDocumentHandler.java

2011-07-12 Thread Jeremias Maerki
Hi Mehdi Sure, but imagine someone cuts an AFP appart at some page group boundary to make smaller print streams. The second part might not contain the IMM instruction and might therefore end up printed wrong. The splitter would have to have the intelligence to scan the whole AFP and insert new IMM

Re: svn commit: r1144351 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPDocumentHandler.java

2011-07-12 Thread mehdi houshmand
Hi Jeremias, Yes, I agree, I wasn't trying to make a point, or even contest the commit. You said in the commit message that you weren't sure why you did that originally, I just wanted to illustrate why. Both ways make sense. Mehdi On 12 July 2011 10:40, Jeremias Maerki wrote: > Hi Mehdi > > Sur

Re: svn commit: r1144351 - /xmlgraphics/fop/trunk/src/java/org/apache/fop/render/afp/AFPDocumentHandler.java

2011-07-12 Thread Jeremias Maerki
Right. I guess that must have been it. Premature optimization! ;-) On 12.07.2011 12:10:59 mehdi houshmand wrote: > Hi Jeremias, > > Yes, I agree, I wasn't trying to make a point, or even contest the > commit. You said in the commit message that you weren't sure why you > did that originally, I ju