Fix FIXME: Emblems should be attached to icons via

Use GdkRectangle in PanGrid

Use GdkPoint in PAN_ITEM_TRIANGLE coord

Use util_clip_triangle() in pan_item_tri_new()

Remove unused parameters in pan_item_tri_new().

Use gdk_rectangle_intersect() in pan-view

Check for GTK4 migration regressions

- Additional test to ensure that GTK4 compatibility functions are
prefixed by gq_
- Fix found errors

Simplify util_clip_region()

* use gdk_rectangle_intersect()
* convert pointers to references

Bug fix: rotate plugin

Remove junk from the script file.

Include unit_tests option in test runs

Previous "test 1" commit should have been here.

test 1

Switches to using isolate-test.sh everywhere we launch Geeqie from tests/checks.

This allows us to run all of those tests in parallel, and without potentially interacting with processes that might be running outside of the build/test environment.

Updates image-test.sh to include a more robust teardown, and to make it more independent of the rest of the system.

In particular, avoids `pgrep` (which could detect geeqie processes unrelated to this test) in favor of stashing and checking the pid that was launched.

Also makes some more minor fixups to lua-test.sh

Creates a helper utility to run tests isolated from the user's actual environment.

This avoids interference from/with other processes that might be running on the
same host machine (such as the user's own Geeqie processes, or other processes
that were started in parallel by Meson), and also ensures that we can clean up
the test environment without making any permanent changes (especially important
in case of bugs).

Adds support to use the system gtest/gmock as a default, if available, and the subproject versions otherwise.

This follows the pattern from:
https://github.com/mesonbuild/meson/blob/master/test%20cases/frameworks/2%20gtest/meson.build

Partitions the Geeqie tests into three suites, and adds testing-related documentation.

Runs unit tests from main.cc when requested, and integrates that into the build system test plumbing.

To run just the unit tests is now just two steps:
```
$ meson setup -D unit_tests=enabled build
$ meson test -C build -v 'Unit tests'
```

Cleans up and slightly refactors command line interrogation functions in main.cc.

This is in preparation for conditionally executing unit tests when that argument is specified.

Creates some initial unit tests, and adds the build system infrastructure to compile and link them when appropriate.

Plumbs `ENABLE_UNIT_TESTS` through the build system, and conditionally compiles and links googletest into geeqie when unit tests are enabled.

This is following the pattern from:
https://stackoverflow.com/a/57478960

Incorporates the Meson gtest wrap file for googletest, in order to allow Meson to auto-download and -compile googletest when needed.

This is based on the strategy recommended here:
https://stackoverflow.com/a/57478960

The gtest.wrap file originates from here (using version 1.10.0):
https://wrapdb.mesonbuild.com/gtest

Exclude subprojects from markdown lint

This is necessary for a future commit. At the moment it has no effect.

Send stacktrace data also to a /tmp file

Fix meson deprecation

doc/meson.build:35: WARNING: Project targets '>=1.0.0' but uses feature
deprecated since '0.60.0': install_subdir with empty directory. It
worked by accident and is buggy. Use install_emptydir instead.

Check for shell scripts that are not POSIX

Exclude subprojects from shellcheck

This may be necessary for a future commit. At the moment it has no
effect.

Remove ToolbarButtonData

Use simple gchar*.
Also simplify toolbar_apply.

Add MetadataCacheEntry for fd->cached_metadata

Use more g_list_find_custom()

Instead of manual iteration.

Update clang-tidy-check for build options

If the build directory does not exist, run meson with the
-Ddevel=enabled option.

If the build test-image directory does not exist, assume that all
options have not been enabled and print a warning.

Remove unused members in toolbar structs

Widgets are added to containers, no need to keep them in structs.
Remove UseableToolbarItems.
Remove redundant casts, merge callbacks.

clang-tidy: bugprone-suspicious-missing-comma

- Remove the exclude from .clang-tidy file

clang-tidy: readability-redundant-casting

- Fix warnings identified by this flag
- Remove the exclude from .clang-tidy file

Edited by --fix-errors

clang-tidy: bugprone-switch-missing-default-case

- Fix warnings identified by this flag
- Remove the exclude from .clang-tidy file

Eliminate some GitHub action warnings

Update version number of some GitHub actions

Some command line options are not GNU/POSIX compliant (4)

Update the plugins for the change from colon to equals sign.

Eliminate meson deprecation (2)

mkdir --parents is not legal in FreeBSD

Eliminate meson deprecation

The feature '@0@'.format(name) was used - probably derived from:
https://github.com/mesonbuild/meson/issues/5273

This resulted in the warning:
DEPRECATION: Project uses feature that was always broken, and is now
deprecated since '1.3.0'

Rework meson.build files so that this feature is no longer used.

Update .clang-tidy for Ubuntu 24.04

Additional excludes are required for .clang-tidy, presumably because of
a new clang-tidy version.

Simplify vflist_get_formatted()

Use GString for result.
Remove redundant with_stars parameter.

Remove ToolbarButtonData::stock_id

No need to keep it.

Remove last GTK_STOCK items

Replace GQ_ICON_COLLECTION with PIXBUF_INLINE_COLLECTION.
Also use PIXBUF_INLINE_COLLECTION macro instead of "gq-icon-collection".
Replace GQ_ICON_DND with GQ_ICON_DIRECTORY for "Dropped list includes folders." dialog.

Disables clang-tidy check for multiple declarations on the same line for rect dimension variables rx, ry, rw, rh.

Echoing https://www.reddit.com/r/learnprogramming/comments/9zgvlk/comment/ea942uh/
"My recommendation is that you only declare multiple per line if they represent
more or less the same thing, and are used together in such a way that they
logically belong to each other."

Clean up hard-coded hidpi scale factor in debug codepath

Adds comments to rt_source_tile_render

Plus, some further tidying.

Reverts a diagnostic change to rt_hidpi_aware_draw

Significant cleanup and simplification of rt_source_tile_render

Baby steps! Rendering seems to be working, but there are still some bugs.

Include check for untranslated text

- Meson check for untranslated text strings
- Fix some untranslated text strings

Silence GtkAction deprecation warnings

Add aliases for deprecated functions.
Also move gq_gtk_container_add() out of ifdefs.

Deduplicate mark_to_selection and selection_to_mark

Some command line options are not GNU/POSIX compliant (3)

Update the plugins for the change from colon to equals sign.

Adds API documentation to functions in pixbuf-util.h

Also renames some variables to clarify operation.

Also notes but does not perform some function renames that would dramatically clarify what those functions do.

This change should be a no-op.

Prefer internal ViewFile submodule functions

Avoid redundant type checks.

Option to hide window decorations

The option is in Edit/Preferences/Windows

When selected and if supported by the window manager, window title bar,
resize controls, etc will not be displayed.

Deduplicate ViewFile pop menu functions

Deduplicate ViewFile dnd

Tidy AppImage generation files

Auto-completion - split standard and remote options

- Only complete remote options after --remote has been given
- Delete --remote-help option - it has no value

Simplify ViewFile callbacks

Remove unused declarations.
Add vf*_pop_menu_add_items().
Split vf_selection_foreach().
Move ViewFileInfo* structs to submodules.

Bug fix: AppImage notification not working (2)

g_file_query_info does not get the date from GitHub. Use curl instead.

Move common click_fd member to ViewFile

Deduplicate "Show star rating" menu item creation

Add vf_selection_foreach()

Hide implementation and deduplicate code.
Also simplify layout_menu_write_rotate() a bit.

Some command line options are not GNU/POSIX compliant (2)

Update the plugins for the change from colon to equals sign.

Bug fix: AppImage notification not working

AppImage new issue notification did not work if run from an extracted
AppImage.

Update AppImage download script for command line completions

If the extract option is given, create a symbolic link from
$HOME/.local/share/bash-completion to the extracted executable.

If this is not done, the command line auto-completion function will not
work for AppImages,

Command line auto-completion - further enhancements

- Include auto-completion for remaining options.
- Fix some text errors

Option --cache-maintenance is not GNU compliant

https://www.gnu.org/software/libc/manual/html_node/Argument-Syntax.html

The option requires an equals sign as a separator.

Simplify pan_item_image_find_size()

Deduplicate PanCacheData destroying

Some command line options are not GNU/POSIX compliant

The definitions are:
https://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap12.html
https://www.gnu.org/software/libc/manual/html_node/Argument-Syntax.html

Colons in an option have been replaced by an equals sign.
Multi-character short options have been deleted.

The option --cache-render-shared-recurse remains illegal because it
exceeds the three--word limit.

Use references in pan_layout_compute() parameters

Simplify related functions.

Move some types and constants from pan-types to specific modules

Minor change to AppImage download script

Display an animated "spinner" character while extracting the AppImage.

Bug fix: Collections and unmounted drives (2)

Did not compile on FreeBSD

This is a blind fix.

Merge SCROLL_RESET_* macros with PixbufRendererScrollResetType

Rename to ScrollReset.

Bug fix: Collections and unmounted drives

The bug:
If a collection file is on the local filesystem
The collection file contains references to files on a mounted drive
The drive is unmounted
The collection file is opened

Geeqie sees that the files do not exist and deletes them from the
collection file.

The fix:
If Geeqie does not find the file-

If the file path prefix contains /home, /tmp or /usr it is assumed that
the file was on the local drive and has been deleted by the user. It is
ignored.

If the file path prefix contains a mount path, it is assumed that the
file was on a currently mounted drive and has been deleted.

Otherwise it is assumed that the file is on a removable drive that is
not currently mounted. The collection will not be opened.

If this is not the case the user may need to use a text editor to remove
the offending line from the collection file.

Created problem:
When a collection window contents have not been changed, but its
geometry has, the geometry changes will not be saved. The user must make
a minor positional change to any thumbnail to trigger a save.

Drop obsolete GTK_OBJECT_FLOATING

Move some defines from main-defines to modules

Convert to constants.

Move keyboard_scroll_calc() to layout-util

Convert parameters from pointers to references.
Move Shift processing inside keyboard_scroll_calc().

Simplify lists handling in history-list

Simplify max value search in history-list

Remove unused key_press_cb() declaration

Replace custom PI with G_PI

Simplify pixels sorting in image_sim_channel_equal()

And move it to anonymous namespace.

Additional test for temporary comments

- The only sequence checked for is "//~"
- Delete existing temporary comments

Command line auto-completion enhancements

- Limit completions to a list of file extensions known to Geeqie
- Limit completions to a specific extension where possible e.g. gqv
- Change option +t to -T and +w to -w. The plus character affected the
output of _parse_help
- Additional warning to user for invalid parameters - unknown option or
valid remote option used without a --remote command
- New remote.cc function for above feature
- Update man page and documentation

Replace mround with STL lround

Include xmllint checks

Command line auto-completion - AppImages

- Auto-completion does not work with AppImages. Document a work-around.
- Change the format of the completions file to be the same as similar
files.

Simplify ImageSimilarityData comparison

* Run alternate_image_sim_compare_fast() once
* Deduplicate ImageSimilarityData comparison

Implement basic command line auto-completion

_parse_help is used to provide auto-completion for long options.
No other processing is done.

Fix #259: Zoom by drawing rectangle

https://github.com/BestImageViewer/geeqie/issues/259

Menu item View/Zoom/Zoom To Rectangle

Add HtmlBrowser class and replace C array with std::array

Fix 1302: Cut image to clipboard

https://github.com/BestImageViewer/geeqie/issues/1302

Shortcut key Control-X and menu item Cut To Clipboard function as in
other programs.

This feature is only available in the main window.

Fix #1089: Sync xmp:Rating across all sidecar images (2)

https://github.com/BestImageViewer/geeqie/issues/1089

Fix the documentation.

Fix #1303: Automatically hide image overlay in fullscreen mode

https://github.com/BestImageViewer/geeqie/issues/1303

Additional option Edit/Preferences/Behavior/Hide OSD In Fullscreen

Additional meson test - look for stray debug statements

Check if DEBUG_0, DEBUG_BT or DEBUG_FD statements have been erroneously
left in the code.

Fix #1089: Sync xmp:Rating across all sidecar images

https://github.com/BestImageViewer/geeqie/issues/1089

Include Xmp.xmp.Rating in list of tags written to all sidecars.

Fix #1240: Regression: Option to open new full-function window directly is missing

https://github.com/BestImageViewer/geeqie/issues/1240

Ensure that the sort sidebar is displayed if set in a saved window.