Bugs item #1119733, was opened at 2005-02-10 11:02
Message generated for change (Comment added) made by brettporter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=116035&aid=1119733&group_id=16035

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Brett Porter (brettporter)
Assigned to: Maarten Coene (maartenc)
Summary: whitespace problems with pretty printing

Initial Comment:
when using pretty printing whitespace is ignored at
start of a new line (should be trimmed to one when
content is rewrapped), and ignored after end of a tag
(eg <code>foo</code> bar - appears as <code>foo</code>bar)

----------------------------------------------------------------------

>Comment By: Brett Porter (brettporter)
Date: 2005-03-20 14:05

Message:
Logged In: YES 
user_id=108080

Thanks Maarten, that seems to have worked for the first
case. The latter on is something like:

source:
<p>
  Some text
  that is wrapped
</p>

dest:
<p>
  Some textthat is wrapped
</p>

Notice the missing space between text and that.

Another issue I noticed (which may be by design) is that
tags are wrapped even if the character before them is not
whitespace. eg:

<p>some text (<code>foo</code>)</p>

becomes

<p>
  some text (
  <code>foo</code>)
</p>

I'd expect the <code> not to wrap in this case.

Thanks!

----------------------------------------------------------------------

Comment By: Maarten Coene (maartenc)
Date: 2005-03-20 00:21

Message:
Logged In: YES 
user_id=178745

Brett,

I've made another padding fix to XMLWriter (latest revision
is now 1.82) which should at least fix your second problem.
Could you try it again?

I'm not sure what you mean with "whitespace is ignored at
start of new line (should be trimmed to one)". Could you
given an example?

regards,
Maarten

----------------------------------------------------------------------

Comment By: Brett Porter (brettporter)
Date: 2005-02-18 09:14

Message:
Logged In: YES 
user_id=108080

though I picked up new code, it appears to be failing still.
I will try a test case.

----------------------------------------------------------------------

Comment By: Maarten Coene (maartenc)
Date: 2005-02-18 07:35

Message:
Logged In: YES 
user_id=178745

Hmz, my tests are successful. There could be a delay on the 
anonymous CVS. 

Could you try again with today's CVS? And if it still fails, 
could you provide me some code that illustrates the problem?

thanks,
Maarten

----------------------------------------------------------------------

Comment By: Brett Porter (brettporter)
Date: 2005-02-17 11:38

Message:
Logged In: YES 
user_id=108080

as far as I can tell, both cases still exist after
rebuilding from today's CVS.

----------------------------------------------------------------------

Comment By: Maarten Coene (maartenc)
Date: 2005-02-17 08:17

Message:
Logged In: YES 
user_id=178745

I think I've fixed this in CVS, could you verify this?

thanks,
Maarten

----------------------------------------------------------------------

Comment By: Maarten Coene (maartenc)
Date: 2005-02-17 07:44

Message:
Logged In: YES 
user_id=178745

I'll take a look at it.

Maarten

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=116035&aid=1119733&group_id=16035


-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
dom4j-dev mailing list
dom4j-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dom4j-dev

Reply via email to