A few comments about style...
On Sat, Aug 05, 2006 at 07:39:11PM -0400, Sean McBride wrote:
> + extraColsList = ops.get('extraColumns',[])
Don't hesitate to add an additional space. We do not worry too much
about the size of the source. ;) BTW, wouldn't it better to use () instead
of creating a new list every time?
+ extraColsList = ops.get('extraColumns', ())
> + gb = ops['groupBy']
gb? KGB? Don't be shy to write a longer names. ;)
+ groupBy = ops['groupBy']
+ extraColumns = self.select.ops.get('extraColumns', ())
Oleg.
--
Oleg Broytmann http://phd.pp.ru/ [EMAIL PROTECTED]
Programmers don't die, they just GOSUB without RETURN.
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
sqlobject-discuss mailing list
sqlobject-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss
