Commit Graph

607 Commits

Author SHA1 Message Date
Philip Chimento
a458c66cbf girepository: Return pointer array for interface cache
In g_irepository_get_object_gtype_interfaces(), returning the address of
the first GIBaseInfo* does not work reliably, because the GIBaseInfos
are not necessarily stored contiguously. So the second and subsequent
ones might be garbage.

Instead, return the address of the array of GIBaseInfo pointers.

Add a test that verifies the functionality, as well.

This is unfortunately an API and ABI break.
2019-06-20 11:54:10 -07:00
Chun-wei Fan
1a19f05a0e build: Force-include msvc_recommended_pragmas.h on Visual Studio
By doing so, we essentially cover the various compiler flags that we
want to use for non-Visual Studio builds to check for warnings that
might cause real concern.

This also skips the checks for the various GCC-isque CFlag checks that
are scattered in the various build files on Visual Studio builds, since
they are essentially meaningless on Visual Studio builds.
2019-04-30 16:07:39 +08:00
Elliott Sales de Andrade
6747ec55dd Fix a possible use-after-free.
If g_mapped_file_new fails, then `version` will be freed, but it was
already added to the hash table. This means there could be a
use-after-free while doing a lookup on the hash table the next time.
2019-02-02 03:00:15 -05:00
Elliott Sales de Andrade
d17edde568 Fix possible leak of transitive dependency name.
If a transitive dependency appears twice, the original pointer will be
removed from the hash table. Since these names were created by
g_strsplit, they need to be freed, or they will leak.
2019-02-02 03:00:15 -05:00
Rico Tzschichholz
4c0a8388d5 repository: Fix annotation for g_irepository_get_object_gtype_interfaces()
Introduced by 9826d952358c8330d72ecba062f489fbdc31bbd1
2019-01-16 21:04:26 +01:00
Colin Walters
8de3abd642 repository: g_irepository_get_object_gtype_interfaces
Bindings in some cases need to look up information from a GType
dynamically.  Support that better by supplying a cache for this
information.

(Rebased and versioning / gtk-doc stuff added by Philip Chimento.)

Closes #38.
See gjs#55.
2019-01-09 21:44:10 -08:00
Christoph Reiter
9b367e74ef ci: enable -Werror for msys2+meson 2019-01-01 18:06:47 +01:00
Tomasz Miąsko
b12b08df2f Cleanup strict-prototypes warnings 2018-12-20 17:34:38 +01:00
Christoph Reiter
13ba457c22 meson: add default compiler warning flags and enable -Werror on CI
This enables various compiler warnings project wide and disables the triggered
ones for each library/executable. This should give us roughly the same behaviour
as with autotools.

Tested with gcc8 and clang7.
2018-12-18 21:39:33 +00:00
Emmanuele Bassi
cb1d599495 meson: Use copy instead of an empty configuration
The configure_file target has grown a `copy` argument to avoid using an
empty configuration_data object since Meson 0.47.
2018-12-18 12:32:53 +00:00
Christoph Reiter
487e670b1f girepository: Add version macros and functions. Fixes #200
This adds the following macros and functions: GI_MAJOR_VERSION, GI_MICRO_VERSION,
GI_MINOR_VERSION, GI_CHECK_VERSION, gi_get_major_version,gi_get_micro_version,
gi_get_minor_version.

Since we share a prefix with glib we have to namespace these by using the gi_
prefix. g_gi would also work but we already export symbols with gi_ like
gi_cclosure_marshal_generic(), gi_type_tag_get_ffi_type() and
gi_type_info_extract_ffi_return_value(), so let's not add another naming scheme.
2018-12-16 12:51:58 +01:00
Christoph Reiter
db90b3b3ae baseinfo: don't abort when calling g_base_info_get_name() on a GITypeInfo. Fixes #96
GITypeInfo is a GIBaseInfo so calling g_base_info_get_name() on it should do something
sensible.

g_base_info_get_name() has always been documented to return NULL in case no name
is available so return that instead.
2018-12-15 23:33:32 +00:00
Jasper St. Pierre
e01e11be42 giobjectinfo: Fix some documentation for find_vfunc_using_interfaces
See #66
2018-12-15 11:09:23 +01:00
Tomasz Miąsko
292afa1d58 docs: Field offsets are in the units of bytes 2018-12-14 16:28:58 +01:00
Christoph Reiter
5e8e1a761e docs: Remove everything not related to libgirepository and restructure. Fixes #244
The gtk-doc docs were including some bits of overal g-i docs but mostly unfinished
and outdated.

We now have the general docs in sphinx so remove the duplication and make the gtk-docs
just about the libgirepository API and nothing more.

This also renames some titles and fixes some missing links in the struct hierarchy
while at it.
2018-12-14 11:29:11 +01:00
Christoph Reiter
722518e5bc docs: fix gtk-doc warnings and update version infos
Move things around and rename things until gtk-doc is happy.

This also moves the "Since" annotations to the next stable releases and
adds version added info for g_callable_info_get_instance_ownership_transfer()
and g_struct_info_find_field().
2018-12-09 18:26:26 +01:00
Thibault Saunier
4b32b6e116 writer: Include documentation and symbol position in source files
Some documentation tool (as hotdoc[0]) need to have information about
symbol declaration and documentation positions in the source files
to be able to do smart indexing (automatically build the documenation
index).

[0] https://hotdoc.github.io/

Fixes #175
2018-11-28 09:29:07 -03:00
Christoph Reiter
8c36d53cba girnode: remove wrongly added g_assert_not_reached(). Fixes #218
Added in 9535fc48. After the switch it calls _g_ir_node_build_typelib() again
handling the other types.
2018-08-03 08:06:08 +02:00
Christoph Reiter
a060582afe girepository: port from g_type_class_add_private() to G_ADD_PRIVATE()
This lets us enable -Wdeprecated-declarations
2018-07-30 07:53:01 +02:00
Christoph Reiter
cff8329f70 build: enable -Wredundant-decls
The ffi decls were added in cbdd9ee09e367e4dd to work around broken
ffi headers. Let's assume this is fixed now.
2018-07-29 18:57:03 +02:00
Christoph Reiter
a718ebac86 build: enable -Wswitch-default
In case the surrounding code handles missing cases break, otherwise add
a g_assert_not_reached().

The generated parser code triggers this as well, so disable it there only.
2018-07-29 18:57:03 +02:00
Christoph Reiter
7be196f295 build: enable -Wtype-limits
hashv is unsigned, no need to check if >= 0
2018-07-29 18:57:03 +02:00
Christoph Reiter
f02e90f265 build: enable -Wimplicit-fallthrough
and fix a missplaced break
2018-07-29 18:57:03 +02:00
Christoph Reiter
3a5b0c2ae2 build: enable -Wshadow 2018-07-29 18:57:03 +02:00
Christoph Reiter
7cdf5d0d1d build: enable -Wcast-function-type
Where it's easy add dummy args to match the cast; where the target is a subset just
prevent the warning with a cast to void*.

Provide a real copy function for the boxed type code in regress_foo.
This code is never executed afaics, but why not.
2018-07-29 18:56:58 +02:00
Christoph Reiter
ee2ec3343e build: enable -Wsuggest-attribute=format
And use G_GNUC_PRINTF for the suggested function.
2018-07-29 16:50:19 +02:00
Christoph Reiter
573fe41788 build: enable -Wdouble-promotion 2018-07-29 16:49:53 +02:00
Christoph Reiter
9018e7b13c build: enable -Wincompatible-pointer-types 2018-07-29 16:49:23 +02:00
Christoph Reiter
16f2c2ad0b build: enable -Wdiscarded-qualifiers
Except for the Python module because nothing in the CPython API is marked const
and we'd have to cast everywhere.
2018-07-29 16:48:18 +02:00
Rico Tzschichholz
c704898e26 gdump: Fix print-format error while GFlagsValue->value is an unsigned int 2018-07-04 16:56:32 +02:00
Chun-wei Fan
4dafb4d5cf girepository: Properly acquire and check pointer values
On Windows (Visual Studio at least), unsigned longs are always 4 bytes,
on both 32-bit and x64 Windows, so we cannot use unsigned longs to deal
with pointers on 64-bit builds, as pointers are 8 bytes on 64-bit
Windows, which may well render the pointer (which we acquired from
libffi) invalid.

This will fix crashes in PyGObject which are manifested when launching
the cairo-demo example sript (intermittent) and when clicking on
"Interactive Dialog" button in the Dialog demo in the PyGObject GTK+
Code demos before entering anything in Entry 1 and Entry 2, when running
on x64 Visual Studio builds of the GTK+/PyGObject stack.

Also use size_t instead of unsigned long in gthash.c when we check that
memory & 0x3 is 0, to silence compiler warnings from enabling /Wp64,
which is used to detect portability problems on Visual Studio when
doing x86->x64 code builds.

https://bugzilla.gnome.org/show_bug.cgi?id=702788
2018-04-20 18:26:08 +02:00
Nirbheek Chauhan
c01bc6ef0e g-ir-scanner: Don't require SRCDIR and BUILDDIR env vars
When building with Meson, we cannot set environment variables while
running custom targets and our builddir layout is different from
Autotools anyway.

Now g-ir-scanner and friends can autodetect when they're being run
uninstalled by Meson and will find _giscanner.so and the giscanner
python files in the build directory. This is very similar to what
gdbus-codegen uses in glib/gio.

Same for girepository/gdump.c.
2018-03-10 19:05:19 +05:30
Nirbheek Chauhan
3a78885133 meson: Use glib_dep and configinc for girepository
glib_dep is what is actually needed to #include <glib.h>, not
gobject_dep. It works incidentally with system gobject/glib but not
when built via subprojects.
2018-03-10 19:05:19 +05:30
Patrick Griffis
4c8c1a53b8 Initial work on meson port 2018-03-10 19:05:19 +05:30
Emmanuele Bassi
caef88d094 docs: Mention GI_TYPELIB_PATH
The rules for searching typelib files should be properly documented,
as well as the way to modify the search paths.

https://bugzilla.gnome.org/show_bug.cgi?id=699328
2018-01-30 14:01:06 +00:00
Emmanuele Bassi
6b123c6cff Skip unused generic marshaller
You typically want to use the version in GObject, not this one.
2018-01-30 13:43:28 +00:00
Emmanuele Bassi
8d25dd4159 Remove unnecessary transfer annotation
Integers do not need a transfer annotation.
2018-01-30 13:41:15 +00:00
Leslie Giles
5ecbc70a05 Increase MAX_NAME_LEN
There is no real limit on the length of an identifier, and some
toolchains easily hit the current limit of 200 characters.

According to this answer on StackOverflow, 2048 seems to be the limit on
MSVC and ICC:

https://stackoverflow.com/questions/6007568/what-is-max-length-for-an-c-c-identifier-on-common-build-systems

So let's use that.

https://bugzilla.gnome.org/show_bug.cgi?id=764791
2017-12-19 18:51:39 +00:00
Patrick Griffis
ac76bbb53b girepository: Don't skip g_irepository_get_option_group()
This function works fine via introspection

https://bugzilla.gnome.org/show_bug.cgi?id=786665
2017-10-22 07:16:05 -04:00
Emmanuele Bassi
a6b5da2a4f Use locale-independent functions to parse numbers
https://bugzilla.gnome.org/show_bug.cgi?id=788087
2017-09-24 14:32:26 +01:00
Nicola Fontana
52727afbd2 docs: manifest typelib format portability
Apart the endianness of its scalar values the typelib binary format
must be considered arch-independent: make it clear in the docs.

https://bugzilla.gnome.org/show_bug.cgi?id=764116
2017-09-10 10:21:24 +02:00
Rico Tzschichholz
38068ab362 girepository: Annotate array and array-length parameter 2016-12-10 21:44:00 +01:00
Rico Tzschichholz
f3154d62a2 girepository: Annotate iterator arguments as (inout) 2016-09-15 10:27:27 +02:00
Tobias Mueller
79225a62c0 gthash: free cmph objects
If not done, it would leak the memory as address sanitizer reports:

==1294==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 40 byte(s) in 1 object(s) allocated from:
    #0 0x7fa7a94b7602 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x98602)
    #1 0x44c7a7 in __config_new girepository/cmph/cmph_structs.c:11
    #2 0x44aaa7 in cmph_config_new girepository/cmph/cmph.c:291
    #3 0x446fb5 in _gi_typelib_hash_builder_prepare girepository/gthash.c:114
    #4 0x406cf7 in add_directory_index_section girepository/girmodule.c:270
    #5 0x409ee6 in _g_ir_module_build_typelib girepository/girmodule.c:546
    #6 0x404ada in main tools/compiler.c:217
    #7 0x7fa7a70d482f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)

==4091==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 40 byte(s) in 1 object(s) allocated from:
    #0 0x7fc20c854602 in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x98602)
    #1 0x44a3f3 in cmph_io_vector_new girepository/cmph/cmph.c:228
    #2 0x44a965 in cmph_io_vector_adapter girepository/cmph/cmph.c:276
    #3 0x446f9f in _gi_typelib_hash_builder_prepare girepository/gthash.c:113
    #4 0x406cf7 in add_directory_index_section girepository/girmodule.c:270
    #5 0x409ee6 in _g_ir_module_build_typelib girepository/girmodule.c:546
    #6 0x404ada in main tools/compiler.c:217
    #7 0x7fc20a47182f in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2082f)
2016-09-12 16:54:53 +02:00
Emmanuele Bassi
597d32df9c docs: Clarify return value documentation
Be a bit more specific than just saying "the transfer".
2016-05-18 23:12:13 +01:00
Emmanuele Bassi
f6ea169530 docs: Fix return value description
The return value of g_callable_info_get_caller_owns() is not a boolean
any more.
2016-05-18 23:10:46 +01:00
Paolo Borelli
50c7ae999c gdump: do not leak gfile objects 2016-05-08 15:35:13 -04:00
Carlos Garnacho
8a08674b33 girepository: Merge overrides with the regular search path
This reverts commit e81c4681cc88a00fcd841c5a68d860d3714b55d7

The GI_TYPELIB_PATH envvar will still allow overriding the default
typelib dir (based on gobject-introspection libdir), but applications
will have the last say about typelib lookup directories. The resulting
lookup order is now:

- Paths added through g_irepository_prepend_search_path()
- Paths in GI_TYPELIB_PATH
- The default gobject introspection lookup dir

This makes g_irespository_prepend_search_path() work as announced
despite environment variables. If any application was relying on
GI_TYPELIB_PATH overriding the paths of this function call (for e.g.
make check, or to be able to run code inside the project tree), it
is encouraged to set up a similar envvar for their application specific
lookup dir, or perform this override through other means.

https://bugzilla.gnome.org/show_bug.cgi?id=765735
2016-05-02 16:20:31 +02:00
Chun-wei Fan
b4df729c61 Fix code style
Make my last patch to this file conform to the code style in the rest
of this file.
2016-03-17 17:39:43 +08:00
Chun-wei Fan
4e43ed8112 cmph/bdz.c: Work Around MSVC 2012 x64 Compiler Bug
Due to an MSVC 2012 x64 compiler issue, the compiler generates bad code
for bdz.c, so the for loop in assign() continues running until the point
i falls below zero, causing an access violation when we try to do
curr_edge=queue[i]; (line 427 in bdz.c).  Address this issue by breaking
out of the loop at the end of it when i reaches 0 after doing the
necessary processing.

https://bugzilla.gnome.org/show_bug.cgi?id=733595
2016-03-16 22:48:18 +08:00