AppImage build should be only on push, not on pull_request

Remove some extra memory allocations

- Use g_string_free(string, FALSE). Also use len member instead of strlen(string->str).
- Use string literals instead of building string for resources' paths.

Include some missing function statics

Update AppImage notification for new file name

Check date on GitHub for file name Geeqie-latest-x86_64.AppImage for
both x86 and arm.

Create minimal AppImage

The standard Geeqie AppImage contains all features but is about 120MB,
and therefore takes a noticeable time to load. The minimal version is
about 10MB but is still capable of displayed a useful range of images.

Update README for AppImage location

ChangeLog not needed for AppImage

Use GitHub workflow actions to generate AppImage

Replace some deprecated GTK functions

- Replace gtk_{button,combo_box}_set_focus_on_click with gtk_widget_set_focus_on_click.
- Replace gtk_misc_set_padding with gtk_widget_set_margin_start.

Fix #1151: Can't re-add items to collection after accidentally removing them

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

Option in Preferences/Behavior to permit duplicates also solves this
problem.

Remove some deprecated GdkScreen API

Fix assigned but never used warnings

Use more standard icons

- Replace ui_tabcomp.png with go-last named icon,
  fixing gdk_pixbuf_new_from_inline deprecate warning
- Replace gq-icon-rotate-counter-clockwise.png with object-rotate-left named icon
- Replace gq-icon-rotate-clockwise.png with object-rotate-right named icon
- Replace gq-icon-filp.png with object-flip-vertical named icon
- Replace gq-icon-mirror.png with object-flip-horizontal named icon
- Replace gq-icon-info.png with document-properties named icon
- Use recommended pan-{down,up}-symbolic named icons

Fix FIXME: useable_toolbar_items []

Line 64: toolbar.cc
/** @FIXME Should be created by program from menu_entries[]
 * in layout-util.cc */

Use get_action_items() instead.

Allow .desktop files to be linked to programmable mouse buttons

Desktop files can be linked to mouse buttons 8 and 9 on
Edit/Preferences/Behavior

Insert missing button label

Replace handmade spinner with GtkSpinner

Implement remote keyboard actions

Enable keyboard actions to be initiated from the command line.

Additional remote commands:
geeqie --remote --action:<ACTION>
geeqie --remote --action-list

e.g.
geeqie --remote --action:ZoomIn

The action codes can also be found in /Edit/Preferences/Keyboard.

Part fix #1159: Continuous zoom with mouse button

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

Mouse buttons 8 and 9 (labelled as back/forward in Preferences/Behavior)
now work in fullscreen.

Modify part of Collections user interface

Collections .gqv files are expected to be in the default directory.
The user no longer sees file paths or file extensions, only the
collection name.
If the user navigates to a .gqv file located elsewhere and opens it, the
contents will be displayed and operated on as a collection. It will
appear in the "recently used collections" menu (there may be a name
conflict) but not on the "open collection" menu, which shows only the
collections in the default directory.

Remove deprecated GtkArrow

Use non-standard office-calendar icon.

Bug fix: Pan view not working

Fix bug created in commit 689b9c6495f5.

Remove stray comment characters.

Bug fix: Button labels and accelerators

It seems that something has changed in the libraries. An explicit call
to gtk_button_set_use_underline() is required for the accelerators to
work.

Fix #1156:  Status bar remains empty on selecting collection item while navigated to empty folder

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

Drop goto in zonedetect.cc

Use while loop and smart pointers.

Drop goto in secure-save.cc

Use lambda and smart pointers.

(Re)-implement natural and case sorting

The directory and file panes now have natural (numeric) and case
sorting.

Other windows sorting is unchanged.

Drop goto in copy_file()

Use lambda and smart pointers.

Fix #1148: Intermittent crash on removing first item in a collection

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

Use pixbuf_fallback() instead of duplicated code

Bug fix: Include Lua tests

When running as a GitHub action, error messages such as:

(geeqie:12363): dbind-WARNING **: 13:24:40.471: AT-SPI: Error retrieving
accessibility bus address: org.freedesktop.DBus.Error.ServiceUnknown:
The name org.a11y.Bus was not provided by any .service files

are included at the head of the run output. Work around the problem by
taking the last n lines of output.

Include Lua tests

Fix memory leaks in lua.cc

Move declarations to their first use.

Replace MD5Context with GChecksum

- Replace md5_get_digest with md5_get_string
- Add md5_get_string_from_file
- Add MD5_SIZE constant

Remove deprecated GtkAlignment

Use GtkWidget:halign and GtkWidget:valign properties.

Remove deprecated orientation-specific widgets

- Replace GtkHButtonBox with GtkButtonBox.
- Replace GtkHScrollbar, GtkVScrollbar with GtkScrollbar.
- Remove GtkHSeparator, GtkVSeparator with GtkSeparator.

Remove deprecated GTK_STOCK_DIALOG_*

Use standard icon names and rename related GQ_ICON macros.

Remove deprecated GtkHPaned, GtkVPaned

Use recommended GtkPaned.

PR #1139: Remove some of deprecated GTK stuff
https://github.com/BestImageViewer/geeqie/pull/1139

Remove deprecated gtk_adjustment_changed
Remove gtk_tree_view_set_rules_hint
Get rid of half of deprecated stock icons: menu items
Replace deprecated gdk_cursor_new with gdk_cursor_new_for_display
Add fullscreen icon
Replace deprecated gtk_tree_view_get_vadjustment with gtk_scrollable_…
Get rid of another big chunk of stock icons: buttons
Replace literal icon names with #defines in main.h, replace some icon…
Leftover cleanup after previous series of commits
Replace a few left literal icon names with #defines
Fix icons in sort bar and toolbar customization
Remove leftover line

Fix memory leak in log_window_append()

Use deque and string.

Fix usage of vectors

reserve() doesn't allocate elements.

Clarification of Desktop file Category

- Detail that either or both of the following is required
Categories=Graphics;
Categories=X-Geeqie;

- Amend template-desktop.sh for new layout

Remove UNUSED macro

C++ allows unnamed parameter.

Fix some memory leaks

- Use vector to avoid memory leak on array reallocation
- Use unique_ptr to avoid memory leaks

Fix dangling pointer in jxl

Reformat code.

Add CommandBuilder in order to drop goto in editor_command_parse

Various fixes

Use g_list_free_full

Remove parts_list_free

Use g_list_free_full with path_data_free

Remove fullscreen_prefs_list_free

Use g_list_free_full with screen_data_free

Replace collection_list_free with g_list_free_full

Replace dupe_list_free with g_list_free_full

Replace string_list_free with g_list_free_full

Remove list_free_wrapper

Use g_list_free_full

Remove tree_path_free_wrapper

Do not run tests on unsupported image type

Ensure that image types not support by the run-time system are not
tested.

Image test runs - check if Geeqie has crashed

Part fix #1058: Deleting a symbolic link that is a directory results in a crash

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

This patch simply aborts the delete to avoid the crash. It does not fix
the problem.

Remove dead code

Dead code found with:
ag '//.*;' src/

Change doxygen default from C to C++

fix compilation with lua

extern C is needed.

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

Create image from icon name instead of stock id

More unused functions, remove dead code

The dead code was identified by:
https://gist.github.com/sbeyer/192324f8a8470ce36b27
find-code-in-comments.rb

Document hidden duplicates window mouse click

Control + Shift + right click pops up window showing:
name
size
date
dimensions
md5sum
thumbprint

Reorganize unused functions

Some unused functions remain within the code in case they become useful.
Pragmas are set so that the compiler does not produce warnings.
They may be easily found in the documentation generated by Doxygen.

Re-insert geeqie.cppcheck

cppcheck is still useful.

Bug fix: Incorrect regex in document generator

Remove redundant files (2)

Remove redundant files

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