[Gramps-devel] checked-in code may need policy review

[James G. S. (jim) <jg...@sa...>]

I have checked in a change to ArgHandler (r9331) that fixes a CLI bug
(#1358) I discovered in testing. The fix adds a utility function within
the ArgHandler module which I called _rm_files().

The utility removes files in a specified directory.

There are perhaps couple of code policy issues that someone may wish to
review:

Should this be a module-level utility as I made it? It didn't seem
correct to make a class member of such a general function. In fact,
maybe it's general enough that it should go into some project utility
file (Utils.py?).

Any other suggestions or critique related to style issues?

Regards,
..jim