diff options
author | Aleksander Morgado <aleksander@aleksander.es> | 2020-09-03 14:25:58 +0200 |
---|---|---|
committer | Aleksander Morgado <aleksander@aleksander.es> | 2020-09-08 08:06:45 +0000 |
commit | 5adf577f4cb1db0d1df4c2c7eea6156f1ba707a5 (patch) | |
tree | 96fcd47c997821ed6516e8cf3dc91be8d78bc46c /build-aux | |
parent | cbf0a3aa21a31f7ea24ff1fb391f50670ab6ae51 (diff) |
qmi-codegen: (skip) return value in bundle getters and setters
The return value (TRUE or FALSE) is not required in the introspection
support because the method already throws an exception on failure (in
other words, if there is no exception the return value would always be
TRUE), so explicitly (skip) it.
This commit ends up triggering an API break in the bindings generated
via GObject introspection, because the getter and setter calls would
now have one less return argument.
E.g. instead of:
success, imei = output.get_imei()
We should now do:
imei = output.get_imei()
There is no API break in libqmi-glib.
Diffstat (limited to 'build-aux')
-rw-r--r-- | build-aux/qmi-codegen/Field.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/build-aux/qmi-codegen/Field.py b/build-aux/qmi-codegen/Field.py index 5103822..e3d44d3 100644 --- a/build-aux/qmi-codegen/Field.py +++ b/build-aux/qmi-codegen/Field.py @@ -136,7 +136,7 @@ class Field: ' *\n' ' * Get the \'${name}\' field from @self.\n' ' *\n' - ' * Returns: %TRUE if the field is found, %FALSE otherwise.\n' + ' * Returns: (skip): %TRUE if the field is found, %FALSE otherwise.\n' ' *\n' ' * Since: ${since}\n' ' */\n') @@ -206,7 +206,7 @@ class Field: ' *\n' ' * Set the \'${name}\' field in the message.\n' ' *\n' - ' * Returns: %TRUE if @value was successfully set, %FALSE otherwise.\n' + ' * Returns: (skip): %TRUE if @value was successfully set, %FALSE otherwise.\n' ' *\n' ' * Since: ${since}\n' ' */\n') |