Merge branch 'rot-invariant'

This implements the rotation invariant duplicates search, implemented by Cyril
Roussillon.

The Feature is enabled by default but can be disabled in preferences dialog.

Please note that, due a slightly different fingerprint algorithm, the .sim files
needs to be deleted manually and recreated to get the full benefit.

* rot-invariant:
  Allow to configure rotation invariance
  Slightly better similarity samples
  Implementing rotation invariant duplicates search

Allow to configure rotation invariance

Slightly better similarity samples

Slight modification of the image_sim_fill_data, that creates the
similarity signature. When the image size is not a multiple of 32, all
cells had size w/32, so the last w%32 pixels were just ignored, and more
importantly it was not symmetric at all when the signature was built
from the other side, which is annoying for my comparison algorithm. So I
changed the cell size (x_inc) from a constant one to
"round(w_left/n_left)", in order to spread the spare pixels (eg to split
42 pixels into 4 cells, it results in 11,10,11,10 instead of
10,10,10,10). With an image of size w with w%32=16, my
rotation-invariant algorithm results in a similarity of
0.951000 with the original version, and 0.999968 with this modification
(apparently lossless jpeg rotation is not perfectly lossless).

Signed-off-by: Klaus Ethgen <Klaus@Ethgen.de>

Implementing rotation invariant duplicates search

Modifies the functions image_sim_compare and image_sim_compare_fast so
that it compares with the eight possible isometric transformations
(compositions of 90°-rotations, mirrors, transpose,...), using the same
similarity signature.

Signed-off-by: Klaus Ethgen <Klaus@Ethgen.de>

Better configure

Remove obsolete and outdated debian directory

Debian replaces the content of this directory anyway and it does not
match the current project state anymore. So lets remove it.

That does not mean that it is gone forever but then I will most likely
use the original debian one.

debian: point watch file to github

Patch to fix duplicate window update

Fixes #190.

Signed-off-by: Klaus Ethgen <Klaus@Ethgen.de>

Merge Colins changes

* 'master' of git://www.geeqie.org/geeqie:
  Lua Documentation

Lua Documentation

Basic Lua documentation.

Requires expansion and re-work.

Avoids a double-free

Avoids a double-free due to a boolean that would be retrieved from a
deleted/freed structure.

Fix #361.

Signed-off-by: Klaus Ethgen <Klaus@Ethgen.de>

Removed some obsolete tools

Improving desktop template

Small dependency to fully mitigate the marshaler stuff

Remove autocreated files from git

That is a nasty bug. The files gq-marshal.[ch] gets created from
gq-marshal.list but are checked in to git.

I removed that files now from git so they get recreated properly every
time. (Closes #383)

Thanks Russel Winder for the insistent nagging about that.

Update German translations

Issue: #387

Update from pot file

Fixes #364.

Enable greek translation

Thanks to Nikos Papadopoulos.

See #364.

Fix a typo

This bug was introduced in fe69cac by John (2005-03-19).

Fixes #386.

Merge Colins changes

* 'master' of git://www.geeqie.org/geeqie:
  Document toolbar enhancements

Set git version in master branch

Removing not used configure variables

$GQ_PATCH_VERSION and $GQ_PATCH_DATE was never used.

Document toolbar enhancements

Update help file to describe how to put additional icons on the toolbar.

Requires re-work.

Merge Colins changes

* 'master' of git://www.geeqie.org/geeqie:
  Fix #211 Corrupt help file

Fix #211 Corrupt help file

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

Section 4.4 External Editors now displayed.

Requires much more re-work.

Update German po file

Adding Klingon po file

Searching for native speakers. ;-)

Fix #333 Renaming file extensions

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

The Preferences/Files tab has an additional checkbox:
Disable file extension checks

When checked, files can be renamed to a different extension with no
warning issued.

Merge merge request 379

* github/merge-requests/379:
  'Stay above' option now works independently of screen for fullscreen window
  Fix fullscreen - remove all the garbage which breaks things and let Gtk do its job.

Fix crash on drag and drop from Xfe

The issue is that when dragging and dropping from Xfe, paths aren't
escaped like they are when dnd'ing from a GTK app.

'Stay above' option now works independently of screen for fullscreen window

Fix fullscreen - remove all the garbage which breaks things and let Gtk do its job.

The most problematic thing was the `gdk_window_set_override_redirect`
call. That was what made most of the mess. Another bad idea was to deny
resizing of the fullscreen window, which causes window manager to fail
to adjust window size to screen size.

Fixes #222, #218, #103.

Work around GCC6

This problem is really annoying. Left shifting a value does not require
it to be positive in C. However, some specifications define that as
undefined behaviour.

Nevertheless, as used in this case it is perfectly fine and widely used
construct.

Closes #230, #378

Issue #332 Rename function can delete files

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

During a Rename operation, if the user assigns a single destination
filename to multiple source files, an error dialog is shown and the
operation is aborted.

Updated web.

Tuning file types for Github

Updating web

Issue #329 easier way to get cwd when copying

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

A bookmark with the name "." (a single full-stop) is treated as a
special case. Each time it is displayed, its path is loaded with
Geeqie's current working folder. This bookmark, like any other, can be
deleted and re-created at any time.

Update command line options documentation

Update both the Help file documentation for command line options, and
the output of the -h and -rh command line parameters.

Bug fix #227 + enable user to set height of Keywords pane

Bug #227 Keywords section of Info sidebar doesn't work on GTK3 build
https://sourceforge.net/p/geeqie/bugs/227/

Permit the user to set the height of the Keywords pane in the Info
sidebar by editing the line:

<pane_keywords id = "keywords" title = "Keywords" expanded = "true" key
= "Xmp.dc.subject" height = "200"/>

in the configuration file geeqierc.xml

Update for new cogl version

Parameters for cogl_texture_3d_new_from_data have changed.

cleanup of sourceforge references

Signed-off-by: Klaus Ethgen <Klaus@Ethgen.de>

Fix web page

one slash too much

Merge branch 'web-improvement'

* web-improvement:
  Web page update

I changed mostly the links to protocol relative ones.

Web page update

How about dropping as much sf links as possible from the index page?

I attached an updated geeqie.org/index.html.

Re-arranged a bit,
only kept the direct link to the lists.sf subscribe page,
dropped all other sf links,
added two ML archive links,
and re-worded to be more "proud" and self-confident.

It has been eight years since the fork from GQview,
that should not be the one distinct point to make.

Rather, (cl)aim to be the best image viewer.
Sometimes it's not enough to be good.
Sometimes people need to also notice it.

Updating web page

Merge branch 'compiler_warnings'

* compiler_warnings:
  Really fix issue in 2a9e4d6

Really fix issue in 2a9e4d6

Merge branch 'compiler_warnings'

* compiler_warnings:
  This potentially fixes most deprecated warnings
  Fix wrong gtk3 function calls in gtk2
  Export file_data_new_simple into other source files
  Fixed one wrongly function call with pointer

This potentially fixes most deprecated warnings

However, I tested it with gtk 2.46.2 on debian that pretend to be
version 2.24.29. So I still get that warnings.

Fix wrong gtk3 function calls in gtk2

Export file_data_new_simple into other source files

Fixed one wrongly function call with pointer

This is a potentional dangerous call to LCMS function. There was a
compiler warning but nobody cared about before.

Merge branch 'sort-by-ctime'

* sort-by-ctime:
  Applying that sort method also to collections

Applying that sort method also to collections

Merge branch 'sort-by-ctime'

* sort-by-ctime:
  Implementing sorting by file creation

Implementing sorting by file creation

exiv2: Fix use-after-free of the image filename/path.

This caused garbage to be returned when we attempted to use the path to
determine if the file is a RAW image.  Now, RAWs are detected properly
and the proper full-res previews are used.

Signed-off-by: Solomon Peachy <pizza@shaftnet.org>
Signed-off-by: Klaus Ethgen <Klaus@Ethgen.de>

Remove some useless spaces in exiv2.cc

Updating README

Reintroduce file_data_new_simple()

This was removed by Vladimir with
efc8096527d6f743d17c44b9982616a169973f39 to always check for sidecars.

However, the newly introduced function file_data_new_group() is very
expensive. Having it in collection (in sort manager) where the dirs of
that files have huge amount of files inside, that is simply blocking
whole geeqie.

It might be that the file_data_new_group() function need to be disarmed
on other places too.

Tune some debug info

Correcting return type

Allow to choose to show parent folder

With e9fc136 we learned buttons for parent folder in directory list.
This disabled the ".." handle in the view itself.

This was found to be not optimal as the users are used to it.

With this patch, that entry is now configurable in preferences. It is
enabled by default but could be disabled if the user don't like it.

Fix nasty double free crashes

Old GTK+ versions seemed to be robust about this double free so we seen
that crashes or 100% CPU usages just with modern versions.

This patch takes care of it on two places:
1. Hashes will not be freed by g_hash_table_destroy when empty
2. NULLify the enty variable after free

Feature Request #70 Navigation menu option and button for parent directory

This request was made some years ago:

https://sourceforge.net/p/geeqie/feature-requests/70/

I have attached a patch which implements this. Also, keyboard-only users
will be able to use keyboard-accelerators to map a key to the Up function.

Unfortunately there is a usability problem. If the configuration file
geeqirc.xml exists (which it will except for first-time installations)
geeqie will not look at the programmed defaults, but will create the
toolbar from the config file - and the Up icon will not be displayed.

If the user first executes a command such as this:

sed -i '/toolitem action = "Back"/a <toolitem action = "Up" />'
~/.config/geeqie/geeqierc.xml

then all will be well.

This is not ideal, but I see no other solution.

Colin Clark

Changing reference to sourceforge

Change sourceforge to geeqie.org.

Include missing glua.h

Updating web site

Fix toolbar visibility

The toolbar should not be visible on startup when the
configuration file has the line

    toolbar_hidden = "true"

Its visibility state at startup should be the same it
was at the last shutdown.

Merge requests 12, 13 and 14 into master

* origin/merge-requests/12:
  Fix typo in debug messages

* origin/merge-requests/13:
  fix include statements for large file support on 32-bit

* origin/merge-requests/14:
  Add support for Green-Magenta and Yellow-Blue anaglyph modes

NULL-ptr crash in image_read_ahead_done_cb()

It's not easy to reproduce, but if one is mad and follows the steps and
presses the keys a few dozens times, there's a slight chance one can
trigger it. Not much to investigate it further.

Forwarding from Sami Farin 2015-08-07 16:40:44 EDT:

 | to reproduce: quickly keep pressing f (fullscreen) and page down
 | (next file) keys.
 |
 | attached patch is a kind of workaround, I did not inspect further
 | is this threading issue etc

Full backtrace: https://bugzilla.redhat.com/attachment.cgi?id=1027647

Patch: https://bugzilla.redhat.com/attachment.cgi?id=1060482&action=diff&context=patch&collapsed=&headers=1&format=raw

Remove over optimization

Thanks Uwe Ohse <uwe@ohse.de> for this find.

"it looks good, but is dead slow, for each and every call to
gdk_cairo_set_source_pixbuf takes at least 15 milliseconds on my not
really slow machine, which amounts to over 2 seconds if 144 tiles have
to be drawn (tested with many jpegs and a few large pngs).

i propose to remove this optimization until someone fixes it. After
removal geeqie is fast, again."

Add support for Green-Magenta and Yellow-Blue anaglyph modes

fix include statements for large file support on 32-bit

Fix typo in debug messages

vindow -> window

Fix lcms2 integration

Thanks Michael Schwendt <mschwendt@gmail.com> for the bug reporting and
the patch.

updated ChangeLog file

Merge remote-tracking branches 'merge-requests/6' and 'merge-requests/7'

* merge-requests/6:
  Bug 3594998: make lirc initialization quieter

* merge-requests/7:
  Add filedata counting to watch for filedata leaks
  Use FileData locks to avoid expensive reloads with marks enabled
  Add "lock" functionality to keep FileDatas in memory

Merge remote-tracking branch 'origin/merge-requests/5'

* origin/merge-requests/5:
  Fix crash when loading collection listing inexistent files.

This is now a combination of Leons patch 900e65e and merge request of
Laurent. Hopefully that is now a bit more stable.

Fix for the assertion when file gives an empty lookup

Thanks Leon for this patch.

fix segfault while cancel rename dialog

Patch prevents calling gtk_widget_destroy(gd->dialog) twice - second is
in generic_dialog_click_cb:58

Steps to reproduce:
select more images on left section then is seen in on page (need to
scroll), roll to the top, click by right mouse button, select Rename
from menu and cancel it by key Esc (button Cancel is working).

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff79b2b16 in gtk_widget_destroy () from /usr/lib/x86_64-linux-gnu/libgtk-3.so.0

(gdb) bt
 #0  0x00007ffff79b2b16 in gtk_widget_destroy () from /usr/lib/x86_64-linux-gnu/libgtk-3.so.0
 #1  0x0000000000429c6f in generic_dialog_close (gd=0x118d700) at ui_utildlg.c:44
 #2  0x0000000000429cf2 in generic_dialog_click_cb (widget=0xb12220, data=0x118d700) at ui_utildlg.c:58
 #3  0x0000000000429e4a in generic_dialog_key_press_cb (widget=0xb12220, event=0x11f0800, data=0x118d700) at ui_utildlg.c:93
 #4  0x00007ffff78942bc in ?? () from /usr/lib/x86_64-linux-gnu/libgtk-3.so.0
 #5  0x00007ffff6123f28 in g_closure_invoke () from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
 #6  0x00007ffff6134edd in ?? () from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
 #7  0x00007ffff613c922 in g_signal_emit_valist () from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
 #8  0x00007ffff613cf32 in g_signal_emit () from /usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0
 #9  0x00007ffff79b4664 in ?? () from /usr/lib/x86_64-linux-gnu/libgtk-3.so.0
 #10 0x00007ffff78927cf in ?? () from /usr/lib/x86_64-linux-gnu/libgtk-3.so.0
 #11 0x00007ffff7893f95 in gtk_main_do_event () from /usr/lib/x86_64-linux-gnu/libgtk-3.so.0
 #12 0x00007ffff74cee12 in ?? () from /usr/lib/x86_64-linux-gnu/libgdk-3.so.0
 #13 0x00007ffff5e5cea6 in g_main_context_dispatch () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
 #14 0x00007ffff5e5d1f8 in ?? () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
 #15 0x00007ffff5e5d5fa in g_main_loop_run () from /lib/x86_64-linux-gnu/libglib-2.0.so.0
 #16 0x00007ffff789344d in gtk_main () from /usr/lib/x86_64-linux-gnu/libgtk-3.so.0
 #17 0x0000000000492cf1 in main (argc=2, argv=0x7fffffffd4c8) at main.c:921

Updated Chinese locales

Signed-off-by: Klaus Ethgen <Klaus@Ethgen.de>

Merge remote-tracking branches 'origin/merge-requests/10' and 'origin/merge-requests/9'

Merge in some trivial but useful merge requests. (Note that 9 includes 8.)

* origin/merge-requests/10:
  transifex config file allowing to easily use transifex-client

* origin/merge-requests/9:
  Skip over expanded keys.
  Annotate debug_exception() with source file, line, and function.

transifex config file allowing to easily use transifex-client

Skip over expanded keys.

This fixes handling of filenames with % in them.

Annotate debug_exception() with source file, line, and function.

Bug 3594998: make lirc initialization quieter

Increasing debug level to 1 or 2 will give more information, level 0
will only output a message to user if lirc is initialized but can't
read config file.

Fix crash when loading collection listing inexistent files.

Add filedata counting to watch for filedata leaks

This basically counts the number of filedata instances in existence at
any given point in time.  If the number doesn't drop down to where it's
supposed to be, there are leaks.

Use FileData locks to avoid expensive reloads with marks enabled

Add "lock" functionality to keep FileDatas in memory

Adds two primitives to keep a FileData in memory even after its refcount
drops to zero.  Intended to be used in situations when an entire group of
FileDatas _should_ stay in memory as an optimization, even if the code
would continue to function properly even if the FileData were freed.

Fix a segfault caused by .desktop file with no exec= line.

https://bugs.launchpad.net/ubuntu/+source/geeqie/+bug/1056532

Reapply 'call notify functions in in an idle call'

085be43cb79e04341102a9922e0e0f531454089a did some regression that should
have fixed this.

Whitespace cleanup

Merge branch 'ke-lua'

Mostly implementing lua integration.

* ke-lua:
  Revert "call notify functions in in an idle call"
  Implementing some usefull data structures for lua
  First usable lua callback
  Using common path for lua files
  Overlay info with lua
  Adding support for lua scripting

Merge branch 'master' into ke-lua

* master: (185 commits)
  better fix for big-endian architectures
  Minor indentation fixes.
  Cleanup checker shader code.
  Fix exif orientation in clutter rendering mode.
  Silent 2 warnings using explicit type casting.
  Polish translation was updated, thanks to Grzegorz Gibas.
  Update pt_BR translation.
  Add *~, *.rej, *.orig files to git ignore list.
  Fix for support >=automake-1.11.4
  Trim trailing white spaces on empty lines.
  Trim trailing white spaces.
  Add missing vim modeline.
  Fix indentation and few trailing white spaces.
  Clear few more warnings, unused variables mostly.
  Drop unused variable.
  Remove commented out code.
  Remove commented out code.
  Remove commented out code.
  Remove commented out code.
  Remove commented out code.
  ...

better fix for big-endian architectures

Minor indentation fixes.

Cleanup checker shader code.

Size and colors can now be set outside the shader code.