Silly code for popping kwargs
Bug #254128 reported by
John Millikin
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Bazaar Diffstat Plugin |
Fix Released
|
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.
Revision history for this message
| John Millikin (jmillikin) wrote | #1 |
Revision history for this message
| Russ Brown (pickscrape) wrote | #2 |
| Changed in bzr-diffstat: | |
| assignee: | nobody → pickscrape |
| importance: | Undecided → Low |
| status: | New → Fix Released |
To post a comment you must log in.
Yes, that is a better way to do it.
Still quite new to python... Thanks for pointing this out. :)