Run github delete workflows once a week

Bug fix: gtk-builder-tool needs a gui

Bug fix: gtk-builder-tool operates on a single file

Install github workflow libgtk-3-bin for gtk-builder-tool

Convert search-and-run window to GtkBuilder .ui file

Conversion done only for future reference.

Bug fix: Seg. fault when OSD font not set

If Geeqie is started with no geeqierc.xml file and OSD is selected,
there will be a seg.fault.

Set the OSD font in default options - the font set does not seem to
matter.

fix typo

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

auto with g_new

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

clang-tidy: more

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

Revert "undo one conversion and reduce casting"

This change was wrong and crashes.

This reverts commit d4f7d83269fdb13acaa0e5bc25018a8bc75a0ee9.

Fix #1120: Geeqie crashes on problematic geeqie.xml config file

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

Object not unreffed.

Remove exiv2 0.28.0 restriction

https://github.com/BestImageViewer/geeqie/pull/1119

The bug was fixed in the above commit.

Add ExifData as extra argument to exif_item_get_data_as_text().

Bug fix: _XOPEN_SOURCE redefined error

The inclusion of <stdlib.h> causes this error. One solution is just to
move the declaration.

clang-tidy: fix readability stuff

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

Fix #1116: meson: Disabling pandoc disables installation of all documentation

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

clang-tidy: more missing fixes

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

Apply feedback from Colin Clark

Find and use libexecinfo (for backtrace()).

Fixes build on NetBSD.

GitHub action to purge old workflow runs

Add missing header for setlocale().

Add missing header for setenv().

Fix #ifdef for JPEG XL support

Minor text changes to CHECKLIST.md

Release v2.1 files

Update .po files

Update shortcuts documentation

Update create-shortcuts-xml.sh

Update to take into account change from NULL to nullptr and change to
use of static_cast.

Remove unused declarations

add missing stuff

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

more typedef removals

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

clang-tidy: more C cast removals

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

Exclude exiv2 version 0.28.0 from meson.build

Version 0.28.0 of exiv2 can cause a seg.fault in Geeqie.

See https://github.com/BestImageViewer/geeqie/issues/1090 for the
background.

Fix #1106: Split/Split pane sync - Shift+Arrow keys scrolling is out of sync on high zoom

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

I do not know which situations the previous code was designed to handle
- there may be some problems in test cases I did not use.

Fix coding errors

Fix "Use of memory after it is freed" errors identified by clang-tidy

Script to install files needed for development use

Include descriptive text for Generate-ClayRGB1998-icc-h.sh

Update JPEG XL handling

clang-tify: remove void

Found with modernize-redundant-void-args

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

Rationalize shellchecks in meson.build

clang-tidy: use using

Found with modernize-use-using

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

Simple script to generate doxgen documentation

- Ignore ./subprojects folder from doxygen documentation

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