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

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Works For Me
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-05-19 05:21

Message:
Logged In: YES 
user_id=108080

I haven't tried. The combination of problems between the
dom4j upgrade and jaxen upgrade has just broken too many
things, so I've given up and gone back to 1.4... sorry. I
think it's reasonable to close this and reopen if someone
finds it is still a problem.

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

Comment By: Maarten Coene (maartenc)
Date: 2005-05-11 21:24

Message:
Logged In: YES 
user_id=178745

Brett,

are you still having this whitespace problem with wrapped text?
I tried several ways to reproduce this problem, but I didn't
succeed.

regards,
Maarten

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

Comment By: Brett Porter (brettporter)
Date: 2005-03-20 04: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-19 14: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-17 23: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-17 21: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 01: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-16 22: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-16 21: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


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
dom4j-dev mailing list
dom4j-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dom4j-dev

Reply via email to