Re: Patch for XML pretty printer

I haven't looked at the patch, but it might be easy to support both behavior
with short tag <foo/>  and <foo></foo> using a boolean attribute, or 
flag for the executable.

This way, this won't break existing programs


Colin Adams wrote:
> I would say this might break existing programs (if anyone is using
> this to write legacy xhtml, for instance, for sending to old
> browsers). Perhaps consult on gobo-users list first?
>
> 2009/9/3 Jann Röder <[email protected]>:
>   
>> Hi,
>> I wrote a patch for the XML pretty printer to make it use the short form for
>> tags that don't have content or children, i.e. <tag/> instead of <tag></tag>
>> .
>> See attachment.
>>
>> I also noticed, that the pretty printer does not output the XML descriptor
>> (this: <?xml version="1.0" ?>). I'm not sure what's the best way to fix
>> this. Simply adding code in the on_xml_descriptor feature doesn't work
>> because it is never called, or I don't know what to do to have it called.
>>
>> Anyway in the meantime I'd be glad about some feedback regarding my patch.
>>
>> Jann
>>
>>
>> ------------------------------------------------------------------------------
>> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
>> trial. Simplify your report design, integration and deployment - and focus
>> on
>> what you do best, core application coding. Discover what's new with
>> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
>> _______________________________________________
>> gobo-eiffel-develop mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/gobo-eiffel-develop
>>
>>
>>     
>
>
>
>   


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july