Re: [Python-projects] Three new miscellaneous pylint checks

2009-11-23 Thread Alexandre Fayolle
On Monday 23 November 2009 11:36:38 Marc 'BlackJack' Rintsch wrote: > Alexandre Fayolle wrote: > > Therefore, imo it deserves an Error. > > But it is no error. It is perfectly legal to implement `__pos__()` in a > way that makes sense to call it repeatedly. hmmm > Even to implement that two >

Re: [Python-projects] Three new miscellaneous pylint checks

2009-11-23 Thread Julian Scheid
On Mon, Nov 23, 2009 at 11:36 PM, Marc 'BlackJack' Rintsch wrote: > But it is no error.  It is perfectly legal to implement `__pos__()` in a > way that makes sense to call it repeatedly.  Even to implement that two > successive calls increment something to mimic some C++ behaviour.  Butt > ugly, b

Re: [Python-projects] Three new miscellaneous pylint checks

2009-11-23 Thread Marc 'BlackJack' Rintsch
Alexandre Fayolle wrote: > On Monday 23 November 2009 10:49:38 Emile Anclin wrote: >> Hello, >> >> On Wednesday 18 November 2009 22:19:26 James Lingard wrote: >>> x = y = 0 >>> print ++x >>> >>> * Module pylint-misc >>> E9800: 2: Use of the non-existent ++ operator >> I have a little d

Re: [Python-projects] Three new miscellaneous pylint checks

2009-11-23 Thread Alexandre Fayolle
On Monday 23 November 2009 10:49:38 Emile Anclin wrote: > Hello, > > On Wednesday 18 November 2009 22:19:26 James Lingard wrote: > > x = y = 0 > > print ++x > > > > * Module pylint-misc > > E9800: 2: Use of the non-existent ++ operator > > I have a little doubt about this message. I

Re: [Python-projects] Three new miscellaneous pylint checks

2009-11-23 Thread Emile Anclin
Hello, On Wednesday 18 November 2009 22:19:26 James Lingard wrote: > x = y = 0 > print ++x > * Module pylint-misc > E9800: 2: Use of the non-existent ++ operator I have a little doubt about this message. I would not say, there is a use of an "non-existent operator". It isn't an e

Re: [Python-projects] Three new miscellaneous pylint checks

2009-11-20 Thread Nicolas Chauvat
Hi James, On Wed, Nov 18, 2009 at 01:19:26PM -0800, James Lingard wrote: > Below is a new checker that checks three miscellaneous things: > ... Thank you very much for this contribution. http://www.logilab.org/ticket/18952 is the corresponding ticket. If someone is willing to go ahead and integ

[Python-projects] Three new miscellaneous pylint checks

2009-11-18 Thread James Lingard
Below is a new checker that checks three miscellaneous things: -- use of the non-existent ++ operator -- duplicate key in dictionary -- "assert x and y" rather than "assert x" and "assert y" For example, on the following file: x = y = 0 print ++x d = { 'a' : 0, 'a' : 1 } assert ( x == 0 ) and (