Eli Carter wrote: > I'll merge sometime today. > Good! Then we can continue with the time fields.
> Small nit for the previous patch (in [5405]), you changed the API for rows > from rows=None to rows=[]; but didn't make the same change for cols. Any > particular reason for that? It was interesting for rows, in order to keep the append() done just after simple, but not really needed for cols. Besides, the "rows" argument was a new addition to the API so changing it wouldn't break anything ;-) For cols, it's quite likely that there exists code setting it explicitly to None, so changing it now is perhaps not a good idea. -- Christian --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Trac Development" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/trac-dev?hl=en -~----------~----~----~----~------~----~------~--~---
