Include clang-tidy check

- Initial check setup
- Separate log files for GitHub workflow runs

vector conversion

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

add missing free

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

Remove redundant compiler options

Eliminate “might be clobbered” warnings

- image-load-cr3 Will be deleted when libraw and exiv2 handle .cr3 files

- image-load-jpeg - this code is so old it should not be changed

manual loop conversions

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

use vector

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

clang-tidy: remove misleading const

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

clang-tidy: get rid of const in declarations

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

clang-tidy: simplify booleans

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

Insert #ifdef HAVE_ around some #includes

Clean up UNUSED warnings and others

Eliminate gq-marshal compile warning

glib-genmarshal produces code that results in:

warning: ISO C forbids conversion of object pointer to function pointer
type [-Wpedantic]

Insert a pragma at the start of the code to inhibit this.

Eliminate gchar return ignored warning

warning: ignoring return value of ‘gchar*
    g_string_free_and_steal(GString*)’ declared with attribute
    ‘warn_unused_result’ [-Wunused-result]

Eliminate gchar return ignored warning

warning: ignoring return value of ‘gchar*
    g_string_free_and_steal(GString*)’ declared with attribute
    ‘warn_unused_result’ [-Wunused-result]

Eliminate gchar return ignored warning

warning: ignoring return value of ‘gchar*
g_string_free_and_steal(GString*)’ declared with attribute
‘warn_unused_result’ [-Wunused-result]

Eliminate compiler warning - used unititialized

Eliminate variable length array warning

warning: ISO C++ forbids variable length array ‘IDs’ [-Wvla]

Eliminate gchar conversion warnings

warning: ISO C++ forbids converting a string constant to ‘gchar*’ {aka
‘char*’} [-Wwrite-strings]

Eliminate "not used" warnings

Bug fix: GitHub workflow image tests

Avoid running the remote until it is initialized

clang-tidy: fixes from version 16

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

Bug fix: GitHub workflow image tests

libjxl is not in Ubuntu 22.04

Bug fix: GitHub workflow image tests

More fixes

Bug fix: GitHub workflow image tests

Various fixes.

Include faulty image test in GitHub workflow tests

libdwarf is required for Ddevel=enabled on GitHub

libunwind is required for Ddevel=enabled on GitHub

libdw is required for Ddevel=enabled on GitHub

Include image tests in GitHub workflow

File formats no longer supported - .apm, .icns, .wmf

- .apm  Aldus Placeable Metafile
- .icns macOS icon
- .wmf Windows Metafile

Remote --get-file-info method

If no pixbuf has been created, return file class Unknown

File formats no longer supported - .ptx, .x3f

- .x3f (Sigma) Exiv2 no longer suports this format -
https://github.com/Exiv2/exiv2/issues/2325

- .ptx (Pentax) No test images could be found

Bug fix: .xbm files are not recognized unless the mime type is given

Optimize GitHub workflow tests

Run shellcheck and gtk-builder-tool only when all options are disabled.

Update AppImage notification

- AppImages are now on GitHub
- Update regen_potfiles.sh to include .ui files

Fix #1124: Search in keyboard preferences tab

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

Click anywhere in a column and type.

Create .asc files for AppImages

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