Release 0.36.20

x11: Fix return type of XInternAtoms and XGetAtomNames bindings

posix: Fix return-value of mknod() and c-include for tcgetsid()

Introduced with 535f2734d19875ccc1f170841b72eba0c4adad1e

webkit2gtk-4.0: Fix WebContext.initialize_notification_permissions()

Bind it as WebContext.init_notification_permissions() to avoid conflict.

glib-2.0: add MappedFile.from_fd constructor

Add the missing 'g_mapped_file_new_from_fd' constructor, added
with glib 2.32.

Fixes https://gitlab.gnome.org/GNOME/vala/issues/824

xtst: Fix signature of XTest.fake_relative_motion_event()

See https://gitlab.gnome.org/GNOME/vala/merge_requests/67

glib-2.0: Bind strcmp0 as GLib.CompareFunc<string?>

Allows proper usage with experimental non-null

Fixes https://gitlab.gnome.org/GNOME/vala/issues/810

clutter-1.0: Bind Margin, PaintVolume, Point, Rect and Size as struct

Reverts 0cd8009728245300d32009fb029898b4a3ceed8a and its confusion
regards "caller-allocates" out-parameters

Fixes https://gitlab.gnome.org/GNOME/vala/issues/795

cogl-1.0: Fix out-params in Cogl.get_modelview_matrix/projection_matrix/bitmasks

Fixes https://gitlab.gnome.org/GNOME/vala/issues/794

codegen: Include required type-definition when casting from generic pointer

Fixes https://gitlab.gnome.org/GNOME/vala/issues/828

codegen: Use G_TYPE_CHECK_INSTANCE_CAST for comparisons with interfaces

Avoids "comparison of distinct pointer types lacks a cast" warning for
such cases.

codegen: Real structs are allowed by simple generics and passed as reference

Add the required pointer-indirections to fix invalid memory accesses.

Fixes https://gitlab.gnome.org/GNOME/vala/issues/819

codegen: Assign GValue result of function calls to temp-var on copy_value

Regression of a94a28141e1f222d1da7e72bea4cd2d1e5242f41

Fixes https://gitlab.gnome.org/GNOME/vala/issues/819

codegen: Use array_length of collection variable instead of expression

codegen: When freeing local variables don't stop at "switch" on "continue"

So distinguish between BreakStatement and ContinueStatement to make the
correct decision in append_local_free().

Fixes https://gitlab.gnome.org/GNOME/vala/issues/799

codegen: Always use G_TYPE_CHECK_INSTANCE_TYPE for external symbols

and get_ccode_type_check_function() for SourceFileType.SOURCE symbols.

codegen: Make sure to call a corresponding pop_function()

codegen: Make sure to call a corresponding pop_line()

codegen: Make sure to call a corresponding pop_context()

compiler: Add OptionFlags.NO_ARG to deprecated "--thread" option

It was defined with OptionArg.NONE and never took an extra argument.

Regression of d8e810680f370fac7e9a5e80f8d94316707625c2

tests: Add GMutex and GRecMutex test to increase coverage

tests: Null-terminate arrays for compatibility test of uint8 and uchar

Fixes https://gitlab.gnome.org/GNOME/vala/issues/809

tests: Check uchar[] and uint8[] compatibility

tests: Properties with NoAccessorMethod return heap-allocated structs

girparser: Handle "function-macro" by skipping them

See https://gitlab.gnome.org/GNOME/gobject-introspection/merge_requests/159

girparser: Skip 'attribute' elements

girparser: Add support for string "ctype" metadata

https://gitlab.gnome.org/GNOME/vala/issues/793

girwriter: Write implicit fields for arrays and delegates

girwriter: Write implicit parameters and properties for generics

girwriter: Only replace existing GIR files if they changed

Same how it is done in CodeWriter.write_file()

girwriter: Populate gir_namespace/version attributes of external namespaces

girwriter: Don't write c:include tags for symbols from external packages

girwriter: Properly write attributes for non-GObject classes

girwriter: Don't write "property" attribute for non-gobject properties

girwriter: Properly resolve GLib.TypeInterface instead of hardcoding it

This makes sure to pick up the dependency on GObject-2.0.gir

girwriter: Mention that this file is generated and not meant to be modified

Same as already done for generated vapi files.

tests: More "unassigned variable" tests to increase coverage

tests: Extend "foreach" tests to increase coverage

tests: Extend "assigned local variable" control flow tests

tests: Extend "assigned local variable" control flow tests

Cover condition statements of WhileStatement and ConditionalExpression

parser: Set correct end source-reference of Block for embedded statement

parser: Improve source-reference of declaration block for ForStatement

parser: Improve source-location for local Constant declarations

parser: Don't include trailing semi-colon to ExpressionStatement

parser: Improve source-location for LocalVariable/DeclarationStatement

parser: Set correct end source-location for Block

This location was off by one token.

Release 0.36.19

codegen: Always assign original variable when consuming instance to destroy

Fixes https://gitlab.gnome.org/GNOME/vala/issues/781

vala: Use global CodeContext to retrieve root_symbol

pango: Fix StringBuilder parameter of scan_word()

gstreamer-1.0: Fix TypeFind and TypeFindFactory method references

gobject-introspection-1.0: Fix TypeInfo and TypeTag method references

Fixes https://gitlab.gnome.org/GNOME/vala/issues/778

girparser: Create correct output in get_default_lower_case_suffix()

It must match the output of CCodeAttribute.get_default_lower_case_suffix()

Add metadata support to set lower_case_csuffix

See https://gitlab.gnome.org/GNOME/vala/issues/778

tests: Add "struct (un)boxing" variant test

codegen: Move implicit GValue cast for comparison to BinaryExpression

Handle "==" and "!=" only as it was done before.

This generates correct c-code for boxed simple-types and will perform
value-based comparisons.

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

codegen: Resolve generics in sizeof-expression of parameter initializer

glib-2.0: Don't annotate dedicated GLib.assert_*() functions with [Assert]

Fixes https://gitlab.gnome.org/GNOME/vala/issues/769

vala: Variadic parameters are not supported for async methods

parser: Implicit creation method of abstract classes must be protected

vala: Allow assignment of 0U to enum types

Don't restrict unsigned 0 literal to flags-enums though.

ccodegen: Use unsigned default value for flags-enum

gsignal: Use guint to represent flags-enum in custom GMarshalFunc

tests: Extend "detailed signal" test to increase coverage

Mark defined constructors of abstract classes as protected

Release 0.36.18

girwriter: Always use get_gir_name() for TypeSymbols

poppler-glib: Update to 0.74

girwriter: Support transfer-ownership="container" for arrays

gobject-2.0: Add SignalHandler.disconnect_by_data()

codegen: Respect array_length_cname attribute for global fields

tests: Add regression test for modulo operation on double and float

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

girparser: Correctly set array_length_type for delegates returning an array

Fixes https://gitlab.gnome.org/GNOME/vala/issues/754

clutter-1.0: Drop deprecated from Actor.pick()

The signal "pick" was deprecated with 1.12, but not the virtual method.

Fixes https://gitlab.gnome.org/GNOME/vala/issues/625

vapi: Drop gdk-x11-3.0 from GIDL_BINDINGS

gtk+-unix-print-*.0: Fix PrintJob.get_page_ranges(), Printer.get_hard_margins()

gdk-3.0: Fix ownership of FrameClock.get_current_timings/get_timings()

g-i: Fix return value on error in start_discriminator()

vala: Parser doesn't ensure "protected" to be applied on TypeSymbol members

Probably ObjectTypeSymbol should be enforced.

vala: Check inferred variable_type of LocalVariables

codegen: Replace some unfortunate asserts with internal error reports

codegen: Properly handle and catch inner-error of finally-block

If all inner-errors are caught there is no jump-out of the current finally-
block therefore the control-flow analyzer is happy. To make this work the
surrounding inner-error must not be used or changed here.

Fixes https://gitlab.gnome.org/GNOME/vala/issues/742

codegen: Don't apply address-of operator on literals when casting to array

Regression of 5f316333d6a803e4240b5cccc95a6abd2294137b

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

girparser: Async methods don't allow out-parameters before in-parameters

In vala out-parameters are always handled in the *_finish implementation
and therefore are asynchronous. Report a warning for GIR sources where
convert them to pointer-types to create usuable signatures for bindings.

See https://gitlab.gnome.org/GNOME/vala/issues/636

clutter-1.0: Fix BindingPool.override_action()

compiler: Report deprecated command-line option "--thread"

libsoup-2.4: Fix Message.add_*_handler() by unhiding user_data parameter

See https://gitlab.gnome.org/GNOME/vala/issues/731

codegen: Emit struct declaration typedef before resolving its fields

This a regression when compiling a circular dependency of delegates with
structs.

Regression of 7adb3a45bb8d663c0cfca55af26b7e6f7292b14a

See https://gitlab.gnome.org/GNOME/vala/issues/318

codegen: Use a parameter-map for creation of delegate declaration

This is required to take instance_pos into account and will allow the error
parameter to be put before the target (aka instance) in delegates.

See https://gitlab.gnome.org/GNOME/vala/issues/728

gobject-2.0: Minor syncing from GIR

gdbus: Cast instance and result of g_async_initable_new_finish() call

This will silence those incompatible-pointer-types warnings

sqlite3: Correct return C type of Statement.column_text & Value.to_text

gdbus: Properly set annotations field of GDBus*Info struct to NULL

Found by -Werror=missing-field-initializers

tests: Extend "DBus errors" tests to increase coverage

docs: Add a basic CONTRIBUTING.md file that links to the relevant Wiki page

codegen: Don't check boolean values for (in)equality in GTask API

This resolves any value but 0 to true as it is suppose to be.

  /* invalid */
  if (condition == TRUE)
    do_foo ();

  /* valid */
  if (another_condition)
    do_bar ();

See https://gitlab.gnome.org/GNOME/glib/issues/1636

gio-2.0: Fix File.replace_contents_bytes_async()

Closes https://gitlab.gnome.org/GNOME/vala/merge_requests/37

gobject-2.0: Use correct array-length-type for returned arrays

glib-2.0: Use correct array-length-type for returned arrays

As usual length is returned by an implicit out-parameter where the
provided target variable must have a matching data type.

Discovered by -fstack-protector-all

See https://gitlab.gnome.org/GNOME/gitg/issues/171

vala: Report invalid instance member access to property

See https://gitlab.gnome.org/GNOME/vala/issues/605

codegen: Use temp-vars for ellipsis out-arguments to fix memory management

Fixes https://gitlab.gnome.org/GNOME/vala/issues/722