Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | New document on changes to the XML signatures | Federico Mena Quintero | 2022-12-12 | 2 | -0/+31 | |
| | | | | | | | | | | Let's document functional changes to help people who write bindings based on the XML. | |||||
| * | DeviceEventController.xml: document the RegisterKeystrokeListener method | Federico Mena Quintero | 2022-12-12 | 2 | -2/+42 | |
| | | | | | | | | | | | | | | | | This method is still used, but is in the process of deprecation by the atspi-device-listener functions. I'm documenting it to have a reference of what the dbus demarshalling code is supposed to do. | |||||
| * | DeviceEventController.xml: fix the signature of the "types" argument | Federico Mena Quintero | 2022-12-12 | 1 | -1/+1 | |
| | | | | | | | | The code uses "u"; the XML was "au" - the argument is really a bitmask. | |||||
| * | Use g_new0 instead of g_malloc | Federico Mena Quintero | 2022-12-12 | 1 | -5/+3 | |
| | | | | | | | | Also, g_new0 can't fail, so don't check its result value. | |||||
| * | spi_dec_init_mouse_listener() - remove no-op function | Federico Mena Quintero | 2022-12-09 | 1 | -28/+0 | |
| | | | | | | | | | | All its code was inside an "#ifdef GRAB_BUTTON", which is not defined anywhere. Anyway, we don't do mouse grabs here anymore. | |||||
| * | CI: run all tests under xvfb-run, i.e. under an X server | Federico Mena Quintero | 2022-12-09 | 1 | -5/+5 | |
| | | | | | | | | This lets us start getting test coverage for the X11 paths. | |||||
| * | CI: install xorg-x11-devel to test the X11 code paths | Federico Mena Quintero | 2022-12-09 | 1 | -1/+2 | |
| | | ||||||
| * | CI: install xvfb-run to be able to run X11 tests | Federico Mena Quintero | 2022-12-09 | 1 | -1/+3 | |
| | | ||||||
| * | Remove unused enum variants | Federico Mena Quintero | 2022-12-09 | 1 | -2/+0 | |
| | | ||||||
| * | tests: use "gdbus wait" after launching the mocks instead of sleeping | Federico Mena Quintero | 2022-12-08 | 2 | -2/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | On a busy system (the CI runners) or one with a cold cache (my machine when running the tests after just having started the CI container image), the "sleep 1" we had in the test runner scripts is not a reliable way for the python/dbusmock process to have acquired the name it is supposed to acquire. So, use "gdbus wait" instead as a reliable way of doing that. This should fix the spurious test failures we've been having in CI. | |||||
| * | Rename de-marshaller.[ch] to marshal-dbus.[ch] | Federico Mena Quintero | 2022-12-08 | 5 | -5/+5 | |
| | | | | | | | | | | | | | | I'll put all the libdbus-based marshallers in that file. Later I want to have a marshal-gdbus.[ch] counterpart, to let us port things gradually to gdbus. | |||||
* | | atk-adaptor: Don't leak the socket in impl_GetChildAtIndex | Mike Gorse | 2022-12-12 | 1 | -0/+1 | |
| | | ||||||
* | | atk-adaptor: Handle sockets in impl_GetChildren | Mike Gorse | 2022-12-12 | 1 | -0/+26 | |
|/ | | | | | | | This was special-cased for ChildCount and GetChildAtIndex, but not for GetChildren. Fixes #98 | |||||
* | Fix links to source files in devel guide | Federico Mena Quintero | 2022-12-07 | 1 | -7/+9 | |
| | ||||||
* | Sigh, fix RST markup | Federico Mena Quintero | 2022-12-07 | 1 | -2/+2 | |
| | ||||||
* | Merge branch 'clang-format' into 'main' | Federico Mena Quintero | 2022-12-07 | 260 | -13470/+13605 | |
|\ | | | | | | | | | | | | | Add an auto-formatting check in CI Closes #97 See merge request GNOME/at-spi2-core!119 | |||||
| * | Note on the order of header files | Federico Mena Quintero | 2022-12-07 | 1 | -0/+20 | |
| | | ||||||
| * | Note how to run clang-format by hand | Federico Mena Quintero | 2022-12-07 | 1 | -3/+6 | |
| | | ||||||
| * | Remove detritus from the markdown->reStructuredText conversion | Federico Mena Quintero | 2022-12-07 | 1 | -4/+0 | |
| | | ||||||
| * | Document the code formatting machinery | Federico Mena Quintero | 2022-12-07 | 3 | -11/+62 | |
| | | ||||||
| * | CI: add a style-check-diff job | Federico Mena Quintero | 2022-12-07 | 1 | -12/+9 | |
| | | ||||||
| * | Remove ci/search-common-ancestor.sh, it's no longer used | Federico Mena Quintero | 2022-12-07 | 1 | -43/+0 | |
| | | ||||||
| * | Update ci/run-style-check.sh | Federico Mena Quintero | 2022-12-07 | 2 | -16/+30 | |
| | | ||||||
| * | Reformat all the *.[ch] files with clang-format | Federico Mena Quintero | 2022-12-06 | 255 | -13391/+13478 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I ran this on each directory with C files: clang-format -i *.[ch] "-i" is the in-place option. I also adjusted the order of #includes for some files which failed to build after that: Clang-format reorders blocks of #include directives alphabetically, but they can be grouped and separated by blank lines. If there is a blank line between blocks, like #include "zork.h" #include "bar.h" #include "foo.h" then it will not put zork.h after the other two. The last two header files will be sorted alphabetically. We can adjust the formatting of chunks of code by hand with comments like these: /* clang-format off */ this code { is, formatted, by, hand; } /* clang-format on */ See https://clang.llvm.org/docs/ClangFormat.html for the general manual and https://clang.llvm.org/docs/ClangFormatStyleOptions.html for the style options and the comments described above. | |||||
| * | Add a .clang-format configuration | Federico Mena Quintero | 2022-12-06 | 1 | -0/+10 | |
|/ | | | | | | Stolen from glib. Most of at-spi2-core's code is in GNU coding style, or something close enough, mostly due to inconsistent indentation. We'll fix it. | |||||
* | Merge branch 'registry-test-coverage' into 'main' | Federico Mena Quintero | 2022-12-07 | 8 | -36/+60 | |
|\ | | | | | | | | | A little more test coverage for registry.c See merge request GNOME/at-spi2-core!118 | |||||
| * | Remove a couple of unused macros | Federico Mena Quintero | 2022-12-06 | 1 | -3/+0 | |
| | | ||||||
| * | Remove unnecessary includes | Federico Mena Quintero | 2022-12-06 | 1 | -2/+0 | |
| | | ||||||
| * | Move X11-specific prototypes to deviceeventcontroller-x11.h | Federico Mena Quintero | 2022-12-06 | 3 | -8/+8 | |
| | | | | | | | | This removes "HAVE_X11" from deviceeventcontroller.h | |||||
| * | Remove superfluous include | Federico Mena Quintero | 2022-12-06 | 1 | -3/+0 | |
| | | ||||||
| * | test_root.py: Test that querying for a property on an unknown interface ↵ | Federico Mena Quintero | 2022-12-06 | 1 | -0/+7 | |
| | | | | | | | | yields an error | |||||
| * | Test the introspection of the registry object | Federico Mena Quintero | 2022-12-06 | 1 | -0/+7 | |
| | | ||||||
| * | test_introspection.py: Factor out function to test a set of advertised ↵ | Federico Mena Quintero | 2022-12-06 | 1 | -6/+17 | |
| | | | | | | | | interfaces | |||||
| * | Rename test_root_introspectable.py to test_introspection.py | Federico Mena Quintero | 2022-12-06 | 1 | -0/+0 | |
| | | | | | | | | Let's put all the DBus introspection tests in the same file. | |||||
| * | tests/registryd: add a fixture to obtain the registry object | Federico Mena Quintero | 2022-12-06 | 1 | -0/+7 | |
|/ | ||||||
* | Merge branch 'remove-unused-methods' into 'main' | Federico Mena Quintero | 2022-12-06 | 6 | -438/+36 | |
|\ | | | | | | | | | | | | | Remove unused DeviceEventController methods Closes #94 See merge request GNOME/at-spi2-core!116 | |||||
| * | SpiDEController: Remove the have_mouse_listener field | Federico Mena Quintero | 2022-12-05 | 2 | -8/+5 | |
| | | | | | | | | | | | | It's never set, and always remained as FALSE. On the other hand, we lave have_mouse_event_listener, which *is* used. | |||||
| * | append_mouse_listener(): Remove unused function | Federico Mena Quintero | 2022-12-05 | 1 | -18/+0 | |
| | | ||||||
| * | notify_mouse_listener(): Remove unused function | Federico Mena Quintero | 2022-12-05 | 1 | -22/+0 | |
| | | ||||||
| * | spi_controller_deregister_device_listener(): Remove unused function | Federico Mena Quintero | 2022-12-05 | 1 | -17/+0 | |
| | | ||||||
| * | spi_remove_device_listeners(): Remove the part that dealt with mouse_listeners | Federico Mena Quintero | 2022-12-05 | 1 | -10/+0 | |
| | | | | | | | | There are no mouse_listeners created anymore. | |||||
| * | spi_listener_clone(): Remove unused function | Federico Mena Quintero | 2022-12-05 | 1 | -11/+0 | |
| | | ||||||
| * | spi_listener_clone_free(): Remove unused function | Federico Mena Quintero | 2022-12-05 | 1 | -8/+0 | |
| | | ||||||
| * | spi_dec_set_unlatch_pending(): Remove unused function | Federico Mena Quintero | 2022-12-05 | 1 | -11/+0 | |
| | | ||||||
| * | Remove variable that is set but never used | Federico Mena Quintero | 2022-12-05 | 1 | -4/+0 | |
| | | ||||||
| * | Remove the other is_consumed | Federico Mena Quintero | 2022-12-05 | 1 | -8/+1 | |
| | | | | | | | | And simplify the call to XAllowEvents() since now there is only one case. | |||||
| * | Remove one auxiliary is_consumed | Federico Mena Quintero | 2022-12-05 | 1 | -9/+2 | |
| | | | | | | | | It was used for the return value of spi_controller_notify_mouselisteners. | |||||
| * | Remove another auxiliary variable that was used to call ↵ | Federico Mena Quintero | 2022-12-05 | 1 | -11/+0 | |
| | | | | | | | | spi_controller_notify_mouselisteners | |||||
| * | Remove one auxiliary variable that was used to call ↵ | Federico Mena Quintero | 2022-12-05 | 1 | -13/+0 | |
| | | | | | | | | spi_controller_notify_mouselisteners | |||||
| * | spi_controller_notify_mouselisteners(): Remove | Federico Mena Quintero | 2022-12-05 | 3 | -72/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | This function immediately returns FALSE if there are no controller->mouse_listeners. The only place that ever added to the controller->mouse_listeners list was spi_controller_register_device_listener(), which we removed in the last commit. So, remove the function altogether and make the two callers do as if it just returned FALSE. |