List parsing is incorrect

[Benoit Garret]

The note parsing code is incorrect.

The "characters" method in a org.xml.sax.helpers.DefaultHandler class is not guaranteed to return the whole string between two tags (eg. if there is a newline in the string).

This means that something like this (hope it won't get too mangled by launchpad):

list
   list-item Test newline newline end-list-item
end-list

will get two bullets, one before the "Test" string and another before an empty line.

[Olivier Bilodeau (plaxx)]

related to bug #364850

in this bug, the parser does send the special & character in another call to characters() so reading this bug description showed me the right way

I'll use StringBuilder for every element parsed by NoteHandler and NoteContentHandler.

[Olivier Bilodeau (plaxx)]

forget what I said about the StringBuilder, this will only work for little meta tags (like title), it won't apply to the note-content since i'm already adding pieces of string to a builder.

What I'm trying to do right now is to keep some basic state in characters() and do the setSpan() in endElement(). I haven't figured the right thing to do with lists yet but the rest is dealt with.

Still, I'm afraid that this will introduce bugs I haven't thought of..

[Olivier Bilodeau (plaxx)]

Attached is my testcase for this bug.

[Olivier Bilodeau (plaxx)]

Just committed a fix for this issue in main/. It worked for the test note attached to this bug and a bunch of my most complicated personal notes.

To be clear: not only list parsing was affected. I modified the handing on all tags and like I said in the commit message, I'm afraid I have introduced regression on weird notes. The ones from tomboy don't stress me that much but anything custom, like coming from snowy or other clients could be more at risk since I don't have test notes for them.

Please bang on it if you have a development environment as this is as close as 0.3 as one will get.