Mic92/sonata — comment on commit eb8bff6dadb23d3424557b9336bda410ebbd6eba
File:  line 0
By multani:

FWIW, ``_execute_bool`` and ``_toggle_clicked`` were a bit more comprehensive than ``int(not int(...))`` and ``int(...)``. I hoped that the MPD wrapper was more intelligent and would now that those attributes were boolean in the first place this kind of type-dance doesn't really belong to the client I think.

Are you sure about the ``status`` change? I imported it as it when I changed the interface of the module, I suspect there was a good reason behind the difference, yet I didn't take the time to actually check.

And I don't like the ``call()`` interface, it should be made "private", since to use it, you need to know the content of the object (ie. you need to access the ``._client`` attribute).
If you want to simplify this, I suggest you to remove ``call()`` completely from the public interface of the object:
* make it a "private" method (single ``_`` is sufficient)
* or inline the content into ``__getattr__``.

I'm not sure the gain from your ``setattr`` is really that interesting, but why not.