summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* ci: allow FreeBSD jobs to failmcatanzaro/sorry-freebsdMichael Catanzaro2021-06-101-0/+2
| | | | | | | | | The FreeBSD CI is flaky. The CI is of only limited use if it fails spuriously. Until a glorious future where GLib has active FreeBSD developers eager to debug its testsuite, it would be pragmatic to accept the CI run as passing if all other platforms are happy.
* Merge branch 'try-tls-binding-tests-again' into 'main'Philip Withnall2021-06-101-8/+1
|\ | | | | | | | | Revert "tests: Deactivate tls-bindings test suite for windows" See merge request GNOME/glib!2130
| * Revert "tests: Deactivate tls-bindings test suite for windows"Philip Withnall2021-06-031-8/+1
| | | | | | | | | | | | | | | | | | This reverts commit 7f2fef5c2691210444a08b2998b003e56b4d59f3. There have been some changes to the gitlab-ci-win32-runner pre-clone script which should be more successful at killing the remaining processes after this test. See https://gitlab.gnome.org/GNOME/glib/-/merge_requests/2008#note_1127126
* | Merge branch '2409-application-must-be-registered' into 'main'Philip Withnall2021-06-101-0/+3
|\ \ | | | | | | | | | | | | | | | | | | gapplication: Guard g_application_mark_busy() Closes #2409 See merge request GNOME/glib!2111
| * | gapplication: Guard g_application_mark_busy()Peter Bloomfield2021-05-171-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | The `GApplication` must be registered before calling `g_application_mark_busy()`. Document that, and add a guard. The same is true for `g_application_unmark_busy()`, but the existing documentation and guard for `busy_count > 0` are enough.
* | | Merge branch 'dbus-guids-docs' into 'main'Philip Withnall2021-06-104-8/+25
|\ \ \ | | | | | | | | | | | | | | | | docs: Expand documentation about D-Bus GUIDs See merge request GNOME/glib!1812
| * | | gdbusconnection: Fix a typo in a code commentPhilip Withnall2021-06-101-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
| * | | docs: Fix capitalisation of ‘GUID’ in a few placesPhilip Withnall2021-06-102-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
| * | | docs: Expand documentation about D-Bus GUIDsPhilip Withnall2021-06-103-5/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Clarify that the terms ‘GUID’ and ‘UUID’ are used interchangeably in the context of D-Bus, and that neither of them are an RFC 4122 UUID. Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
* | | | Merge branch 'gtypeof' into 'main'Philip Withnall2021-06-109-55/+60
|\ \ \ \ | |/ / / |/| | | | | | | | | | | glib_typeof: Move definition to its own header See merge request GNOME/glib!1969
| * | | glib_typeof: Move definition to its own headerXavier Claessens2021-05-179-55/+60
| |/ / | | | | | | | | | | | | | | | It is cleaner to define glib_typeof() in a header included after gversionmacros.h so we can use GLIB_VERSION_MIN_REQUIRED directly instead of doing it everywhere glib_typeof() is used.
* | | Merge branch 'fix_more_warnings' into 'main'Philip Withnall2021-06-102-2/+2
|\ \ \ | | | | | | | | | | | | | | | | Fix more warnings See merge request GNOME/glib!2119
| * | | Fix dropped 'const' qualifier warning in glib/tests/once.cEmmanuel Fleury2021-06-091-1/+1
| | | | | | | | | | | | | | | | glib/tests/once.c(199): warning C4090: 'function': different 'const' qualifiers
| * | | Fix comparison of unsigned expression < 0 is always false warning in ↵Emmanuel Fleury2021-06-091-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | glib/gunicollate.c glib/gunicollate.c:444:17: warning: comparison of unsigned expression < 0 is always false if (xfrm_len < 0 || xfrm_len >= G_MAXINT - 2) ~~~~~~~~ ^ ~
* | | | Merge branch 'feature/tls-cert-additional-fields' into 'main'Michael Catanzaro2021-06-105-0/+153
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | tls: expose SAN details on GTlsCertificate See merge request GNOME/glib!2142
| * | | | tls: expose SAN details on GTlsCertificateRoss Wollman2021-06-105-0/+153
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changeset exposes * `dns-names` * `ip-addresses` on GTlsCertificate provided by the underlying TLS Backend. See https://gitlab.gnome.org/GNOME/glib-networking/-/merge_requests/165 for the corresponding glib-networking changes. Relates: https://gitlab.gnome.org/GNOME/glib/-/merge_requests/2113 Relates: https://gitlab.gnome.org/GNOME/glib-networking/-/merge_requests/156/diffs Relates: https://github.com/microsoft/playwright/issues/6759
* | | | Merge branch '229-regex-match' into 'main'Emmanuele Bassi2021-06-092-10/+25
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gregex: Fix return from g_match_info_fetch() for unmatched subpatterns Closes #229 See merge request GNOME/glib!1751
| * | | | gregex: Fix return from g_match_info_fetch() for unmatched subpatternsPhilip Withnall2021-06-092-10/+25
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If there were more subpatterns in the regex than matches (which can happen if one or more of the subpatterns are optional), `g_match_info_fetch()` was erroneously returning `NULL` rather than the empty string. It should only return `NULL` when the `match_num` specifies a subpattern which doesn’t exist in the regex. This is complicated slightly by the fact that when using `g_regex_match_all()`, more matches can be returned than there are subpatterns, due to one or more subpatterns matching multiple times at different offsets in the string. This includes a fix for a unit test which was erroneously checking the broken behaviour. Thanks to Allison Karlitskaya for the minimal reproducer. Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Fixes: #229
* | | | Merge branch 'confusing_errors' into 'main'Philip Withnall2021-06-091-5/+22
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gdbusconnection: removed a confusing message Closes #793 See merge request GNOME/glib!2118
| * | | | gdbusconnection: removed a confusing messagenitinosiris2021-06-081-5/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The message `No such interface %s on object at path %s` displayed when requested object does not exist, which was kind of confusing. Closes #793
* | | | | Merge branch 'utkarsh181-main-patch-34260' into 'main'Philip Withnall2021-06-091-6/+6
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | compiling.xml: Don't recommend backticks See merge request GNOME/glib!2143
| * | | | compiling.xml: Don't recommend backticksUtkarsh Singh2021-06-091-6/+6
|/ / / /
* | | | Merge branch 'resimplify-w32-veh' into 'main'Philip Withnall2021-06-083-146/+221
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Re-simplify exception handling on Windows See merge request GNOME/glib!2031
| * | | | Fix the math in copy_charsLRN2021-06-081-10/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now we end up returning a pointer to the end of the buffer after we run out of space. On subsequent calls copy_count will end up being 0.
| * | | | Test the wchar_t version of pid-event subst routineРуслан Ижбулатов2021-06-081-29/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also move env setup earlier in the test, to ensure that the child gets the envvars during initialization. Also, don't look for exception codes in stderr, since OutputDebugStringA() doesn't dump stuff there.
| * | | | Convert the crash handler to UTF-16, mostlyРуслан Ижбулатов2021-06-082-37/+37
| | | | |
| * | | | W32: Remove allocations from the crash handlerРуслан Ижбулатов2021-06-081-38/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use OutputDebugStringA() instead of fprintf. The goal for this code is to inform the person running the debugger about the exception that caused the debugger to be attached. This is useful for debugging with gdb, because gdb does not catch Windows exception information (it just displays "Segmentation fault"). OutputDebugStringA() ensures that the output goes to the debugger, and the (ab)use of strcpy() with a stack-allocated buffer ensures that we do not allocate anything while the crash handler is running, nor to we call CRT functions that can be reasinably expected to allocate anything.
| * | | | W32: Initialize debugger stuff in advanceРуслан Ижбулатов2021-06-081-65/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since VEH is invoked when an exception occurs (which, for us, is mostly when the program is already crashing), we should try to avoid doing much processing at that point. Since these things (debugger commandline, a list of extra exceptions to catch) are known in advance, set them up during initialization.
| * | | | W32: Don't use g_getenv() in crash handler or during initializationРуслан Ижбулатов2021-06-081-23/+29
|/ / / / | | | | | | | | | | | | | | | | | | | | The first is to avoid any non-trivial code in the crash handler. The second is to avoid the use of quarks and hash tables (brought in by g_getenv()) during GLib initialization.
* | | | Merge branch '2418-more-stupid-atomics' into 'main'Emmanuele Bassi2021-06-072-38/+63
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | gbitlock: Drop unnecessary volatile qualifiers Closes #2418 See merge request GNOME/glib!2131
| * | | | gbitlock: Minor improvement to documentation formattingPhilip Withnall2021-06-031-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
| * | | | gbitlock: Drop unnecessary volatile qualifiersPhilip Withnall2021-06-031-37/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This follows on from !1719. It drops volatile qualifiers on internal functions in `gbitlock.c`, and casts volatile public arguments to non-volatile versions to avoid the `g_atomic_*()` macros from propagating the volatile qualifier. We can’t drop the `volatile` qualifier from the public API in `gbitlock.h`, as that would unfortunately be an API break. Update the documentation in `gbitlock` to mention that users of the API should not use `volatile`. See http://c.isvolatileusefulwiththreads.com/ Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Fixes: #2418
| * | | | tests: Ignore warning about use of volatile from a test for exactly thatPhilip Withnall2021-06-031-0/+3
| | |_|/ | |/| | | | | | | | | | | | | | | | | | Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Helps: #2418
* | | | Merge branch 'Garray' into 'main'Philip Withnall2021-06-071-0/+21
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | doc: example for GArray and g_array_set_clear_func() Closes #626 See merge request GNOME/glib!2124
| * | | | doc: example for GArray and g_array_set_clear_func()Nitin Wartkar2021-06-071-0/+21
|/ / / /
* | | | Merge branch 'fix/accidental-dllexport' into 'main'Philip Withnall2021-06-072-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | gdbus, win32: Fix accidental dllexport in static builds See merge request GNOME/glib!2139
| * | | | gdbus, win32: Fix accidental dllexport in static buildsOle André Vadla Ravnås2021-06-072-2/+2
| | | | | | | | | | | | | | | | | | | | Should only dllexport g_win32_run_session_bus() when building a DLL.
* | | | | Merge branch '2348-rename-master' into 'master'Philip Withnall2021-06-0723-35/+48
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rename default development branch to main Closes #2348 See merge request GNOME/glib!2140
| * | | | | docs: Add a section to the README about the branch renamePhilip Withnall2021-06-071-0/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Helps: #2348
| * | | | | Rename the master branch to mainPhilip Withnall2021-06-074-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It’s a more inclusive name, has the same tab-completion prefix, and is the default choice for new repositories created locally by git, and on GitHub and GitLab. https://sfconservancy.org/news/2020/jun/23/gitbranchname/ Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Fixes: #2348
| * | | | | docs: Update various external links to use HEAD instead of masterPhilip Withnall2021-06-0717-26/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update several links to allow the remote to use its configured default branch name, rather than specifying `master` as the default branch name. This will help avoid breakage if any of these projects rename their default branch in the future. Fix a few of the links where they were hitting redirects or had moved. Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Helps: #2348
| * | | | | ci: Allow origin to choose default branch for gobject-introspectionPhilip Withnall2021-06-071-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rather than specifying the default branch explicitly, allow the origin to choose it. Specify the `--single-branch` option explicitly to make it clear the CI only needs one branch; this is however already implied by the `--depth` option so is not strictly necessary. This will help avoid breakage if gobject-introspection changes its default branch name in future. Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Helps: #2348
| * | | | | subprojects: Pin gtk-doc to a specific versionPhilip Withnall2021-06-071-1/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | One of the points of subprojects is to allow building against a version of a dependency that the project has been tested against, so don’t try and build against gtk-doc master. Building against master is actually currently fine, but is a slight risk. Making this change also removes one more instance of `master` from `git grep master` results, which helps #2348. Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Helps: #2348
* | | | | Merge branch '2417-file-replace-contents-etags' into 'master'Emmanuele Bassi2021-06-072-1/+153
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | glocalfileoutputstream: Fix ETag check when replacing through a symlink Closes #2417 See merge request GNOME/glib!2138
| * | | | tests: Add missing setlocale() call to file testsPhilip Withnall2021-06-071-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This ensures that non-ASCII characters in the test output are printed correctly in the logs. Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
| * | | | glocalfileoutputstream: Fix ETag check when replacing through a symlinkPhilip Withnall2021-06-072-1/+150
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit 87e19535fe, the ETag check when writing out a file through a symlink (following the symlink) has been incorrectly using the ETag value of the symlink, rather than the target file. This is incorrect because the ETag should represent the file content, not its metadata or links to it. Fix that, and add a unit test. Signed-off-by: Philip Withnall <pwithnall@endlessos.org> Fixes: #2417
* | | | Merge branch '2416-tls-certificate-fix-invalid-read' into 'master'Philip Withnall2021-06-072-3/+38
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tests/tls-certificate: Add a unit test Closes #2416 See merge request GNOME/glib!2136
| * | | | tests/tls-certificate: Add context to the new testPeter Bloomfield2021-06-051-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | Add a comment to `pem_parser_no_sentinel()` to explain what it is testing and how it works.
| * | | | tlscertificate: Avoid possible invalid readPeter Bloomfield2021-06-041-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | In various places, do not read past the end of the data. Fixes https://gitlab.gnome.org/GNOME/glib/-/issues/2416
| * | | | tests/tls-certificate: Add a unit testPeter Bloomfield2021-06-041-0/+30
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Test whether g_tls_certificate_new_from_pem() can handle a non-nul-terminated PEM. Helps https://gitlab.gnome.org/GNOME/glib/-/issues/2416
View Lorry Controller Status