Updates

gdbus-codegen: don't assert non-negativity for unsigned values

The code generator sprinkles a few asserts in its output of the form:

g_assert (prop_id - 1 >= 0 && prop_id - 1 < %d);\n

prop_id is unsigned, though, so this generates a compiler warning for
me.

This commit changes the code to merely check for prop_id != 0 instead of
prop_id - 1 >= 0

GVariant: Better handling of invalid UTF-8

g_variant_new_string() hits a g_return_if_fail() when given invalid
UTF-8.  That's certainly the right thing to do, but
g_variant_builder_add() uses this function internally and crashes when
it returns NULL due to the user passing a non-utf8 string.

We can protect the internal code by returning "[Invalid UTF-8]" in this
case while also making the problem easier to debug (and less fatal).

Closes #632631.

gdbus-codegen: Make sure we pass -Werror

_g_value_equal might not always be used, so mark it G_GNUC_UNUSED
to make sure we're -Werror clean.

Updated Norwegian bokmål translation

GDBusObjectManagerServer: Allow setting :connection property

.. and add a C setter to do this. Also make the C getter return a
reference since the property may be set from another thread. Also
change the constructor to _not_ take a GDBusConnection since this is
something you almost always want to do _after_ creating it. The
API/ABI break is fine as there has never been a GLib release with this
type.

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

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

Rename g_unix_pipe_flags to g_unix_open_pipe

From IRC discussion, people liked this name more.

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

Remove extraneous decorations

Since in a doc comment takes no leading '@'.

glib-unix.c: Don't leak FDs if pipe2() succeeds

Only remove the lines that are no longer needed

Revert "[build] Remove stale LDADD lines from tests/Makefile.am to allow build"

This reverts commit b0874ce28e910121fb2d94f6f9183bfdc08e8eaf.

Use g types in public API for consistency

GCancellable: Use g_unix_pipe_flags with FD_CLOEXEC

The old code was creating a pipe and setting FD_CLOEXEC
non-atomically.

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

g_unix_set_fd_nonblocking: New API to control file descriptor blocking state

And use it in relevant places in GLib.

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

glib-unix.h: Unconditionally include unistd.h

danw points out it's part of POSIX.

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

Fix usage of _GNU_SOURCE

_GNU_SOURCE must be defined before including any other (system)
header, so defining it in glib-unix.h (and hoping no one has included
anything else before that) is wrong. And the "#define _USE_GNU"
workaround for this problem in gnetworkingprivate.h is even wronger
(and still prone to failure anyway due to single-include guards).

Fix this by defining _GNU_SOURCE in config.h when building against
glibc. In theory this is bad because new releases of glibc may include
symbols that conflict with glib symbols, which could then cause
compile failures. However, most people only see new releases of glibc
when they upgrade their distro, at which point they also generally get
new releases of gcc, which have new warnings/errors to clean up
anyway.

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

Add back an erroneously-removed line to tests/Makefile.am

Drop translated g_warning/g_error

http://mail.gnome.org/archives/desktop-devel-list/2011-March/msg00069.html

glib/Makefile.am: add gmain-internal.h to sources

It's a new private header.

This commit adds it to the Makefile so it's properly distributed
with the sources.

gobject: Warn if someone uses the wrong property in g_object_get/set

"Interface::property" was not detected before and led to a crash.

Added UG translation

Fix compiler warnings

Fix autoconf 2.68 warnings

autoconf 2.68 is very insistent that AC_LANG_SOURCE/AC_LANG_PROGRAM
must be used in certain places, to avoid quoting/lack-of-quoting
problems, or something. Fix.

Bug 648966 - Update g_unichar_iswide and g_unichar_iswide_cjk

Add tests.

[build] Remove stale LDADD lines from tests/Makefile.am to allow build

Automake 1.11 wasn't happy.

Update .gitignores

Some documentation additions

Avoid an unused variable warning

Also, clean up some whitespace

Fix make check

g_thread_init() causes a hash table to be allocated (in read_aliases).
Since hash tables are now a bit larger, we need to bump one of the
probe sizes to avoid our probe slice being used for the aliases
hash table.

Added UG translation

GHash: make sets with refcounted keys work correctly

When keys == values, we have to be careful about the order in
which we replace their elements.

GHash: introduce a "set" mode

Make hash tables start out in a mode in which they don't store
values at all, until the first insertion of a non-identical
key-value pair.

This reduces memory requirements by 1/3 when using hash tables
to store sets.

Based on a patch by Morten Welinder,
https://bugzilla.gnome.org/show_bug.cgi?id=644437

GHash: eliminate one of the lookup_nodes functions.

Kill g_hash_table_lookup_node and rename g_hash_table_lookup_node_for_insertion
to g_hash_table_lookup_node.  Since at this point we already check for
toombstones in all callers of g_hash_table_lookup_node this doesn't make
a difference.

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

GHash: split nodes array into seperate arrays.

This reduces memory requirements by 1/6 on 64-bit machines since
no padding is needed.  It also puts less strain on the memory
allocator since we no longer need one giant slab of memory.

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

GHash: use macros to check for magic hash values.

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

Update VS2008/2010 README.txt

Tell people about the usage of ZLib to avoid the
problem such as the one mentioned in Bug 633977

Bug 648966 - Update g_unichar_iswide and g_unichar_iswide_cjk

Update to Unicode 6.0.  Also attach Python script that generates
the tables.

Remove duplicate GDBusInterface declaration

It was in gdbusinterface.h but also in giotypes.h.  Remove it from the
former.

Update docs to take comments/suggestions from Matthias into account

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

Merge branch 'master' into gdbus-codegen

GFilterInputStream: close-base-stream should not be construct-only

This is a runtime changable property (by the methods for it), must
be a cut and paste error.

Use SIGTRAP (via G_BREAKPOINT()) if G_DEBUG=fatal-warnings

On Linux with gdb, it's much more convenient to debug programs using
G_DEBUG=fatal-warnings if we send SIGTRAP instead of abort() by
default.  The default handler for both is to terminate the process.

In particular this makes it more easily possible to debug a warning
that's not the first in a program; you can skip past it and
go to the warning you care about.

The "aborting..." message is removed since it's no longer accurate,
and anyways was never very useful; crashes should show up in ABRT/apport
type crash catching systems.

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

Updated Spanish translation

glib/gvariant.c: Fix variable name in example code

Drop stray reference to gthread.la

We now build this test multithreaded in gthread/tests.

glib/Makefile.am: Remove stray tab

glib-unix: New API to watch some Unix signals

This new API allows watching a few select Unix signals;
looking through the list on my system, I didn't see anything
else that I think it'd reasonable to watch.

We build on the previous patch to make the child watch helper thread
that existed on Unix handle these signals in the threaded case.
In the non-threaded case, they're just global variables.

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

gmain: Prepare child watch handling for more generic signal handling

In preparation for supporting more Unix signals such as SIGHUP,
SIGTERM etc.,

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

glib-unix: New Unix-specific API

GLib historically has been designed to be "mostly" portable; there
are some functions only available on Unix like g_io_channel_unix_new(),
but these are typically paired with obvious counterparts for Win32.

However, as GLib is used not only by portable software, but components
targeting Unix (or even just Linux), there are a few cases where it
would be very convenient if GLib shipped built-in functionality.

This initial patch is a basic wrapper around pipe2(), including
fallbacks for older kernels.  This pairs well with the
existing g_spawn_*() API and its child_setup functionality.

However, in the future, I want to add a signal() wrapper here,
complete with proxying the signal to a mainloop.  I have initial code
for this, but doing it sanely (including factoring out gmain.c's
private worker thread), is a complex task, and I don't want to block
on that.

See also gwin32.h for Win32 specific functionality.

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

Update gio.symbols for previous two commits

GDesktopAppInfo: Add g_desktop_app_info_get_generic_name

Necessary for rebasing gnome-menus on top of GDesktopAppInfo.

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

GDesktopAppInfo: Add g_desktop_app_info_get_categories()

This is needed to rebase gnome-menus on top of GDesktopAppInfo.

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

Fix %z in g_date_time_format()

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

GHashTable: Small optimization of remove-all

Don't enter the loop if we are not going to notify anyway.
Pointed out in bug 646013.

GHashTable: Add a test for remove-all functionality

Whitespace cleanup

Allow caching have_qsort_r which re-enables cross-compiling

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

Fix mailing list link in README

Patch by Thomas Andersen, bug 647594

GDBusServer: Documentation: Improvements.

Provide a fuller description and lead people away if they arrived
here just looking for a way to provide a regular D-Bus service.

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

Use G_SIGNAL_MUST_COLLECT for VARIANT signals

Bug #643624.

Fix up some harmless FALSE <> NULL confusions

Reported in bug 643134.

GSequence: Make g_sequence_iter_move behave as documented

As pointed out in bug 658313, moving before the begin iter is
supposed to return the begin iter, not the end iter. Also add
a test for this behaviour.

GDesktopAppInfo: Add "filename" property for bindings

GDesktopAppInfo violates the GObject rule that your C constructors
should just be thin wrappers around g_object_new().  While GKeyFile
isn't introspctable, this patch allows from JavaScript:

var app = new Gio.DesktopAppInfo({ filename: '/path/to/foo.desktop' });

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

Squash some uninitialized variable compiler warnings

From GCC 4.6.

GSimpleAsyncResult: push thread context around callback

When an old pre-thread-default-context API that takes an explicit
GMainContext wants to call a gio API, it must call
g_main_context_push_thread_default() before, and
g_main_context_pop_thread_default() after the gio call, so that the
gio method will return its result to the desired GMainContext.

But this fails for methods like g_socket_client_connect_async() that
make a chain of multiple async calls, since the pushed/popped context
will only affect the initial call.

Fix this by having GSimpleAsyncResult itself push/pop the context
around the callback invocation, so that if the callback queues another
async request, it will stay in the same context as the original one.

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

Updated Turkish translation

gdbus-codegen: Fix typo in generated docs

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

gdbus-codegen: Generate GDBusObject{,Proxy,Skeleton} subtypes

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

Don't ignore SUPPORTS_STARTUP_NOTIFICATION for commandline GAppInfos

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

Add VS 2008 compilation support for some utilities

-Added projects to compile the glib-compile-schemas and gsettings utilities
-Update .vsprops to install these in "install" phase
-Distribute these projects also

Add VS 2010 compilation support for some utilities

-Added projects to compile the glib-compile-schemas and gsettings utilities
-Update .vsprops to install these in "install" phase
-Distribute these projects also

Fix build broken by previous commit

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

gdbus-codegen: Include docs for generated code in the GIO docs

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

docs: fix typos in networking classes

GDBusObjectManagerServer: make export() return whether the object was removed

This is useful in conjunction with g_warn_if_fail().

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

GSocketService: clarify transfer semantics of incoming connections

The @connection parameter to the ::incoming signal is (transfer none),
so you need to ref it if you want to keep it.

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

gdbus-codegen: Document the generated FOO_PEEK_BAR() and FOO_GET_BAR() macros

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

gdbus-codegen: Generate gtk-doc section docs

This is nice to have if using gtk-doc on the generated code. We could
also generate -sections.txt and .types files but we don't do that
right now...

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

gdbus-codegen: Set nick+blurb for properties to shut up gtk-doc

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

gdbus-codegen: It's GDBusProxyTypeFunc, not GDBusProxyFunc

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

gdbus-codegen: Document the generated ObjectManagerClient structs

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

gdbus-codegen: Add "Returns:" blurb for generated get_proxy_type()

This was forgotten.

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

gdbus-codegen: Generate docs when generating C code

This is handy for both gtk-doc and gobject-introspection.

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

API: testutils: Add g_test_fail()

This allows tests to fail in a nonfatal way and the test runner can
continue if invoked with -k.

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

testutils: Sprinkle code with newlines

Readable code ftw!

testutils: Return number of bad tests from g_test_run_suite_internal()

In particular do not return a boolean disguised as an int.

testutils: Remove unused variable

testutils: Return a boolean from g_test_case_run()

Return value is intened to be TRUE for success, FALSE for failure.
Currently we return TRUE all the time.
Previously the test returned 0 all the time.

GDBusObjectManagerClient: Add a GDestroyNotify to the user_data

For bindings, obviously.

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

GDBus: Add some gobject-introspection annotations for new code

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

gdbus-codegen: Make generated code introspectable

Specific changes

 - Use get_type(), not get_gtype() for the GType function
   - so we need to use the lower-case name type_ for properties called type
 - Don't return a function pointer, just make the function returned
   available instead
 - Add (type) annotations in constructors so g-ir-scanner detects them as such
 - Add (transfer none) annotations to property getters
 - Add (out) annotations to D-Bus method call functions

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

gdbus-codegen: Nuke debug spew

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

gdbus-codegen: Never emit empty PropertiesChanged signals

It's not that it's actually a bug to do so per se, strictly speaking,
it's just pointless and wasteful.

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

gdbus-codegen: Don't send out PropertiesChanged if value ends up not changing

A fairly typical pattern is to have code that does

 foo_set_bar (object, "");
 if (some_condition)
   {
     foo_set_bar (object, "yes");
   }

where some_condition is often true every time @object is updated.

With this code, bar is essentially always "yes" but because of how
gdbus-codegen works, useless PropertiesChanged events got scheduled
and sent out. With this patch, we avoid that by always keeping the
original value around and comparing it only when we deem it's time to
send out the ::PropertiesChanged signal (typically in an idle but can
be forced by the user via flush()).

Also add a test case for this.

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

Fix some compiler warnings from gcc 4.6

gapplication: Fix typo in property

gdbus-codegen: Fix obvious bug in _g_variant_equal0() implementation

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

gdbus-codegen: Add support for the org.freedesktop.DBus.Deprecated annotation

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

GApplication: #include "gsettings.h"

Since we call g_settings_sync() from there now...

Merge remote-tracking branch 'gvdb/master'