DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=22277>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=22277

IllegalArgumentException in setOutputLineIncrement()





------- Additional Comments From [EMAIL PROTECTED]  2003-08-13 07:15 -------
I believe Remy's commit from earlier today ("Don't merge SMAP entries in the 
outptStartLines aren't consecutive") fixes the IllegalArgumentException 
problem (thanks, Remy!).  If that's right (I've tested it, but then I tested 
my original patch as well, so I'm not taking that as a guarantee), it means my 
workaround patch can be removed without IllegalArgumentException recurring.  
On the other hand the workaround patch also keeps the outputStartLine=0 
LineInfos out of the SMAP, so taking it out before we've fixed the root cause 
of outputStartLine=0 could be considered a regression, on the grounds that an 
incorrect SMAP is worse than an incomplete one.

Also, if Jasper were to start SMAPping page directives, then

        TemplateText<page import="org.foo.*">

would generate an IllegalArgumentException if it weren't for Remy's fix.  This 
means that IllegalArgumentException is a bug in its own right, and not just a 
side effect of the outputStartLine=0 problem.  So to keep things clear, I 
think we should use this ticket to track IllegalArgumentException only, and 
open new tickets as needed to track outputStartLine=0.

I'll close this ticket in a few days if there are no objections to this 
approach, and no reason to doubt Remy's commit.  If someone else wants to 
close it sooner, that's fine with me.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to