Alexander Belopolsky <belopol...@users.sourceforge.net> added the comment:

On Sat, May 15, 2010 at 6:32 AM, Mark Dickinson <rep...@bugs.python.org> wrote:
> New patch (factorial3.patch) addressing all of Alexander's points except the 
> one about including Python source somewhere.

Thanks for making the changes.  I think we converged to a really neat
implementation.

>
> I also expanded the lookup table to 20 entries on LP64 systems.
>

It's a matter of taste, but I was taught that C allows trailing commas
in initializers specifically for the cases like this to avoid using a
leading comma.

----------

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

Reply via email to