[Cython] [Fwd: Re: Beginner-friendly tickets]
Stefan Behnel
stefan_ml at behnel.de
Fri Mar 20 14:24:49 CET 2009
Prajwal Suhas P wrote:
> Can someone review this patch for ticket#203?
Looking at the patch, I noticed that it duplicates a lot of code. Could
you try to limit this to the parts that are really different? This would
make the code a lot easier to read and the patch easier to review.
For example, one section uses "temp_range_bound", where the other spells
"self.bound2.result()". Assigning "self.bound2.result()" to
"temp_range_bound" in the "else" case would make the first code output
identical in both conditional branches, so that it could happen once
outside of the blocks.
Stefan
More information about the Cython-dev
mailing list