2009/4/14 Francesco Cosoleto <[email protected]>

> scrubxml() is a member of Site class, this looks as wrong.


yes, it looks quite wrong.


> Is a new
> function really necessary?


The first version of the function justified a function by itself (
https://fisheye.toolserver.org/browse/pywikipedia/trunk/pywikipedia/wikipedia.py?r1=6555&r2=6567).
It's not that necessary now =)

Data truncation when find() doesn't return a value >= 0 may be omitted too.
>

Not quite sure of what you mean, but go ahead, be bold =)

-- 
Nicolas Dumazet — NicDumZ [ nɪk.d̪ymz ]
_______________________________________________
Pywikipedia-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/pywikipedia-l

Reply via email to