307593 - Remove unused and broken template class |CWordTokenizer|

Status: VERIFIED FIXED

(Core :: DOM: HTML Parser, defect)

Description

[Hans-Andreas Engel]

Currently, the template class |CWordTokenizer| is broken (Bug 104479).  However,
since it is never used anyway, it should be removed.

Comment 1

[Hans-Andreas Engel]

Note that it is suggested to maybe use it for parsing the XML declaration. 
Because this code is somewhat optimized and is only ~70 lines in total, probably
not much overall clarity would be gained by using a working implementation of
|CWordTokenizer| which requires ~120 lines.  So even if |CWordTokenizer| would
be fixed, it would not be very useful in the current code.

Comment 2

[Hans-Andreas Engel]

Attachment: Remove template class |CWordTokenizer|

Comment 3

[Blake Kaplan (:mrbkap) (inactive)]

Comment on attachment 195338 [details] [diff] [review]
Remove template class |CWordTokenizer|

Off with its head! r=mrbkap

Comment 4

[Johnny Stenback (:jst)]

Comment on attachment 195338 [details] [diff] [review]
Remove template class |CWordTokenizer|

sr=jst

Comment 5

[Blake Kaplan (:mrbkap) (inactive)]

I took the liberty of checking this patch in. Marking this bug as FIXED. Thanks!

Comment 6

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified via LXR and Bonsai (source-code validation).