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=24775>.
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=24775

[PATCH] padding-left in PDF Renderer





------- Additional Comments From [EMAIL PROTECTED]  2003-11-19 09:07 -------
Hi Glen,

sorry for not specifying -u on diff, will do in future. Also I think the line 
endings may be wrong in those diff files, i.e. CRLF instead of LF. Let me know 
if you want me to resubmit.

As for AbstractRenderer, the bits I changed had nothing to do with PDF 
specific or padding-left stuff. Take a look at 
renderBlock/renderBlocks/renderBlockViewPort if you follow the code through 
you will see in several place comments about save IPD/BPD for block container. 
Regardless of whether specific renderers are overriding one or more of these, 
there are several bits that are unnecessary.

I can understand your caution here. If preferred you can simply apply the BLM 
and PDFRenderer patches and padding-left will work. The changes to 
AbstractRenderer were simply a clean-up exercise which made the code IMHO a 
little easier to understand. I probably should have submitted these changes as 
a separate patch.

Reply via email to