summaryrefslogtreecommitdiff
path: root/dbind
Commit message (Collapse)AuthorAgeFilesLines
* Reformat all the *.[ch] files with clang-formatFederico Mena Quintero2022-12-065-781/+832
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* dbtest.c: Ensure that we have a bus connectionFederico Mena Quintero2022-07-121-3/+2
|
* Remove test for Novell.ICEDesktop.DaemonFederico Mena Quintero2022-07-071-45/+0
| | | | I have no idea what that even was :)
* dbtest:demarshal() - Assert that the message is iterable, or things will go ↵Federico Mena Quintero2021-12-141-0/+3
| | | | | | | | | | really bad down the line Fixes static-scan warnings of this kind: ../../../dbind/dbtest.c:83:18: warning: The left operand of '==' is a garbage value [core.UndefinedBinaryOperatorResult] g_assert (v2 == 42); ~~ ^
* dbind_any_demarshal_va() - remove unused variablesFederico Mena Quintero2021-11-231-13/+8
| | | | | va_arg() will just consume each value; there is no need to store it anywhere.
* test_struct_with_array() - use the correct variable in the assertionsFederico Mena Quintero2021-11-231-2/+2
| | | | The code is testing the demarshaled value, which lives in q, not in p.
* Remove unused variableFederico Mena Quintero2021-11-231-1/+0
|
* dbtest: print to stdout instead of stderrSamuel Thibault2020-08-021-9/+9
| | | | | Printing to stderr makes testsuite analysis think there were some errors.
* Relicense as LGPL 2.1Mike Gorse2019-08-285-20/+20
| | | | Fixes https://gitlab.gnome.org/GNOME/at-spi2-core/issues/2
* Fix outdated FSF addressMike Gorse2019-08-275-10/+10
|
* Avoid compiler warningsEmmanuele Bassi2018-05-171-4/+4
| | | | | Use the appropriate pointer type when testing dbind_any_marshal() and dbind_any_demarshal().
* support building a static libraryRafael Kitover2018-03-051-6/+4
| | | | | | | | | | | | Change the atspi library definition in meson.build to `library()` instead of `shared_library()` so that when meson is called with `--default-library static` a static library is built. For this to work, also change the `declare_dependency()` in `dbind/meson.build` to be a list of sources instead of a `link_with:` `static_library()`. https://bugzilla.gnome.org/show_bug.cgi?id=793762
* Drop autotools supportMike Gorse2017-10-301-27/+0
|
* Add meson build files to Makefile.amMike Gorse2017-06-191-0/+3
| | | | Fix typo
* build: Modernise configure.acwip/mesonEmmanuele Bassi2017-05-241-3/+0
| | | | | | | | We use AM_SILENT_RULES but ask for automake 1.9 at the minimum, which didn't have AM_SILENT_RULES (they were introduced in 1.11.2). Additionally, we want AM_PROC_CC_C_O, not AC_PROG_CC, to find out our C compiler.
* Add Meson build systemEmmanuele Bassi2017-05-241-0/+16
| | | | | | | | | | | | | Meson is a meta-build system which provides us with various advantages over Autotools: - it's portable to various platforms - it's easy to understand - it's fast - it's well maintained Various libraries in the GNOME stack have been ported to it, including GTK+ itself.
* Get rid of the DBIND_ALIGNOF_* definitions and directly use ALIGNOF_*Thomas Petazzoni2016-04-013-24/+12
| | | | | | | | | Since the AC_CHECK_ALIGNOF() macro fills the config.h header with ALIGNOF_<type> definitions, there's no longer a need for the DBIND_ALIGNOF_<type> definitions. This commit gets rid of them, and also gets rid of the dbing-config.h file. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
* dbind/Makefile.am: specify subdir-objects automake optionThomas Petazzoni2016-04-011-0/+3
| | | | | | | | | | | | | | | | | | | As recommended during autoreconf: dbind/Makefile.am:1: while processing Libtool library 'libdbind.la' dbind/Makefile.am:23: warning: source file '../atspi/atspi-gmain.c' is in a subdirectory, dbind/Makefile.am:23: but option 'subdir-objects' is disabled automake: warning: possible forward-incompatibility. automake: At least a source file is in a subdirectory, but the 'subdir-objects' automake: automake option hasn't been enabled. For now, the corresponding output automake: object file(s) will be placed in the top-level directory. However, automake: this behaviour will change in future Automake versions: they will automake: unconditionally cause object files to be placed in the same subdirectory automake: of the corresponding sources. automake: You are advised to start using 'subdir-objects' option throughout your automake: project, to avoid future incompatibilities. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
* configure: use AC_CHECK_ALIGNOF() instead of custom DBIND_CHECK_ALIGNOF()Thomas Petazzoni2016-04-011-8/+8
| | | | | | | | | | | | | | | | | | DBIND_CHECK_ALIGNOF() is a custom autoconf macro from at-spi2-core that tests the alignment required on the current platform for a given C type. Unfortunately, this macro uses AC_TRY_RUN(), which requires running a test program on the target platform, which is fundamentally incompatible with cross-compilation. Therefore, this patch switches the configure.ac code to use AC_CHECK_ALIGNOF() instead, which is available in the standard set of autoconf macros, and is able to test the required alignment for C types without using AC_TRY_RUN(). Consequently, it adjusts the dbind/dbind-config.h.in definitions to use the definitions created by the AC_CHECK_ALIGNOF() macros, which are not substituted by autoconf, but directly available from config.h. Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
* Fix some warningsMike Gorse2014-09-151-13/+1
|
* Use git.mk to maintain .gitignore.Evan Nemerson2014-06-071-0/+2
|
* Fix atspi_text_get_bounded_rangesMike Gorse2013-11-201-0/+4
|
* dbind: remove call to atspi_dbus_connection_setup_with_g_mainMike Gorse2013-09-051-1/+0
| | | | | | | | | dbind isn't currently running a GMainLoop, so it shouldn't call atspi_dbus_connection_setup_with_g_main, and calling this is harmful if it is being called from a context other than the default context, since D-Bus messages will no longer be delivered. https://bugzilla.gnome.org/show_bug.cgi?id=707218
* Don't call dbus_connection_dispatch recursivelyMike Gorse2013-09-021-2/+7
| | | | | | dbus_connection_dispatch is trying to lock the connection when it is already locked, resulting in a deadlock, so just don't call it recursively for now.
* Dispatch dbus messages after method calls when no main loop runningMike Gorse2013-08-211-1/+7
| | | | | | | | | | If no main loop is running, then the dbus connection's incoming message queue was never emptied, so with, eg, dogtail, dbus_connection_send_with_reply_and_block would take longer and longer to look through the message queue to find a reply. This also effectively led to a memory leak. https://bugzilla.gnome.org/show_bug.cgi?id=701145
* Cancel pending calls when giving up on themMike Gorse2013-07-101-3/+9
| | | | | | | | | | | When timing out without having received a reply from a pending call, then cancel the call so that it will be removed from the DBusConnection's queue. This will hopefully prevent the call and the SpiReentrantCallClosure from being leaked. Note that this code is still behaving oddly (why isn't a notification being sent by libdbus when the specified amount of time has ellapsed?)
* Fix atspi_device_listener_new_simple and some other warningsMike Gorse2012-11-261-3/+3
|
* Don't distribute dbind-config.hMike Gorse2012-11-121-1/+3
| | | | | | | Dbind-config.h is generated by configure and should not be part of the release. https://bugzilla.gnome.org/show_bug.cgi?id=688083
* Don't leak errorsWilliam Jon McCann2012-09-142-10/+9
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=684033
* Fix GError sent when receiving a reply of the wrong typeMike Gorse2012-04-101-2/+2
| | | | | When a method reply of the wrong type was received, the parameters were still sent backwards when formatting the GError.
* Fix message displayed when a call returns the wrong typeMike Gorse2012-02-231-1/+1
| | | | | The message displayed by dbind for a type mismatch had its parameters inverted.
* Have dbind_method_call_reentrant_va check the signature of repliesMike Gorse2012-01-201-0/+11
| | | | | | When making a method call, we should check that the signature of the reply matches what we expect and throw an error if not. Otherwise we may crash.
* Avoid making calls on processes thought to be hungMike Gorse2012-01-151-2/+8
| | | | | | | | | If a method call times out, record the process as being hung, and send a ping. Consider the process to be no longer hung when a response to the ping is received. Hopefully this will prevent long delays if an AT makes consecutive calls on an application that is not responding.
* Fix the buildMike Gorse2011-05-291-1/+1
| | | | | Include atspi-gmain.h from dbind rather than atspi.h, since atspi.h includes atspi-enum-types.h, which has not yet been built.
* Dynamically allocate reentrant call closuresMike Gorse2011-05-221-10/+17
| | | | | | | Putting reentrant call closures on the stack can be dangerous if set_reply is called after the caller has timed out and returned, for instance, so better to dynamically allocate them. This will hopefully fix some crashes and hopefully not leak memory.
* Build fixes.Patrick Welche2011-05-141-2/+1
| | | | | | | | | | - atspi-application.h makes use of DBusConnection* . - -Werror-implicit-function-declaration doesn't contain an equal sign in my copy of gcc, and is gcc specific. - The default behaviour of AC_CHECK_LIB glib-2.0 is to add -lglib-2.0 to LIBS which can then cause the search for a dlopen function to fail because the flags to find libglib aren't included. Solution is to tell AC_CHECK_LIB to do nothing(!)
* Release v1.91.92 and fix some build errorsAT_SPI2_CORE_1_91_92Mike Gorse2011-03-211-2/+2
|
* Add some missing prototypesMike Gorse2011-03-212-1/+3
|
* Bump version, remove dbus-glib dependency, and add pc fileMike Gorse2011-03-211-1/+1
| | | | | | Add in a modified version of dbus-gmain.c so that we can stop dependeing on dbus-glib and include a currently unreviewed fix for FDO#35115. Also add a pc file and bump version to 1.91.92.
* Add missing standard headersFrederic Crozat2011-02-011-0/+1
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=641164
* Update copyright and add some missing license infoMike Gorse2011-01-315-1/+94
|
* Fix check for message timeoutMike Gorse2011-01-301-0/+14
|
* Fix some memory leaksMike Gorse2011-01-061-1/+1
|
* Fix building from a separate build directoryMike Gorse2010-12-281-0/+1
|
* dbind_send-and_allow_reentry: Avoid crashing if destination is NULLMike Gorse2010-12-281-2/+3
|
* Use peer-to-peer connections when availableMike Gorse2010-12-041-2/+5
|
* Add time-out, and support inspecting our own appMike Gorse2010-12-011-30/+6
|
* Various fixesMike Gorse2010-11-292-1/+50
| | | | | | | Events and DeviceEvents are now sent with transfer full to work around a possible pygi bug. Various other fixes.
* Really commit fixes; make unit tests passMike Gorse2010-11-283-17/+74
|
* Various component fixes, and fix dbind_method_call_reentrant_vaMike Gorse2010-11-173-15/+17
|
View Lorry Controller Status