Message 105806 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	belopolsky
Recipients	belopolsky, draghuram, mark.dickinson, rhettinger, stutzbach
Date	2010-05-15.12:47:41
SpamBayes Score	8.955802e-05
Marked as misclassified	No
Message-id	<AANLkTinlk7FVFR5FFeDQiHUwV_RMtf7CaYI7N26xQhgE@mail.gmail.com>
In-reply-to	<1273919527.19.0.894385195397.issue8692@psf.upfronthosting.co.za>

Content
On Sat, May 15, 2010 at 6:32 AM, Mark Dickinson <report@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.

On Sat, May 15, 2010 at 6:32 AM, Mark Dickinson <report@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.

History
Date	User	Action	Args
2010-05-15 12:47:42	belopolsky	set	recipients: + belopolsky, rhettinger, mark.dickinson, draghuram, stutzbach
2010-05-15 12:47:41	belopolsky	link	issue8692 messages
2010-05-15 12:47:41	belopolsky	create