| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
| |
Commit 99bdfd1b introduced a direct call to pthreads_setname_np in the
'thread' test case. Because we are directly calling pthreads functions
from this file now, we need to make sure we link it with the system
thread library flags (as we already do for another file).
https://bugzilla.gnome.org/show_bug.cgi?id=765712
|
|
|
|
|
|
|
| |
This ensures that g_socket_get_fd() will return -1 after the socket has
been closed.
https://bugzilla.gnome.org/show_bug.cgi?id=765959
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Later Visual Studio versions does not allow one to define known keywords,
even if they are actually not known to the compiler. Avoid this issue by
checking more conditions before we define inline as __inline:
-We are not building under C++ mode.
-We are on Visual Studio 2013 or earlier.
Where both of these conditions need to hold true.
https://bugzilla.gnome.org/show_bug.cgi?id=765990
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This reverts commit ca189aa32138c0127db2f9bd99d28b121393b063.
The new g_autoptr symbol definition in GDBus generated code breaks
existing code that uses gdbus-codegen and defines its own auto clean up
symbols.
This needs further discussion before cherry-picking to the stable
branch.
https://bugzilla.gnome.org/show_bug.cgi?id=763379
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The rest of the generated classes gained g_autoptr support in fd6ca66,
but this one is still missing. Because whatever_proxy_new_finish() and
whatever_proxy_new_sync() are declared as returning a Whatever *
instead of a WhateverProxy *, and the generated method-call stubs
act on a Whatever *, it's reasonably common to want to declare a
g_autoptr (Whatever).
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugzilla.gnome.org/review?bug=763379
Reviewed-by: Colin Walters <walters@verbum.org>
(cherry picked from commit cbbcaa4dd70a5b28ea62bf408d4fc3e1e2f7f507)
|
|
|
|
|
|
|
| |
v_long is 32 bits on Win64, v_pointer is 64 bits. On most other platforms the
size of long and pointer is the same, so it's usually not a problem.
https://bugzilla.gnome.org/show_bug.cgi?id=758738
|
|
|
|
|
|
|
| |
Commit f45ec47 fixed a race condition in this test, but one change was
omitted; maybe lost in a rebase.
https://bugzilla.gnome.org/show_bug.cgi?id=749606
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=748530
|
|
|
|
|
|
|
| |
Add a test case for unreffing an object from a GContextSpecificGroup
immediately after firing a signal, before allowing the mainloop to run.
https://bugzilla.gnome.org/show_bug.cgi?id=762994
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GContextSpecificGroup has been somewhat broken for a rather long time:
when we remove the last reference on an object held in the group, we try
to clean up the source, but fail to actually remove it from the
mainloop.
We will soon stop emitting signals on the source (due to it having been
removed from the hash table) but any "in flight" signals will still be
delivered on the source, which continues to exist. This is a problem if
the event is being delivered just as the object is being destroyed.
This also means that we leave the source attached to the mainloop
forever (and next time will create a new one)...
This is demonstrated with the GtkAppChooser dialog which writes an
update to the mimeapps.list file just as it is closing, triggering the
app info monitor to fire just as it is being destroyed.
Karl Tomlinson correctly analysed the problem and proposed this fix.
https://bugzilla.gnome.org/show_bug.cgi?id=762994
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This works by using semi-documented[1] exception to tell the debugger
that a thread needs to have its name changed.
If this exception is not caught and handled by something, it will crash
the process, so we need to set up our own handler in case there's no
debugger attached or the debugger can't handle this type of exception.
Since SEH is not supported by gcc on i686 (at the moment), we need to use VEH
instead. For completeness the MSVC-oriented code still uses SEH, although
there is no reason why it shouldn't work with the VEH variant used by MinGW.
VEH handler has to be set up somewhere (g_thread_win32_init () works nicely)
and removed once it's not needed (g_thread_win32_process_detach () is added
expressly for that purpose). Note that g_thread_win32_process_detach() is
only called when glib is unloaded by FreeLibrary(), not when glib-using
process is terminating.
This exception is known to work with WinDbg, and adding support for it into
GDB proved to be feasible (GDB patch will be sent upstream, eventually).
[1] https://msdn.microsoft.com/en-us/library/xcb2z8hs%28v=vs.71%29.aspx
https://bugzilla.gnome.org/show_bug.cgi?id=747478
|
|
|
|
|
|
| |
We now prefer pthread_setname_np when available, and don't
need the linux specific API anymore. Also change the test
for this functionality to use pthread_getname_np.
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=747478
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GApplication has accepted any valid bus name as an application ID since
before the time of D-Bus activation. This includes bus names with '-'.
Several applications have even attempted support bus activation with
these names, going as far as installing D-Bus service files, without
realising that they are silently falling back to fork()/exec() on
account of the name containing a dash.
The reason for the problem is that D-Bus object paths cannot contain
dashes. We solved this problem privately in an unspecified way inside
of GApplication but substituting '_' in this case, but never made this
part of the Desktop Entry Specification.
The fact that these apps with '-' in the desktop file names aren't
actually using D-Bus activation is beside the point: their intent here
was clear. Let's avoid forcing them to rename their desktop files again
by simply accepting '-' in desktop file names and munging the path in
the way that GApplication did so historically.
The new path escaping code here has been copied more or less verbatim
from GApplication's own code for the same purpose, with only the removal
of one irrelevant part.
An update to the desktop entry specification will follow.
https://bugzilla.gnome.org/show_bug.cgi?id=764754
|
|
|
|
|
|
|
|
|
| |
Some packages might have some parts that are built for certain build
configs, meaning that they could have .pc files of their own, such as
Pango, where PangoFT2 is optionally built. Allow such an option if
needed.
Also remove some trailing whitespaces.
|
|
|
|
| |
This is helpful to people using the g++ --pedantic option.
|
|
|
|
|
| |
D-Bus activation is a thing now; bring the local_command_line
docs in sync with reality.
|
|
|
|
|
|
| |
Fix the example, as well as the consistency in the terms.
https://bugzilla.gnome.org/show_bug.cgi?id=748806
|
|
|
|
|
|
|
|
|
| |
Character entities are not supposed to be supported by gtk-doc¹ and
fix the spelling of ‘optional’
¹https://bugzilla.gnome.org/show_bug.cgi?id=758137
https://bugzilla.gnome.org/show_bug.cgi?id=758174
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=747107
|
|
|
|
|
|
|
|
|
| |
Inserts a paragraph in the start of the description
explaining briefly the concept of GVariant as a
variant datatypes using examples and explaining
a few use cases where GVariant can be useful.
https://bugzilla.gnome.org/show_bug.cgi?id=748806
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=764685
|
|
|
|
|
| |
g_action_parse_detailed_action_name() doesn't exist, it's
g_action_parse_detailed_name() instead.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Newer versions of GCC are particularly verbose in relation to
formatting errors, use GCC pragmas to disable warnings for this
section.
gdatetime.c: In function ‘test_strftime’:
gdatetime.c:1334:3: error: ‘%c’ yields only last 2 digits of year in some locales [-Werror=format-y2k]
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^
gdatetime.c:1334:3: note: in definition of macro ‘TEST_FORMAT’
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gdatetime.c:1334:3: error: ‘%g’ yields only last 2 digits of year [-Werror=format-y2k]
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^
gdatetime.c:1334:3: note: in definition of macro ‘TEST_FORMAT’
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gdatetime.c:1334:3: error: ‘%x’ yields only last 2 digits of year in some locales [-Werror=format-y2k]
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^
gdatetime.c:1334:3: note: in definition of macro ‘TEST_FORMAT’
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
gdatetime.c:1334:3: error: ‘%y’ yields only last 2 digits of year [-Werror=format-y2k]
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^
gdatetime.c:1334:3: note: in definition of macro ‘TEST_FORMAT’
"a%a A%A b%b B%B c%c C%C d%d e%e F%F g%g G%G h%h H%H I%I j%j m%m M%M " \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Note that the pragma is outside the function as older versions of GCC
don't support pragma inside functions.
https://bugzilla.gnome.org/show_bug.cgi?id=764575
|
|
|
|
|
|
|
|
| |
The current statfs() compilation tests all fail because statfs() expects
the first argument to be non-null. Pass a dummy path instead of NULL to
satisfy the compiler.
https://bugzilla.gnome.org/show_bug.cgi?id=764574
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=764092
|
|
|
|
|
|
|
| |
The behaviour of g_strjoinv() isn't explicitely explained when the array
contains less than 2 items. This removes the guesswork.
https://bugzilla.gnome.org/show_bug.cgi?id=764092
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=749606
|
|
|
|
|
|
|
|
| |
This was confusing some static analysis. Through canonicalize_filename()
at construction time, we guaranteed that ->filename is canonical and
absolute, so g_path_skip_root() should never fail.
https://bugzilla.gnome.org/show_bug.cgi?id=731988
|
| |
|
|
|
|
| |
Signed-off-by: Trần Ngọc Quân <vnwildman@gmail.com>
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=748474
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Mirrors
https://git.gnome.org/browse/glib/commit/?id=ad2092bc680e434c3d17600988ec9b20f52eebef
except I didn't also change gobject/.
https://bugzilla.gnome.org/show_bug.cgi?id=763821
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This ensures that the generated file is always the same (not dependent
on the build machine's environment), making the build reproducible.
Thanks to Jérémy Bobbio <lunar@debian.org> for the Debian bug report and
patch.
https://bugzilla.gnome.org/show_bug.cgi?id=763617
|
|
|
|
| |
Spotted by Coverity (CID: #1353385).
|
|
|
|
|
|
|
|
| |
find_file_in_data_dirs() doesn’t actually clear output_path to NULL on
failure, so this prevents a use-after-free on that (fd == -1) error
path.
Spotted by Coverity (CID: #1352981).
|
| |
|
| |
|
| |
|
|
|
|
| |
Spotted by Coverity (CID: #1325405).
|
|
|
|
| |
Spotted by Coverity (CID: #1352961).
|
|
|
|
| |
Clarify that it’s a critical //warning//, not a critical something-else.
|
|
|
|
|
|
|
|
| |
Use the appropriate type for comparisons and assignment of wide chars.
Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=762202
|
|
|
|
|
|
| |
Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=762202
|
|
|
|
|
|
| |
Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com>
https://bugzilla.gnome.org/show_bug.cgi?id=762202
|