Skip to content

Use bound instead of binded #65

Closed
Closed

Description

@souliane

There are a few classes like DbusBindedAsync that should IMHO be renamed to use bound instead of binded.

I am not a native speaker, but the correct conjugation seems to be bound (e.g. this source, but there are many others).

Activity

  1. igo95862 commented on Apr 19, 2024

    @igo95862
    Collaborator

    Yes, the name is pretty bad.

    Those objects actually are the descriptors: https://docs.python.org/3/glossary.html#term-descriptor

    When a class attribute is a descriptor, its special binding behavior is triggered upon attribute lookup.

    But I am not sure what is the correct name for what the descriptor returns from its __get__ method.

  2. igo95862 commented on Apr 20, 2024

    @igo95862
    Collaborator

    Another alternative naming is drop the bind name all together.

    For example,

    • DbusMethodAsync: D-Bus method not accessed from any specific object (or accessed from class)
    • DbusLocalMethodAsync - D-Bus method accessed from a local object
    • DbusProxyMethodAsync - D-Bus method accessed from a proxy object

    Can also be LocalDbusMethodAsync / ProxyDbusMethodAsync. What do you think @souliane ?

  3. souliane commented on May 2, 2024

    @souliane
    Author

    Hey, sorry for the late reply.
    Honestly, I don't have any preference... Beside the fact that binded sounds wrong, I am OK with the current naming.
    Feel free to pick whatever names you prefer :-)

  4. igo95862 commented on Dec 14, 2024

    @igo95862
    Collaborator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions