Preparing for release v2.3

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)

Fix deprecation warning for poppler >= 0.82

Let image loader backend decide how to process image buffer

Remove ImageLoaderBackendFuncLoad.
Also move static functions to anonymous namespace.

Include Code Correctness check in GitHub test runs

Minor documentation update

Update cache-maintenance description.

Bug fix: clang-tidy checks missing sub-dirs

Sub-dirs ./src/pan-view and ./src/view-file were not checked

Fix HAVE_* config values checks

Deduplicate cr3 image loader

Make it wrapper of jpeg image loader.
Add HAVE_RAW define check to make consistent with usage in image-load.
Move static functions to anonymous namespace.

Remove plugin refresh-thumbnail from POTFILES

Fix representing percent in c-format string

Fix include-what-you-use warnings

Use #if to check 0/1 config value

Bump Exiv2 minimum version requirement to 0.18

Remove obsolete code.

Remove redundant GdkRGBA memcpy

Fix missing translation

Include anonymous namespaces in Doxygen output

Sort headers using clang-tidy

Add include order rule to CODING.md.

Cleanup main.h header

* Include compat.h, config.h, debug.h, intl.h, options.h, typedefs.h where they are actually used.
* Remove standard library includes.
* Remove unused gdk-pixbuf includes.
* Make other headers self-contained.
* Move defines not related to main.cc from main.h to main-defines.h.

Include backward.h where it is actually used

Document additional file management options

Document the special plugin options to override the standard commands
for file copy, move, rename, delete and folder create.

clang-tidy: readability-isolate-declaration

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

Fix #1242: Renaming files results in a crash

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

Fix bug from GTK4: Reduce the use of gtk_widget_destroy() 6d72bf20cc

clang-tidy: modernize-macro-to-enum

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

Clang-tidy fix resulted in indentation dissimilar to that used so far.

This awk script was run on the .diff file:
/^+[A-Z]/  {
sub (/\+/,"+\t",$0)
print
next
}

$0 !~ /^+[A-Z]/  {
print
next
}

The following files were hand-edited:
thumb-standard.cc
ui-bookmark.cc

clang-tidy: bugprone-integer-division

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

clang=tidy: readability-uppercase-literal-suffix

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

clang-tidy: readability-suspicious-call-argument

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

clang-tidy: readability-misleading-indentation

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

Fix clang-tidy: readability-non-const-parameter (5)

Fix clang-tidy: readability-non-const-parameter (4)

Fix clang-tidy: readability-non-const-parameter (3)

Fix clang-tidy: readability-non-const-parameter (2)

Fix clang-tidy: readability-non-const-parameter

The change suggested for exiv2.cc fails on further checks.

clang-tidy: readability-non-const-parameter

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

clang-tidy: readability-inconsistent-declaration-parameter-name

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

clang-tidy: readability-else-after-return

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

Additional excludes for testing

Additional excludes for getting error-free test runs

Fix errors in testing scripts

clang-tidy: readability-duplicate-include

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

clang-tidy: modernize-use-auto

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

clang-tidy: modernize-use-using

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

clang-tidy: bugprone-macro-parentheses

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

clang-tidy: misc-redundant-expression

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

clang-tidy; -bugprone-suspicious-string-compare, misc-unused-parameters

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

clang-tidy: modernize-redundant-void-arg

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

Ensure default .geeqierc.xml is used in tests

Set XDG_* for local tests so that local user file are not affected.

clang-tidy: modernize-use-nullptr

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

Revise testing

- Move ancillary file checks from meson.build to test-ancillary-files.sh
- Scripts to run checks locally - test-all.sh, clang-tidy-check.sh
- revise .clang-tidy so that no errors are shown for current sources
- remove files erroneously in .git
- include .shellcheckrc

Fix #1236: Window Layouts with vertical split are shifted to the left if they are narrower than the application menu

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

Add additional scrolled box for both menu and toolbar.

Limit definition of XOPEN_SOURCE to Linux, where it's needed.

The definition breaks the build on NetBSD where it hides symbols
(to match what the standard says).

Fixes #1230.

Use GdkRectangle for LayoutOptions::log_window

Fix permissions for geocode-parameters plugin

Check and revise .desktop files

Include a .desktop file check using desktop-file-validate.
Revise .desktop files - most had an incorrect Categories entry.

Add keywords to desktop file

Release v2.2 files

Update .po files

Update template-desktop.sh

Take account that the menu is now in a file menu-classic.ui

Fix #1223: "Confirm move files to trash" not working

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

Check for move to trash missing.

Option to select menu style

User option to select a "hamburger" style menu icon or the classic text
style.

The option is in Edit/Preferences/General.

Fix #1227: webp images load as black even with every prerequisite being green during compile

https://github.com/BestImageViewer/geeqie/issues/1227
https://github.com/BestImageViewer/geeqie/issues/1076
https://github.com/BestImageViewer/geeqie/pull/1225

Revert to using image-load-webp.cc
This Geeqie webp decoder will take precedence over an installed
webp-pixbuf-loader.

This decoder does not support webp animations.

If the user wants to use an installed webp-pixbuf-loader (e.g. to get
animations), Geeqie must be compiled with the option -Dwebp=disabled.

Fix #1184: Put main menu items into one hamburger menu

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

A "hamburger" style menu icon is located to the left of the path entry
box.

Key F10 activates the menu.

Bug fix: SAR window crashes when escape pressed

Reduce the height of toolbars

The height of the toolbars was determined by a minimum size set in
GtkToolbar().
GtkToolbar is not in GTK4.
Horizontal boxes are used instead which result in a smaller icon size.

Use devel=enabled option for full-size AppImages

If the user crashes Geeqie, there may be useful data output.

Fix #1224: Dragging a keyword to another position causes Geeqie to crash

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

Incorrect change in commit 6d72bf20cc

Remove web page links status from README

There are too many false alarms to make this a worthwhile notification.

Bug fix: Run-time errors when removing a toolbar icon

- In Edit/Preferences/Toolbar Main|Status, when an item is removed there
are run-time errors
- Replace g_object_unref() with gtk_container_remove()

Minor change to Debian install script

Include Help file documentation for Open With

Update Help file

Update link to Unison in Other Software

Update developer install script

Include mdl snap.

Change GitHub issue templates to new style

The GitHub issue templates are in ./.github/ISSUE_TEMPLATES/

Check markdown files for syntax

- Include markdown checks using mdl - different methods are used for
local checks and GitHub actions
- Fix syntax errors in some .md files

Bug fix: Help files are not included in AppImages

Include yelp-tools (yelp-build) as a dependency.

Fix #1215: Unable to build with webp-pixbuf-loader on non-Debian systems

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

Include check for pacman.

I assume that DNF is still a problem.

Fix broken links in Help files

Include website status in README

The website project (https://github.com/geeqie/geeqie.github.io) has a
dead link check run every day.
Include the status in README.

Add loongarch64 support

Log Window F1 handling

- Pressing F1 in the Log Window send the selected text or complete line
to the command specified in Edit/Preferences/Behavior/Log Window F1
Command. It is left to the user to determine the action taken.
- Debug statements parameter order changed from  __FILE__, __func__,
__LINE__ to  __FILE__, __LINE__, __func__

Additional documentation for developers

Mostly to describe the source of non-standard icons.

exiv2: Fix use-after-free and double-free with exiv2 0.28.x

In exiv2 0.28.x, DataBuf is only a wrapper for std::vector.
Exiv2::DataBuf::data() returns a shallow copy of its underlying
data.  On Exiv2::DataBuf::reset(), all of its data is invalidated,
includes the data returned by Exiv2::DataBuf::data(). Thus, all usages
of said data is use-after-free, attempt to use its later is double-free.

Make a copy ourselves instead.  This method works for the old exiv2, too.
Since Exiv2::PreviewImage::size() is available from forever, and both
Exiv2::PreviewImage::copy() and Exiv2::PreviewImage::pData() are
introduced in a same change (commit 9578e8123 in 2008).

Fixes: #1201
Fixes: https://github.com/void-linux/void-packages/issues/47547

Open With feature

The File menu has an Open With menu item, and there is an Open With
plugin.

In both cases the standard GTK application chooser dialog is called.