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.

Documentation update - plugin filter

Note that a restart is required for a plugin of this type to be
registered.

Fix #1241:  'Go to directory view' uses hard-coded default values

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

If the image is already displayed, bring that window to the top.

If not, a new layout window is opened using the values set by the user
in Edit/Preferences/Windows/Use Current Layout For Default.

If this option has not been set, use the fixed default values.

Cleanup rt_overlay_get_position() and use in rt_draw_cb()

New plugin - set file mtime to Exif.Image.DateTime

Deduplicate rt_border_draw

Use g_list_free_full to free draw_queue and draw_queue_2pass

Remove unused RendererTiles::tile_cols

Convert GET_{LEFT,RIGHT}_PIXBUF_OFFSET macro to function

Also cleanup a bit.

Release v2.4 files

Update documentation and translations

Fix #1289: tar.xz contains symlinks that are wrong for plugins (3)

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

Fix #1289: tar.xz contains symlinks that are wrong for plugins (2)

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

Fix #1289: tar.xz contains symlinks that are wrong for plugins

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

Also fix scripts so that tests can be run from the source tar.

Fix #1296: AppImage doesn't distribute libffmpegthumbnailer

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

Silence GTK deprecation warning in unused function

Also use G_GNUC_BEGIN/END_IGNORE_DEPRECATIONS macros instead of pragmas.

Replace deprecated gdk_screen_make_display_name

Ensure plugins check for external command

Ensure all plugins check that required external commands such as
exiftool are installed.

Convert group_keys to std::array

Enable and fix clang-tidy readability-simplify-boolean-expr

Replace some macros with inline functions.

Replace deprecated gdk_window_get_pointer

Simplify comparison a bit.

Aditional remote command - window list

Silence GTK deprecation warnings in champlain-gtk

Also remove unused pragma

Fix FreeBSD build

c351a3a4fc698924a49dca09991aa0c5192ec345 follow-up.

Include check compile on FreeBSD as a GitHub action

Include appstreamcli in ancillary files check

appstreamcli is used to lint org.geeqie.Geeqie.appdata.xml.in

Eliminate some warnings from the appdata file

Adapt appdata to stricter requirements

Flathub nowadays has stricter requirements for the appdata.
This commit adapts the appdata file accordingly

Fix some GTK deprecation warnings

Blind fix macOS build

Use std::swap instead of temporary values

Use p_step to avoid has_alpha comparison in loop

Include icon name column in Preferences/Keyboard table

Simplify util_clip_triangle()

Add PanColor to simplify pan functions

Move some defines to cc files.

Replace single value enums with constexpr <type>

Commit a4ce95bc clang-tidy: modernize-macro-to-enum, resulted in some
enums containing a single value.

These are now replaced by a constexpr <type> statement.

The script used to identify these single value enum statements is now
included as a meson test.

Deduplicate FILE_COLUMN_POINTER

Deduplicate cell_renderer_height_override()

Updated Swedish translation

Release v2.3 files

Update translations

Bug fix: No toolbars on first entry

When Geeqie is started for the first time and geeqierc.xml does not
exist, the toolbars were not set to the default values.

Fix build with LTO

* Move structs and some functions to anonymous namespace
* Replace some defines and enums with constants
* Replace macro with inline function

Compile modules based on HAVE_* features

Remove Code Correctness checks on GitHub

muxee/clang-tidy-action runs only on pull requests.

It is necessary to run the checks also on pushes.

Run Code Correctness checks on GitHub

Remove set but not used variables

Add file class Archive to the Search page

Move open_archive() to separate module

Also make parameter const and move static functions to anonymous namespace.

Fix untranslated text

Fix warnings in exiv2

Also fix indentation.

For POTFILES, look for whereami and zonedetect in third-party folder

Update some untranslated text strings

In some cases, double quotes within a text string have been replaced by
single quotes to make it easier to search for untranslated strings.

Run Code Correctness checks on all source files

In tests, run Code Correctness checks on all source files rather than
just modified files.
Files in the third-party directory are not checked.

Fix #1248: Crash when hiding file list

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

Fix problems resulting from 6d72bf GTK4: Reduce the use of
gtk_widget_destroy()

Move third-party sources to separate sub-directory

Move any third-party sources that have been copied verbatim to a
separate sub-directory.
This allows project checks to avoid these files.
These files are unlikely to change and do not need to be tracked for
changes.

Bug fix: Pan view not displayed correctly

Bug resulting from 684059d  GTK4: GtkTable

Simplify usage of set_page_num and get_page_total

Convert ImageLoaderBackend to interface and backends to implementations

Add parameters to ImageLoaderBackend methods.
Remove unused members from backend implementations.
Rename ImageLoaderBackend loader_new to init.
Move callbacks aliases to ImageLoaderBackend and rename SizeCb to SizePreparedCb.

Include Code Correctness check in GitHub test runs (2)