Silly code for popping kwargs

Bug #254128 reported by John Millikin on 2008-08-02
2
Affects Status Importance Assigned to Milestone
Bazaar Diffstat Plugin
Low
Russ Brown

Bug Description

The code in "cmd_diff.run()" for popping the "stat" and "stat_diff" parameters is convoluted and silly. The optional default parameter to "dict.pop()" is a better solution.

Russ Brown (pickscrape) wrote :

Yes, that is a better way to do it.

Still quite new to python... Thanks for pointing this out. :)

Changed in bzr-diffstat:
assignee: nobody → pickscrape
importance: Undecided → Low
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers