Thanks for the response. As you said, the speedup is likely negligible and might only matter if there are loads of custom fields, but as I was looking at the code of this section anyway (we were backporting a changeset) I felt I might as well raise the issue.
In the future I'll make an enhancement ticket and provide a patch so that someone can just OK and apply it. Kind regards, Ian On Thursday, 21 November 2013 12:49:30 UTC, Ian Clark wrote: > > Hi all, > > Looking at the *grouped_stats_data()* method in > roadmap.py:340<http://www.google.com/url?q=http%3A%2F%2Ftrac.edgewall.org%2Fbrowser%2Ftrunk%2Ftrac%2Fticket%2Froadmap.py%23L340&sa=D&sntz=1&usg=AFQjCNFqrhM_KgWYMzVqSo_gJkAuRRcFXA>it > seems as though it would be more optimal to add a break clause after > line 357 if we've found our field. > > Kind regards, > > Ian > -- You received this message because you are subscribed to the Google Groups "Trac Development" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/trac-dev. For more options, visit https://groups.google.com/groups/opt_out.
