Merge branch 'TrackMergeSplitImprovements'

Add some protection against crazy elevation numbers that cause viewing a track's properties to crash

Detail on how to regenerate reference help

[I8N] Mark vikaggregatelayer.c for translation

I8N added in SHA1:affcc0f201d869d534f305e2b955e2afcc2975fe

[I8N] Mark file vikwaypoint.c for translation.

I8N required from SHA1:c9570f8616c6eec62695f29757cd6913efadebdd

Fix memory leak detected by cppcheck 1.56

Tidy ups to keep cppcheck 1.56 happy.

Prevent obscure crash on key press when layer is not selected but create track tool is enabled

Ensure secondary key press processing only occurs for window level tools
Fixup commit SHA1: a71145218059e3e8153b9e58493d3a0d57743324

Never use g_error() as it aborts the program. Use g_critical() instead.

The program can still function despite such issues. Carry On Regardless.

Add python tool to convert a Viking tile cache into a MapBox MBTiles format file.

See: http://mapbox.com/mbtiles-spec/

This tool is heavily inspired from http://github.com/mapbox/mbutil

It is particularly useful for transferring the .mbtile file for viewing the predownloaded set of tiles on a smartphone device.

Rework a_file_load() to always load .vik files in preference to other kinds and other clean ups.

Add more comments about how loading is handled and store the value to return so that tidy up statements occur only once.

Suggest default filename with .vik file extension when saving a new file.

Revert Stonehenge.jpg back to the previous version *without* GPS information.

The idea here is this file doesn't have a location in the EXIF info - but it's well known location.
Thus useful for testing geotagging functionality with the Stonehenge.gpx file.
Accidently committed it in SHA1:f2fcc2d2bb4ff6930eb15651dda5e45b2bc49956, restore to back to SHA1:60a695608872f7e733b8bc030426e62f1e42def6

Use the signalling method to the update statusbar from OSM traces background thread.

ATM this introduces a small memory leak, note OSM traces isn't heavily used so this is not significant.
This is better than potentially crashing from multi thread GUI updates.

Allow statusbar update signal method to specify which part of the statusbar to update.

Move window code that is unnecessarily in main.c to vikwindow.c

Much better scoping of the window code. Minor rename of functions and update header appropriately.

Fix crashing due to GUI updates from the background thread on GPS Upload.

Mistake in commit: 10aa2b84d0a30270ee618057f6d67665177d3fc9
Ensure vik_trw_layer_uniquify() happens in main thread.

Fix CPU usage going to 100% when statusbar items update is called.

Ensure proper function defintion and hence return the right value so the function is *not* continually called.
Thanks for nothing compiler/C language!

git ignore the generated PDF manual.

Releasing version 1.3.2

Fix 'make distcheck' to have a clean run.

Need to ignore icon cache file.

Prevent compiler warning.

Follow up to commit: 8ad7a81eb73130da7324036477e2d9f4c1d13cca

Minor code tidy up to match staticness of function definitions.

Fix non slippy maps from complaining about lack of function definition.

Follow on from commit:2673b29d1b6f58ed54cad164bab5d325960bb8c6
Best to define a function to return a value (even if it is always false) for TMS/WMS/Terraserver Maps.

Remove Direction Finding from the Help as direction finding via Google services has been disabled.

Disable direction finding via Google services as they no longer support KML output.

From https://developers.google.com/maps/documentation/directions/ only JSON and some kind of XML
GPSBabel doesn't support GeoJSON and Google Maps XML (google) doesn't seem to work any more either
[http://www.gpsbabel.org/htmldoc-development/fmt_google.html]

Fix usage of vvp in gpslayer in uploading TRW layer data.

Follow on from commit: 10aa2b84d0a30270ee618057f6d67665177d3fc9

Ignore built viking.exe file under Windows.

Fix DEM icon usage following internal rename.

Name changed in commit: 79dce0cbecc73aef904c1529467195416c1bfe96

git ignore output from make dist

git ignore files generated in the data directory

Remove old and unused src/.depend file

Reduce warnings when compiling on Windows

Update ‘gdk_pixbuf_ref’ deprecated functions to the g_object_ref equivalents

Ensure correct selection of track or waypoint following name usage rework.

Follow on from commit: 3d2ad4f8f73e4fad30df83199a83d6e947a88786

Remove usage of selected item name property in the window level as it is now unnecessary.

Add a background block to the track drawing distance to make the text easier to read.

Use gdk_draw_layout() rather than gdk_draw_string() in track distance drawing

Fix flickering map background when creating a new track and panning the map with the mouse.

Due to the new way of drawing the next potential trackpoint, the pixmap held becomes invalid when the display is panned.
Thus a way of preventing this update is made by exposing the middle button clicks to the tools when they explicitly request it.
Hence the potential trackpoint drawing is temporarily disabled when the map is panning.

Rework the create track drawing with a specific GC on which to draw the next potential point.

This means this GC is reset on each update and then redrawn. It also ensures the correct colour is used.
This is more reliable than trying to undraw previous information via the GDK_INVERT method, especially for Windows systems.

Fix for Debian#685817: Statusbar Update From Thread Instability

Move the direct statusbar GTK update from the background thread into a signal to the main thread.
The statusbar update in the main thread is then performed according to GTK scheduler via the idle update mechanism.

Fix openwindow internal type to match actual usage.

[Geotagging] Improve detection of image files having GPS information.

Encountered some images which have just the EXIF_TAG_GPS_VERSION_ID but nothing else.
So to confirm GPS info, check existance of more EXIF GPS tags (Latitude and Longitude)

Thus when such images are requested for geotagging they won't be erroneously skipped over.

Fix GPSBabel Dynamic Device List generation on Windows Systems.

Rework babel_general_convert() to be cross platform and support the progress callback on Windows systems.

Minimize the platform specific code by using g_child_watch_add() for cleanup operations and fdopen() or _fdopen() to access the stdout stream.
The previous Windows version of the function has been removed and now all systems use the same babel_general_convert().

Hence with support of the progress callback, the means the Acquire GPS Device list and Import File With GPSBabel file types will be dynamically generated on Windows.

Add zoom by bounds capability when using the zoom tool and holding the SHIFT key and left click to select the zoom in area.

Similar to zooming in on the OpenStreetMap default map.

In the zoom tool mode, when the SHIFT key is held down and then left mouse button is pressed it draws a bounding box area on the display.
When the left mouse button is released the zoom is performed on the selected area.

Enable zoom tool to recenter the display on the clicked position when the CTRL key is held at the same time.

More intuitive zoom actions involving the mouse to retain the location under the pointer within the viewport bounds.

As per request SF#3547438, which is clearly much better behaviour - since clearly this is the point of interest the user wants to see and thus subsequent zoom requests keep focusing on that area without having to recenter the display.

The modifiers of CTRL+SHIFT and mouse zoom retain keeping the center point in the center (which is in fact a reversal of the previous setting - unlikely to effect many users, as I didn't even know of this feature myself).

Note the keyboard & menu zoom requests always maintain the center point.

[Geotagging] Update RPM spec for new dependency.

Improve feedback for handling erroneous GPX.

Extend current mechanism for reporting specific file load errors by adding a message for GPX read failures.

[Geotagging] Fix SF#3564795: Crashing when attempting to create waypoints from EXIF images without enough GPS information.

Don't try and use the returned NULL waypoint!

Prevent crashing when attempting to import with GPSBabel a file and not specifying the file type (due to a malformed (null) result of the GPSBabel file types list).

Prevent crashing when attempting to acquire data with a malformed (null) result of the known GPS devices list.

Fix lack of feedback when reading erroneous GPX

When GPX file is incorrect, no error is returned. The user is not
informed that his file is incorrect and can simply imagine the file
is empty.

The status of the XML parser is now returned in order to fire the
right handling.

[MAPS] Add OpenSeaMap source

Closes #SF3555603

Fix usage of directory "doc/reference" missing even it is used in configure.ac. Automake hangs here if called with autoreconf.

Also the configure option --enable-gtk-doc does not work due to this missing directory.

Solve this issue by using AM_CONDITIONAL() to enable the "gtk-doc"-compile feature properly.

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Clean gnome-doc-utils.make in target "dist-clean"

According to common autotools rules this should be cleaned in maintainer mode only.

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Further tab fixes in Makefiles

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Remove doubled defined "XP" in src/Makefile.am (conplained by automake)

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Fix the missing tabulator

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Fix an incompatible make file rule (using "%"); this is complained by automake.

The broken rule is substituted by proper new one.

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Enhance the "make dist" target wrapping the package also in a tar.bz2 as well as a zip package

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Ensure the pan toolbar button is activated by default to match the default tool mode.

Enable merging segments of a track from the Track->Combine submenu.

Add function to merge segments of a track.

Fix implicit ordering of Trackpoint property dialog buttons.

Change from GList to individual widgets, this means the sensitivity of the desired button is handled correctly.

Tidy up: simpler and better use of trackpoint free methods in TRW Layer.

Fix potential loss of track segment in vik_track_remove_dup_points()

Also use better trackpoint free method.

Enable deletion of track points with the same timestamp from the Track -> Delete Points submenu.

Add functions to count and delete track points with the same timestamp.

Add new 'Delete Points' track submenu and allow deleting duplicate track points.

Update vik_track_remove_dup_points() to return the number of points deleted

Add Track Split by Segments into Track menu options.

Consistent use of unique_sublayer_name function when creating new tracks via all split operations.

Internal reorganization of unique_sublayer_name function for reuse

Make splitting a track consistant in that it never asks the user the name of the new tracks.

. Avoid this extra step as it can be unnecessary. Changing the track names can be performed later if necessary.
. Multiple same named tracks are now allowed anyway

Remove joining tracks via the Track Point dialog as it was a 'weird hack'.

. Only worked when dialog opened via Track Edit tool (i.e. not via Select tool)
. Unclear for the user how it is meant to work
. Superseded by the Append method via the track menu option and dialog to select the other track

This cleans the code up somewhat.

Move View Track menu option above submenu options.

Add new append track command to join tracks not caring about the whether they have timestamps.

Hence allows joining 'routes' and tracks. Only one track can be appended at a time to control ordering of the append.

Move track merge options into a new submenu called 'Combine'

Add track split at trackpoint as an option to the menus.

Make it available only when a trackpoint is selected.

Rework track split at trackpoint into a function for later reuse

Move split track menu options into a new submenu

Fix merge track by times for using the internal track id properly.

Follow on from commit: 3d2ad4f8f73e4fad30df83199a83d6e947a88786

Generally rewrite it to make it more understandable and remove some implicit behaviour.

Fix display of track edit dialog values when invoked via Track Edit dialog.

Broken by track naming rework commit: ce4bd1cfaab763dad3aab9b20edce22eb57aac0f

Fix split track at trackpoint for using the internal track id properly.

Follow on from commt: 3d2ad4f8f73e4fad30df83199a83d6e947a88786

Fix the untranslated "top layer" context menu.
Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Fix radio button labels not translated in dialog windows.

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Add and use keyboard accelerator shortcuts to be able to create the most likely used layers.

Add keyboard accelerator marker for layers used in menus.

[I18N] Ensure all layer names are marked for translation

Rework Tool interface for give better control for Tool menu labels and tooltips.

Add some keyboard shortcuts to the help.

Fix header as vik_trw_layer_get_waypoint is required again.

Rework Track Draw by Velocity into Draw by Speed with a simple traffic light colouring scheme.

Determine the colour of the trackpoint (and/or trackline) relative to the average speed
 *  . slow points are red
 *  . average is yellow
 *  . fast points are green

An option is provided to control the percentage factor away from the average speed to determine the transistions to the differing colours.

This now removes the previous Draw by Velocity mode options, which was unclear how it worked and unlikely any one actively uses it.
Thus this means the .vik file format has changed slightly but other than generating warnings all other features will still work.

For tooltips use plain text so that comments with HTML markup or similar are shown in a literal fashion rather than causing parsing failures.

Add simple direct on disk access map type to tiles ideally pregenerated in the OSM tile layout format.

Geotagging: add option to overwrite existing waypoints.

This is especially useful when after the first attempt, the waypoints were generated are in the wrong position and you want to repeat the tagging process but with a different time offset. Such as having daylight saving time issues (e.g. GMT v BST)

Fixup making the track name a property of the track.

Missed geotag usage in commit ce4bd1cfaab763dad3aab9b20edce22eb57aac0f

Windows: Really fix opening pictures

Signed-off-by: Rob Norris <rw_norris@hotmail.com>

Remove create_realtime_trackpoint call in cleanup which can crash.

Crashes in g_list_last on my system, although I don't understand why.
However what is the point of creating another trackpoint when stopping, especially since the track is immediately deleted!?
Hence the call has been removed.

Fix crashing when deleting a GPS layer which has started a realtime tracking.

Ensure real time disconnect method is closed to cleanup and stop IO events.