action exporter: use GSource* instead of source id

Keep track of the GSource* of our event dispatch idle instead of using
source ID.

action exporter: cancel pending events on unexport

If there are events pending when we unexport the action group, free them
and cancel the pending idle.

GMenuModel exporter: remove workaround

With Bug 665733 fixed, we can remove the workaround (which was causing
thread safety problems).

Add testcase for threaded menu exporter use

This currently fails

Add a test for multithreaded action group exporter use

GApplication dbus: publish the menus again

Fully implement app-menu and menubar properties

GApplication: make menu properties readwrite

Otherwise the notify signals won't be emitted.

menumodel test: don't get stuck in a loop

Sometimes randa and randb end up having the same state, causing them to
return the same stream of 'random numbers'.  This is a problem for the
testcase that is looping to find unequal menus.

If we find ourselves in this state, throw one of the random generators
away and recreate it so we have a better chance of getting some unequal
menus.

menu/action exporter docs fixup

gio symbol cleanups

Menu model exporter: clean up the API

Give it the same treatment as the exporter for GActionGroup just got.

There is a wart here: the exporter attempt to re-enter GDBusConnection
when it is freed in order to cancel outstanding name watches.
GDBusConnection holds its own lock while calling the destroy notify, so
the attempt at reentrancy results in a deadlock.

We have a workaround to deal with that for now...

GApplication: menu can change after registration

Allow the menu to be changed after registration.  This is quite useful
for setting up the menus from the ::startup handler instead of having to
do it before registration because it lets you skip the work if you're
not the primary instance.

Action group exporter: clean up the API

Make it look more like a typical GDBusConnection API with integer
registration ID and corresponding unexport call.  Kill the 'query' call.

GApplication: simplify dbus impl

The error handling on register() was just totally out of hand before.
Clean that mess up.

Take out the menu export for now as well.  It will be added back again
later.

GApplication dbus: use XML for introspection

The hand-written structures are hard to read and cause quite a lot of
relocations at library load time.  Avoid that.

GApplication: implement GActionMap

and deprecate g_application_set_action_group().

Fix some GMenu and GMenuItem leaking

GApplication: make distinction about menus

Rename g_application_set_menu to g_application_set_app_menu and make a
couple of fixups.  Clarify the documentation about exactly what this
menu is meant to be.

Add g_application_set_menubar and document that as well.

gsimpleaction: Ensure actions are enabled

Include gactionmap.h in gio.h

Minor doc improvements

Add GActionMap to the docs

GSimpleAction: Fix to comply with constructor rules

foo_new_*() must be pure wrappers around g_object_new(), otherwise
their functionality is inaccessible to bindings.

add GActionMap interface

This is an interface to represent GSimpleActionGroup-like objects (ie:
those GActionGroups that operate by containing a number of named GAction
instances).

gmenu exporter: put submenus in separate groups

Keep sections in the same subscription group as the parent (since they
will be needed immediately) but put submenus in a separate group.

menu tests: keep mirror of proxy

Create a 'mirror' model of the proxy for the testcase.  In addition to
testing that the proxy model emits the proper signals this also keeps
the proxy alive (by holding references to it from the mirror).

The previous code would create the submenu proxies and destroy them
right away (from the recursive step in the equality comparison
functions).  This means that the subscription would go out over D-Bus
and the proxy would be destroyed before it returned.  Keeping the model
alive allows it to be actually updated.

Shorten a variable name

Expand dbus action group tests

The new tests check that activation and state changes propagate
back.

Some more assertions

Add some dbus action group tests

Fix the dbus action group query_action implementation

Make stopping an action group export work

rework GMenuProxy links

Only resolve the link at the point that we pull it through the API
rather than at the point that we first are told about it.  This reduces
the lifespan of subscriptions and, more importantly, avoids a tricky
reference cycle issue.

g_menu_model_get_item_link: be careful with refs

Don't unref the hashtable until after we already take the ref on the
value that was contained in the hashtable, otherwise we may kill the
value.

Add debug code

This adds a dump method to show the entire tree of proxies.

Avoid cross-talk between tests

Each test needs to remove the sources that it attaches
to the default main context, or else things will work
fine in isolation, but go bad in a full test run.

Typo fix

Remove unused variable

Add a test for menu subscriptions

Take out excessive width

Reduce excessive width

whitespace fix

Add some tests for links

Add parser roundtrip tests

GMenuMarkup: fixes uncovered by roundtrip testing

The print function was forgetting to emit type information for
attributes, and the parser was not handling types properly either.

Improve docs

Specify that g_menu_item_set_attribute_value consumes floating
@values.

GApplication: Add a menu example to the docs

GMenu: Enforce attribute name restrictions

The code assumes in various places that ':' does not occur
in attribute names. We are a little more strict than that,
and only allow lowercase ASCII, digits and '-'.

Reduce excessive width of gmenu.h

Match up a parameter name

Describe the org.gtk.Menus interface

Even though we consider the interface to be an implementation
detail, we should have internal documentation of the interface.

Describe the org.gtk.Actions interface

Even though we consider the interface to be an implementation
detail, we should have internal documentation for the interface.

Rename exporter APIs

There are no public 'exporter' objects, so don't allude to them
in the function names. At the same time, we want to make it clear
that these functions are D-Bus specific.

The new APIs are
g_action_group_dbus_export_start
g_action_group_dbus_export_query
g_action_group_dbus_export_stop
g_menu_model_dbus_export_start
g_menu_model_dbus_export_query
g_menu_model_dbus_export_stop

GActionGroup: Mention GMenuModel in the docs

Add g_application_set_menu

This adds ane asy way to export menu information alongside
the actions that are already exported by GApplication.

Add GMenuModel testcases

Add GMenuProxy

GMenuProxy can be used as a client-side wrapper
for the a menu model that has been exported on D-Bus.

Add GMenuModel D-Bus exporter

Add GMenu markup

These functions serialize and deserialize a GMenuModel
to and from XML.

Add GMenu

Add GMenuModel

GApplication: use GDBusActionGroup

instead of the internal version of the same

Add GDBusActionGroup

GDBusActionGroup can be used as a client-side wrapper
for the an action group that has been exported on D-Bus.

GApplication: port action client to new D-Bus API

for compatibility with GActionGroup exporter

GApplication: use GActionGroup exporter

instead of home-grown internal implementation of the same

Add GActionGroup D-Bus exporter

Revert "Use an XML blob for gapplication interface info"

This reverts commit 8013401f6e4edb01dd31b716c179806b55d4064b.

Revert "Forgot one interface..."

This reverts commit 30580bdca7764525a49899dd56dcdde25a300a21.

Updates

Clarify docs around main loop vfuncs

Mention that the vfuncs are no longer used, and tweak the
wording around what g_application_run() does.

https://bugzilla.gnome.org/show_bug.cgi?id=665391

Bug 665733 – GDBusConnection holds lock while calling destroynotify

Fix this problem by always running the destroynotify from an idle.

https://bugzilla.gnome.org/show_bug.cgi?id=665733

Signed-off-by: David Zeuthen <davidz@redhat.com>

unicode: Fix a few issues with G_UNICHAR_MAX_DECOMPOSITION_LENGTH

Raised by Matthias in bgo#665685 but which I didn't spot until after pushing
commit 3ac7c35656649b1d1fcf2ccaa670b854809d4cd8.

Renames G_UNICHAR_MAX_DECOMPOSITION_LEN to G_UNICHAR_MAX_DECOMPOSITION_LENGTH
and fixes a few documentation issues.

See: bgo#665685

Bug 665685 — Add a #define for the max length of a Unicode decomposition

Add G_UNICHAR_MAX_DECOMPOSITION_LEN for the maximum length of the
decomposition of a single Unicode character.

Closes: bgo#665685

GDBus: Note in docs that g_dbus_node_info_new_for_xml() is using GMarkup

This was discussed in bug 665634.

https://bugzilla.gnome.org/show_bug.cgi?id=665634

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBus: fix link to D-Bus spec

Signed-off-by: David Zeuthen <davidz@redhat.com>

Minor doc fix.

Minor

Make the qsort_r check cross-compile friendly

Pointed out by Daniel Mack.

https://bugzilla.gnome.org/show_bug.cgi?id=665607

outputstream: Clarify docs of flush()

After questioning the semantics of flush on IRC, it seemed necessary to
clarify what flushing is supposed to do. The Linux man page for fflush()
seemed to cover it perfectly, so I just copied it.

I did not add the "via the underlying write mechanism" part as that in
my opinion is not something subclasses should need to guarantee.

Send CHANGES_DONE_HINT on file moves if no IN_CLOSE_WRITE is emitted

This patch makes GFileMonitor to emit EVENT_CHANGES_DONE_HINT when
EVENT_CREATED is emitted but the file is not opened for writing.

On file moves across different mounted volumes, inotify will always emit
IN_CREATE and IN_CLOSE_WRITE (plus other events).
This translates into GIO's _EVENT_CREATED and _EVENT_CHANGES_DONE_HINT.

On file moves across the same mounted volumes, inotify will emit
IN_MOVED_FROM/IN_MOVED_TO which will be translated into
_EVENT_DELETED/_EVENT_CREATED GIO's side. No _EVENT_CHANGES_DONE_HINT is
emited afterwards.
Under such circumstances a file indexer does not know when actually the
file is ready to be indexed, either waiting too much or triggering the
indexing twice. On small devices it's not advisable.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=640077
Bug-NB: NB#219982
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Tomas Bzatek <tbzatek@redhat.com>

GDBusConnection: propagate error correctly if flags are inappropriate

Previously, this would fail the assertion
"connection->initialization_error != NULL" after the label "out".

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=665067
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: David Zeuthen <davidz@redhat.com>

Link to libpcre correctly when static linking

https://bugzilla.gnome.org/show_bug.cgi?id=665298

Updated Spanish translation

GDBusInterfaceSkeleton: make it possible to export on multiple connections

This is useful in peer-to-peer connections.

With minor changes by David Zeuthen <davidz@redhat.com>.

https://bugzilla.gnome.org/show_bug.cgi?id=662718

Signed-off-by: David Zeuthen <davidz@redhat.com>

GVariant: add preconditions to ref. counting functions

These prevent GVariants from accidentally being brought back to life after
being freed, and should make it easier to track down ref. counting issues.

Closes: bgo#665184

Updated Norwegian bokmål translation

GMemoryInputStream/GMemoryOutputStream: fix bug in previous commit

A g_input_stream_read_async() implementation can't call
g_input_stream_read() on itself directly because it will fail because
the pending flag is already set. So fix that by invoking the vmethod
directly rather than calling the wrapper. Likewise with
GMemoryOutputStream.

Add a test to gio/tests/memory-input-stream.c to catch read_async
failures in the future.

GMemoryInputStream/GMemoryOutputStream: make these properly subclassable

The async methods were directly invoking their own implementations of
the sync methods, making it impossible for a subclass to reimplement
them.

https://bugzilla.gnome.org/show_bug.cgi?id=664635

Don't abort if g_ascii_strtod tests fail

https://bugzilla.gnome.org/show_bug.cgi?id=652560

Remove two useless lines of code

https://bugzilla.gnome.org/show_bug.cgi?id=648516

Fix a little comment error in the _GTreeNode struct

https://bugzilla.gnome.org/show_bug.cgi?id=648516

Clarify local_command_line docs

https://bugzilla.gnome.org/show_bug.cgi?id=641720

Do not use static GTypeInfo and GInterfaceInfo

Either g_type_register_static_simple (used by G_DEFINE_TYPE_EXTENDED)
and G_IMPLEMENT_INTERFACE use automatic variables for GTypeInfo and
GInterfaceInfo structs, while tutorials and source code often use
static variables. This commit consistently adopts the former method.

https://bugzilla.gnome.org/show_bug.cgi?id=600161

gnetworkmonitornetlink.c: Make the error messages less geeky

This makes several different errors have identical messages, but
they're pretty much "can't happen" situations anyway.

https://bugzilla.gnome.org/show_bug.cgi?id=664406

gzlibdecompressor.c: fix a comment

Add missing annotation to g_file_set_attribute()

g_file_set_attribute() also permits a NULL value for value_p, and requires it
to be NULL to unset it. Also fix the wrong variable name in the documentation.

Signed-off-by: Martin Pitt <martin.pitt@ubuntu.com>

Forgot one interface...

Use an XML blob for gapplication interface info

This is both more readable and more efficient. Double win !

gtester: Add command line option to skip tests

https://bugzilla.gnome.org/show_bug.cgi?id=664809