Run shellcheck in meson tests

Run shellcheck on script files

Run using:

for file in $(find . -not \( -path ./subprojects -prune \) -name "*.sh")
do
realfile=$(realpath "$file")
shellcheck --norc --format=diff --shell sh
--enable=add-default-case,avoid-nullary-conditions,check-unassigned-uppercase,deprecate-which,quote-safe-variables
"$realfile"| patch "$realfile"
done
exit

Plus manual edits.

clang-tidy: use nullptr

Found with modernize-use-nullptr

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: use override

Found with modernize-use-override

Signed-off-by: Rosen Penev <rosenp@gmail.com>

exiv2: use unique_ptr

Shorter and no delete needed.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

meson: set cpp standard to 14

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: replace int with bool

Found with modernize-use-bool-literals

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Run GitHub actions also on pull_request

manual fixes for C casts

clang-tidy does not fix these as they require both const and
reinterpret_cast.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

remove const_cast

No need for it. Nothing is modified.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

undo one conversion and reduce casting

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: C to C++ casts

Ran with:

run-clang-tidy -fix -checks=google-readability-casting

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Fix #1100:

https://github.com/BestImageViewer/geeqie/issues/1100#issuecomment-1581251045

Another signature for .heic files.

clang-tidy: modernize-use-auto

clang-tidy: modernize-use-auto

Signed-off-by: Rosen Penev <rosenp@gmail.com>

convert loops to for range

simpler

Signed-off-by: Rosen Penev <rosenp@gmail.com>

remove bad form in if statement

clang-tidy suggests braces here. instead of that, fix the condition

Signed-off-by: Rosen Penev <rosenp@gmail.com>

remove register

depricated and removed in future standards

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Fix #1093: Double click for full screen

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

Fix #1090: exiv2 0.28.0 support

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

Bug fix: AppImage notification

Do not check for an AppImage update if a release version is being run

Fix #1096: Debian install script fails on Debian 11.7

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

Eliminate warnings when creating .pot file

Include check for existence of xvfb in meson.build

Update GitHub actions file (part 2)

- Run apt-get update
- Use specific Ubuntu version number

Update GitHub actions file

- Run apt-get update
- Use specific Ubuntu version number

Documentation: Use G_SOURCE_CONTINUE and G_SOURCE_REMOVE

Fix typo in Readme file

Include package status in the Readme file

Fix typo in previous commit

Include executable test in GitHub action

- Include basic executable test - get version
- Use -D<option>=disabled for the no-options test

Include a build status badge in the Readme file

Include GitHub action to check builds

Protect another g_memdup2 in exif.cc with ifdef

Replace g_memdup with safer g_memdup2

g_memdup is declared as deprecated in current GLib, safer replacement g_memdup2 is available since GLib-2.68.

Code is protected by GLIB_CHECK_VERSION macro, so should build on older distros as well.

See https://docs.gtk.org/glib/func.memdup2.html

Add missing #ifdef for DEBUG_FD

Amends commit cd72fa8

Revert "FIXME: this can be rather slow and blocks until the size is known"

This reverts commit 8aeeb06fa5ab39f9e58938c2a8af3029fd67cbff.

Uncomment the callbacks.

Revert "FIXME: this can be rather slow and blocks until the size is known"

This reverts commit 8aeeb06fa5ab39f9e58938c2a8af3029fd67cbff.

gdk_pixbuf_get_file_info() can only get the file info from file types that it knows.

Bug fix: Copy icon not displayed

In the Copy dialog, the Copy icon was shown as a broken icon. Reverting
from gtk_image_new_from_icon_name() to gtk_image_new_from_stock() solves
the problem. I do not know why.

Do more casting to avoid build errors

Use emblems for some folder icons

- Use emblems for read-only, symlink and no-access folder icons
- Use standard folder icon for shortcuts and bookmarks
- Remove redundant GQview icons

n.b. It was not possible to get g_emblemed_icon_new() to work, so the
interim solution is to use gdk_pixbuf_composite()

Don't use too new *_DEFAULT defines

Remove "dot" entry from shortcuts pane

The "dot" entry had no meaning in the shortcuts pane, so is removed from
the defaults.
Current users can now delete the entry with a right-click.

Fix #1080: Cannot build with meson 0.56.2

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

Drop stale intltool references

Moved to gettext when porting to meson.

Signed-off-by: Sam James <sam@gentoo.org>

Bug fix: Generation of translators list

Wrong text offset used

Fix #1079: Delete a keyboard shortcut

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

Additional button on Preferences/Keyboard to clear selected shortcuts

Credit webpage artwork

- Credit webpage artwork
- Remove unused variable

Revise gresources use

Auto-generate list of authors and translators

- Get list of authors from git log, ordered by number of commits
- Get list of translators from headers in the .po files

- Lists are displayed in the About dialog

Bug fix: Error in typecast

Fix #1077: allow split view with user specified layout

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

Additional layout - 3 horizontal panes

Fix c++ compiler warnings

... and this one...

Fix c++ compiler wrnings

Most of the remaining warnings eliminated

Fix c++ compiler warnings

Geeqie compiles without the -fpermissive option,

Fix c++ cmpiler warnings

GdkModifierType cast

Fix c++ compiler warnings

Bug fix for previous commit

Fix c++ compiler warnings

const gchar * casts

Fix c++ cmpiler warnings

Include the gconstpointer casts missed from the previous commit

Fix c++ compiler warnings

An awk script was used to generate the changes. Hand editing was also
required:

function ltrim(s) { sub(/^[ \t\r\n]+/, "", s); return s }
function rtrim(s) { sub(/[ \t\r\n]+$/, "", s); return s }
function trim(s) { return rtrim(ltrim(s)); }

BEGIN {
}

/invalid conversion from ‘gpointer’/ {

split($0, file, ":");  # file-name line-number
split($0, fields, "[‘’]"); # get the typecast - these two characters are
UTF8 and are NOT the same

split(fields[6], cast, "*")

getline;

split($0, code, "[|=;]");

print "sed -i '" file[2]"s/"code[3]"/ static_cast<" cast[1] " *>("
trim(code[3]) ")/' " "./src/"file[1] ;

Change c-style typecasts to c++ style

Change the c-style typecasts inserted in commit 3de5f843397ae5f to c++
style static_casts

Fix more c++ compiler warnings

Fix some c++ compiler warnings

GTK4: button-press-event deprecated

GTK4 migration
- replace button-press-event with gtk_gesture_click

GTK4: button-press-event deprecated

GTK4 migration
- replace button-press-event with gtk_gesture_click
- include the mechanism for HAVE_GTK4 (but this is not for users)

Add my name to translators

Update Dutch translation (nl)

Update documentation for --selection options

Update man page and Help file/Reference for new command line options.

Ref #1025: "Move to trash" does not do anything (cross-filesystem)

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

Co-authored-by: BecauseTheWorldIsRound
- A warning dialog is displayed if trash fails
- A workaround is documented

Bug fix: Layout window id not set

If there is no geeqierc.xml file, the lw->options.id is not set. This
leads to a glib critical error in some cases.

Adds a `--selection-remove` command to the remote API.

This allows removing any selected file from the selection set, either by filename or full filepath.

Adds a `--selection-clear` command to the remote API.

This simply clears the current selection.  Note that this _does not_ re-add the current image to the selection.

Adds a `--selection-add` command to the remote API.

This allows adding any listed file to the selection set, either by filename or full filepath.

Note that this is responsive to the UI, so (for instance) will rely on which directory is open, and whether sidecars for a given file are expanded or not.

Adds a `--get-selection` remote command that shows selected files (including sidecars) in the same style as `--get-filelist:`

Bug fix: keyword_store not initialized

If no geeqierc.xml file, the keyword_store was not initialized which
resulted in glib critical errors.

Bug fix: Glib Critical error when no configuration file

If geeqierc.xml does not exist, there is a critical error -
file_data_pool is not initialized.

Extended stack trace

- Extended stack trace using backward-cpp. Enabled by option "devel"
- Trap more signals when not using backward-cpp
- Remove tee stderr to log window - does not work with backward-cpp, but
is of low value

Bug fix in debug backtrace

Document animated webp files

Use webp-pixbuf-loader instead of image-load-webp()

webp-pixbuf-loader also supports animation

Fix #1067: Restore zoom quality after viewing a GIF

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

Seg fault handler

- Local handler for seg. fault
- Stack dump on seg. fault
- Stack dump option in meson_options.txt
- Bug fix in DEBUG_BT()

Fix error in README - meson configure

Update usage documentation in plugins template

Bug fix: Remote option --list-add parameter

If the parameter to --remote --list-add: is not a full path to an
existing file Geeqie will crash.

Additional doc. for X-Geeqie-Filter in Sort Manager

Bug Fix: Use GResource for icon files

String concatenation error.

Use GResource for icon files

Include DEBUG_BT() and DEBUG_FD() functions

- DEBUG_BT print backtrace of Geeqie calls
- DEBUG_FD print dump of FileData hash file

Fix #1061: meson fails to detect strverscmp

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

Delete Sort By Number option. Use Natural Sort Order on
Edit/Preferences/File Filters instead.

Fix #1002: New build-dependency xxd

https://github.com/BestImageViewer/geeqie/issues/1002#

Fix #1002: New build-dependency xxd

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

Include checks for xxd or xxdi.pl in meson.build. If neither exist,
disable color management.

Fix #1002: New build-dependency xxd

xxdi.pl doesn't accept output file as a second parameter and needs a redirect >. Since vim xxd in case of no second parameter uses stdout by default, it is safe to add a redirect in any case.

Fix #990: Display full-resolution embedded jpg from CR3 files

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

- Use libraw in preference to exiv2, as it will extract the largest
preview image from a .cr3 file

- Revise export-jpeg plugin to also use exiftool when it detects a
larger number of preview images than exiv2, and additionally include
image auto-rotate

Script to compile linuxdeploy on arm

Bug fix: Ensure LUA API is displayed in Help files

The source file suffix is now .cc and not.c

Bug fix: Typo in Help file

Note that Geeqoe can be run on Windows 11

Preferred date format for documenttaion

Use ISO format (YYYY-MM-DD)

Reminder to update Wikipedia entry in CHECKLIST.md