Use correct stat struct for ftw()

check_dir_mtime() is called by ftw() and is given
the real stat struct, not its glib version (which may
or may not be the same as "struct stat").

This is irrelevant for MSVC (it has no ftw()) and
works correctly for MinGW-w64 (which declares stat
structures correctly). If mingw.org complains, add
a special ifdef for it later.

See if _MSC_VER is defined before checking its value

Fix missing default switch case in DllMain

Fix missing default switch cases in gtkimcontextsimple

Fix some warnings in W32 part of gtkmain

* Add missing default cases
* Cast string literals as non-const

Fix a typo in W32 event-handling code

It's quite old, but mostly harmless (both "message == WM_KEYUP"
and "message = WM_KEYUP" evaluate to not-FALSE, and message
value is not used after that line).

Fix a warning about uninitialized variable

Remove unused code in gdksurface-win32.c

Fix more signedness issues in W32 backend

A quick and dirty fix for a circular allocation issue

query_targets() tried to write to struct that
wasn't yet allocated.
This code is going to change soon, so this is a temporary
fix until then.

Fix a missing default case warning

It shouldn't be possible to get DPI_STATUS_PENDING here.
If it does happen, that would be a bug that should be fixed.

Remove unused functions in W32 backend

Fix signedness issues, remove unused code

Remove some more unused variables

Fix various type mismatch warnings

Fix gdk_win32_cairo_context_end_frame prototype

Remove unused variables (mostly in W32 code)

Fix wrong format strings in various places

Update Polish translation

Update POTFILES.in

picture: Don't use g_str_equal() with potential NULLs

docs: Add GtkEventControllerKey to index

dnd: Ref the GdkDrop during the DND operation

It might go away if the DND takes too long otherwise...

quartz: Remove gtkdnd-quartz.c

It shows up in my git diff calls and is completely broken.

x11: Remove useless check

The check survived from GTK2 when that function could still return
GdkPixmap and GdkFont objects and was accompanied by this comment:

  /* We may receive events such as NoExpose/GraphicsExpose
   * and ShmCompletion for pixmaps
   */

clipboard: Remove return statements from void function

gtk-demo: Use GtkImages to select the puzzle

iconhelper: Always size contents to icon size

No more special casing for paintables. If you want the special case for
paintables, you should use GtkPicture.

image: Remove gtk_image_set_keep_aspect_ratio()

and gtk_image_set_can_shrink().

Images are meant to always be icon-sized, they can never shrink below
that.

And images are icons, so they are meant to be square. If they are
not, we pretned that's by accident and keep aspect ratio.

gtk-demo: Make sliding puzzle demo use GtkPicture

gtk-demo: Port images demo to use a GtkPicture in places

video: Use a Picture instead of an Image

picture: Introduce

This commit introduces GtkPicture, which is supposed to complement
GtkImage.

GtkImage will be adapted to always display an icon, while
GtkPicture displays regular imagery.

puzzle: fix some compiler warnings

These slipped under the radar, sorry.

widget: Use priv pointer less

Instead declare a priv local. We should do this even if we don't remove
the priv pointer from GtkWidget entirely, just to stay consistent with
new code we introduce.

Update Galician translation

puzzle: Add mouse support

Make it so that clicking on a puzzle piece moves enough
pieces to move the empty space there, if it is possible.

puzzle: Use plain labels

The previous code looked cool in the UI, but was too much
of a hack in the code.

Add tweaks

Allow setting the puzzle size, and allow chosing between
the rose and the radioactive animation. Change the default
size to 3. 6 is just too hard.

puzzle: Add a reshuffle button

Keep aspect ratio of the image

Since we don't have a good way to control the window size tightly,
make the content keep aspect ratio.

demo: Add the sliding puzzle demo

Merge branch 'master' into 'master'

Fix reference to user styles in gtkstyleprovider.h

See merge request GNOME/gtk!184

font-button: Suggest non deprecated function in docs

gtk_font_button_get_font_name() has been deprecated in 3.22 and
its public API is removed from GTK4.

Fix reference to user styles in gtkstyleprovider.h

Merge branch 'wip/move-to-rect-public' into 'master'

gdk: Make gdk_surface_move_to_rect public

See merge request GNOME/gtk!179

SizeRequest: Round px values up for min CSS sizes

Otherwise, requesting a min size in em where the equivalent in px had a
fractional part would lead to the widget getting allocated 1 too few px.
You could see this in the CSS property vs. allocation in the Inspector.

Note that margin/border/padding are left alone: the rationale is that we
do as browsers do, and Benjamin said we already do that for those,
whereas his tests on min-(width|height) showed otherwise. My subsequent
analysis indicated it to be far less clear-cut than that, but he remains
unconvinced that we should ceil() all the things! So just do these ones.

https://gitlab.gnome.org/GNOME/gtk/issues/1088

gdk: Make gdk_surface_move_to_rect public

This is the API used by GtkMenu to properly position menus on the screen
without requiring GTK to query the menu window's position or the work
area of where the window is positioned. It makes it possible to position
popup windows properly when using Wayland.

Make this API available to external users so custom popup windows can be
positioned properly as well.

Related: https://gitlab.gnome.org/GNOME/gtk/issues/997

popover: Add missing close paren in docs

colorbutton: Remove priv pointer

aspectframe: Remove unused member

bin: Remove public GtkBinPrivate typedef

statusbar: Remove garbage from header file

Nobody will ever read those comments; documentation belongs into the
soruce file so gtk-doc actually picks it up.

statusbar: Remove priv pointer

widget: Add docs for {get,set}_focus_child

With get_ being still private.

gl renderer: use w axis vector from graphene

Instead of initializing our own one every time.

Merge branch 'patch-1' into 'master'

GtkWindow: Handle non-square icons with height > width correctly

Closes #657

See merge request GNOME/gtk!170

GtkWindow: Handle non-square icons with height > width correctly

widget: Fix example code in doc

Merge branch 'gesture-docs' into 'master'

Gesture stuff

See merge request GNOME/gtk!169

x11: Don't set NET_WM_PID when sandboxed

It is not useful, and some window managers misinterpret it and
add some "runs as root" indication to the window decoration.

See https://github.com/mate-desktop/marco/issues/301

gdk: Add a private api to find sandboxes

This will be used in more places in the future.

events: Mark static function as such

gdk: Get rid of gdk_event_free()

Events are objects, so use g_object_unref().

gesturemultipress: Don’t fire ::released after ::cancel

Causing a grab in the handler for ::pressed by, e.g., popping up a
context menu will cause the gesture to be canceled and, subsequently,
::end and ::released to be fired, all while the button is still
physically pressed. That results in no event being available to the
::released handler and garbage coordinates, given that
gtk_gesture_get_point() returns FALSE.

Emitting ::released can be avoided by checking the return value
gtk_gesture_get_point().

gesture: Fix code snippet

GTK_EVENT_SEQUENCE_ACCEPTED seems to never have been a valid enumeration
value. GTK_EVENT_SEQUENCE_CLAIMED is the closest in meaning.

gesture: Fix get_last_event() docs

Querying the event sequence of a gesture will always yield NULL for
non-touch events, but passing NULL in to calls to
gtk_gesture_get_last_event() is a perfectly valid use case.

gdkseatdefault: Don't hide GdkSurface on grab failure

Application is not expecting that.

Bug found due gdk_seat_grab() failure on Lock Screen. When user
Unlock the screen, the application is visible but does not receive
enter-event any more on X11/GNOME.

Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1485968
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1571422

Signed-off-by: Victor Toso <victortoso@redhat.com>
Signed-off-by: Carlos Garnacho <carlosg@gnome.org>

gtkpopover: Fix thinko in key press event type check

That code branch is meant to check for key events, seems obvious we want
GDK_KEY_PRESS, not GDK_BUTTON_PRESS (which also broke the branch right
below).

Makes us all able to dismiss popovers again.

Revert "popover: Fix setting priv->button_pressed in ::event handler"

This reverts commit d638ff3afa7f3ccfebf2d5de23616109742d622b.

Doesn't look like the right fix. Another one coming that fixes dismissing
through both pointer and keyboard.

Remove an unused struct

Positive side-effect: visual c no longer has an empty struct
to complain about.

popover: Fix setting priv->button_pressed in ::event handler

There were two cases in that if/else cascade handling the
event_type == GDK_BUTTON_PRESS case, so priv->button_pressed never got
set.

gdkscreen-x11: Remove redundant typedef

Causes warnings with clang.

gl renderer: Remove an outdated comment

Merge branch 'master' into 'master'

icontheme: Keep dir_mtimes in order

Closes #1115

See merge request GNOME/gtk!161

filechooserwidget: Don't chain up in ::event

GtkWidgetClass.event is NULL.

gl renderer: Use offsets for more node types

Update Polish translation

a11y: Stop using ::event

We can just as well use notify::has-focus for the purpose of
focus tracking, and we can at the same time avoid emitting the
deprecated AtkObject::focus-event signal.

ci/msys2: force disable vulkan

A recent dependency change in MSYS2 made it pull in vulkan, which made
meson think it's available but it somehow links against the system vulkan dll
instead.

Disable vulkan for now.

SpinButton: Fix an obviously wrong arg description

:climb-rate is not about what you get when you single-click on a button,
as this implied: it's what happens if you hold down a button or a key.
Fix the description of @climb_rate to new(), and while here, mention the
key in the blurb of :climb-rate itself.

icontheme: Keep dir_mtimes in order

Don't reverse the order each time we insert a theme. Reverse it only
once, after all themes have been loaded.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/1115

Merge branch 'kill-key-events' into 'master'

Kill key events

See merge request GNOME/gtk!160

Merge branch 'issue792' into 'master'

wayland: check native surface for crossing events

Closes #792

See merge request GNOME/gtk!148

Drop the ::key-press/release-event signals

These are the last event-type specific signals,
we are just relying on the generic ::event signal now.

treeview: Stop using ::key-press-event

Use a generic ::event handler instead.

spin button: Stop using ::key-release-event

Just use the generic ::event signal.

font chooser dialog: Stop using ::key-press-event

Use the generic ::event instead.

button: Stop using ::key-release-event

Use the generic ::event signal instead.

file chooser widget: Stop using ::key-press-event

Use a generic ::event handler instead.

file chooser entry: Stop using key-press-event

Just use the generic ::event signal for now.

entry: Stop using ::key-press-event

Use a generic ::event handler instead.

StyleContext: Don't advise using invalidate()

It doesn't exist anymore! Also, set_display() is not needed/used in the
demo cited, and we were still referring to gtk3-demo, not gtk4-demo.

Range: Up should only mean ++ if we are a GtkScale

The last round of patches to get the desired direction of value move in
response to scrolls/keypresses on scales had the inadvertent side effect
of giving the opposite direction on scrollbars. Seeing as gtkrange.c is
already a collection of hacks, add another so that fix only holds if the
instance is a GtkScale, since that is what those patches were aimed at.

Close https://gitlab.gnome.org/GNOME/gtk/issues/1065

emoji completion: Stop using key-press-event

The generic ::event signal will do.

Remove key-press-event from more docs

This signal will be going away, so stop mentioning it so much.

gtk-demo: Avoid key-press-event

popover: Drop key_press_event handler

We can do the same in the generic ::event handler.

emoji: Hide recent section when empty

This is part of https://gitlab.gnome.org/GNOME/gtk/merge_requests/143
by Julian Sperber.

widget: Properly remove event controllers in finalize

We were mutating the list while iterating over it. This was not a
problem before since remove_controller just set the controller pointer
to NULL instead of actually removing it from the list of controllers.