Preparing for release v2.4

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)

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