On 22 mars 12:57, Scott Pilkey wrote: > Hello, Hi, > I came up with this fix a little while ago, but haven't gotten a > chance to submit my proposed fix until now. > > It's very straight-forward. In the existing code, you only check the > first item in any node list, when the entire list actually needs to > be checked to see if the defining method exists. I just deleted a > few lines, added a loop, and bingo, all finished. It fixes the > example problem in the ticket description, and I believe it is > correct. > > I have attached the diff (hg diff > pylint9018.diff). If there's > anything else required to get this patch approved (assuming it's > good), just let me know. could you please add a test to the functional test suite (see test/input and test/messages, more info on the ml archives) demonstrating the pb is fixed and won't show up anymore?
-- Sylvain Thénault LOGILAB, Paris (France) Formations Python, Debian, Méth. Agiles: http://www.logilab.fr/formations Développement logiciel sur mesure: http://www.logilab.fr/services CubicWeb, the semantic web framework: http://www.cubicweb.org _______________________________________________ Python-Projects mailing list Python-Projects@lists.logilab.org http://lists.logilab.org/mailman/listinfo/python-projects
