Looks ok :)<br><br><div class="gmail_quote">2009/3/22 Peter Oberndorfer <span dir="ltr"><<a href="mailto:kumbayo84@arcor.de">kumbayo84@arcor.de</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
Actually return the values detected by unicodeTest().<br>
Make the old deprecated encodingName() method still return values.<br>
The new encoding method is preferred since it does not leak memory.<br>
<br>
Fix a memory leak in setProberType<br>
Rename encodingNameByteArray() to encoding() as suggested<br>
Remove a unused #define<br>
Add some unit tests<br>
<br>
thanks to Jeff Mitchell for parts of the patch<br>
---<br>
<br>
This is my work in progress for today.<br>
This makes the unicodeTest() function useful again.<br>
I plan to continue tomorrow, and to add some more testcases,<br>
test if things still work properly without a BOM.<br>
And finally if everything is OK, and everybody agrees, commit the patch.<br>
I might need some help backporing it.<br>
<br>
 kate/document/katebuffer.cpp             |    2 +-<br>
 kdecore/localization/kencodingprober.cpp |   30 ++++---<br>
 kdecore/localization/kencodingprober.h   |    4 +-<br>
 kdecore/tests/CMakeLists.txt             |    1 +<br>
 kdecore/tests/kencodingprobertest.cpp    |  126 ++++++++++++++++++++++++++++++<br>
 kdecore/tests/kencodingprobertest.h      |   36 +++++++++<br>
 6 files changed, 184 insertions(+), 15 deletions(-)<br>
 create mode 100644 kdecore/tests/kencodingprobertest.cpp<br>
 create mode 100644 kdecore/tests/kencodingprobertest.h<br>
<br>
</blockquote></div><br>