SilentGhost <ghost....@gmail.com> added the comment:

facts_found.strip(" ").rstrip(";")

strip is redundant since facts_found is a first element of partitioning by the 
same string. rstrip is wrong since you're potentially deleting more than one 
character (there is no test for that).

In your test you're checking whether returned facts contain every requested 
fact, this is not guaranteed by RFC. Also, not sure what the last for statement 
is supposed to mean. Is it a typo?

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue11072>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to