Comment 1 for bug 1650816

Sure, please submit a patch. But please also consider the following before you do.

trove runs pylint on the code base as part of every commit; see tools/trove-pylint. But, we currently do not look for warnings; pylint is run with -E.

If you'd like to fix this bug, then you will notice that there are 5081 warnings which we currently ignore (because of -E) and the warning W0621 occurs 19 times.

Speaking only for myself, the patch you submit is most welcome if it includes some tooling (such as a change to the pylint tool that we do run) to prevent the same error from creeping back into the code base, and a good reason why this particular warning (which occurs 19 times) is worthy of special treatment over the 5081 warnings that we ignore along with the many errors which we also ignore.

As to why the code is this way, I didn't look.