Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dbus: Cleanup the dbus-service.h file and corresponding makros #875

Merged
merged 4 commits into from
Aug 13, 2021

Conversation

mtomaschewski
Copy link
Member

  1. remove the unused .parse() function to gain the possibility to specify the
    PROPERTY makros with SET_FN or GET_FN to define only .get or .set
    handler function, e.g.:
    NI_DBUS_GENERIC_BYTE_ARRAY_PROPERTY(bss, IEs, ies, SET_FN).
    • SET_FN => from DBus to C
    • GET_FN => from C to DBus
  2. remove the unused rw argument from NI_DBUS_GENERIC_DICT_PROPERTY.

From gitlab merge request wicked-maintainers/wicked!167 by Clemens Famulla-Conrad

@mtomaschewski mtomaschewski changed the title [dbus] Cleanup the dbus-service.h file and corresponding makros dbus: Cleanup the dbus-service.h file and corresponding makros Jul 23, 2021
mtomaschewski added a commit that referenced this pull request Jul 23, 2021
@mtomaschewski mtomaschewski force-pushed the dbus-service-macros branch 2 times, most recently from 6db51ab to 24c1983 Compare July 23, 2021 10:16
1) remove the unused .parse() function to gain the possibility to specify the
   `PROPERTY` makros with `SET_FN` or `GET_FN` to define only `.get` or `.set`
   handler function, e.g.:
      ```NI_DBUS_GENERIC_BYTE_ARRAY_PROPERTY(bss, IEs, ies, SET_FN)```.
    - `SET_FN` => from DBus to C
    - `GET_FN` => from C to DBus
2) remove the unused `rw` argument from `NI_DBUS_GENERIC_DICT_PROPERTY`.

From gitlab merge request wicked-maintainers/wicked!167 by Clemens Famulla-Conrad
mtomaschewski added a commit that referenced this pull request Jul 23, 2021
@mtomaschewski mtomaschewski merged commit af34279 into openSUSE:master Aug 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants