Don't append newlines inside a span

[Alir3z4]

the pull request on mainstream have conflicts with code base, aaronsw/html2text#87.

[Alir3z4]

@mcepl the original commiter, @Moishe didn't provide any tests for it, he did actually but it had nothing to do with the change.

Branch fix-#7-dont-append-newlines-inside-span is there with the @Moishe changes, it only needs new unit-test to be good to get merged ;)

[mcepl]

Branch fix-#7-dont-append-newlines-inside-span is there with the @Moishe changes, it only needs new unit-test to be good to get merged ;)

Ah, sorry, I missed that you’ve made it into branch, and I have just created the same as pull-request.

[harshil93]

I am wondering whether this has been fixed or not. I wasted my 2 days in finding out this.

This is the file. https://dl.dropboxusercontent.com/u/32632425/170594 .

I tried the nltk.clean_html() . It goes into an infinite loop.

[Alir3z4]

@mcepl Your branch fixed it? I remember I fixed it in my branch as #7 (comment) but it needed a test case.

[mcepl]

Actually, we've (or at least I ;)) concluded in https://github.com/html2text/html2text/pull/1 that it is actually WONTFIX, and html2text behaves correctly.

Or do I miss something?

[Alir3z4]

This issue is closed and marked as wontfix and invalid.