aidotse / aidotse/LeakPro

Double attachment of methods to MIAHandler

Offen
#394 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Python
Sterne
23
Forks
28
Ø Merge
4 T. 12 Std.
Gemergte PRs (30 T.)
5

Beschreibung

**Problem:**
This code

```
# Attach methods to Handler explicitly defined in AbstractInputHandler from user_input_handler
for name, _ in inspect.getmembers(AbstractInputHandler, predicate=inspect.isfunction):
if hasattr(user_input_handler, name) and not name.startswith("__"):
attr = getattr(user_input_handler, name)
if callable(attr):
attr = types.MethodType(attr, self) # ensure to properly bind methods to handler
setattr(self, name, attr)
```

is runed both in the MIAHandler.__init__() and the LeakPro.setup_handler(). It's unnecessary to run it twice?

**Solution:**
Can be removed from MIAHandler.__init__() or from LeakPro.setup_handler(). If removed from LeakPro.setup_handler(), the code needs to be added to MINVHandler.__init__().

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.