Document previous translation updates

Import Launchpad translation updates

Include HACKING file in distribution.

Fix Georef mpp from coords.

Conversion from the image mpp to Viking zoom level is dependent on the latitude when in LATLON mode.

Make OSM services the default by including these modules first.

Simplify filter actually uses the unsigned integer value.

Fix Google maps positional web link.

Format seems to have changed recently.

[QA] Fix issue spotted with cppcheck 1.70

Remove unused northernHemisphere variable

[QA] Output console message if g_mkdir_with_parents() is unsuccessful.

CID#34603: Protection in copying into fixed size buffer

CID#34584+CID#34583: Prevent deference after null check

CID#34611+CID#34611: Prevent use of uninitialized scalar variables

Handle potentially unknown horiz_units, although this shouldn't normally happen

[QA] CID#34576: Remove logically dead code

[QA] CID#34575: Remove logically dead code.

Items are always considered for rename when moved.
The unique name can be straight copy of the original.

[QA] CID#34543: Adjust curl_get_etag_func to use pointer to structure rather than the raw address.

CID#131392: Avoid potential null pointer dereference

CID#34574: Fix freeing the appropriate variable.

[QA] Generally print warnings to the console for low level file operations
(since they don't normally go wrong)

CID#34548
CID#34549
CID#34550
CID#34552
CID#34558
CID#34560
CID#34562
CID#34564
CID#34568
CID#34569
CID#34571
CID#34572
CID#34573

[QA] CID#34547: Path value is checked so not worried about function return value.

[QA] Coverity: Unchecked return value from library

Explicitly ignore return values - when removing temporary generate files

CID#131383
CID#131382
CID#131381
CID#34570
CID#34563
CID#34561
CID#34557
CID#34555
CID#34546

Fix track tooltips minutes value should be rounded.

Thus need to ensure in floating point otherwise the minutes value gets truncated.

CID#101113: Result not floating point

Minutes value should be rounded, thus need to pass in as floating point.

[QA] CID#101114+CID#101115: Result not floating point

No need for round since the truncated integer division value is needed for the hours value.

[QA] CID#131384: Check function return value.

[QA] CID#131386: Check function result

[QA] CID#131385+CID#131387: Be more explicit that we don't care about the result of this function call.

[QA] CID#131393: Missing break in switch

[QA] CID#131394: Check result of g_fopen()

Add warnings if the LatLonTZ file can not be accessed or opened.

[QA] CID#131401+CID131400 - Improve printf arg types.

[QA] CID#131402: Fix resource leak in handling metatile errors.

Fix creating waypoints from JPGs when using gexiv2

SF Bugs#126: Fix crash in gdk_rgb_convert_0888 due to using deallocated memory.

There is reference counter in cached pixel buffers.
Previously it always equalled one although multiple tasks could be using it.
Thus when the cache is flushed, memory was always deallocated but then another task would attempt to use it and crash.

Thus now the cached pixel buffer is tracked properly with unref() after being used,
 so only when reference count is zero it is automatically deallocated.

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

Disable Search and Routing with Google as they no longer support the APIs Viking used.

The only supported file formats seem to be JSON and XML in their own custom schemas (and not currently supported by GPSBabel).
Also now you need to supply an application key for all Google APIs.

Fix availability of Geotag consistency checks when using gexiv2

Remove --with-search configure options since they are not used.

They haven't been used since the search location web service engine was made selectable at run time back in 2010.

Improve place search failure message to say if it is a service level issue.

Expand vik_goto_tool_get_coord() to flow up if there is an error communicating with the web search provider.

Protect against possible out of bounds array access.

[QA] Coverity 'Uninitialized pointer read' CID#34608 + CID#34609.

Force initialise to NULL values, even if these variables get assigned from the function calls.

[QA] Coverity 'Buffer not null terminated' issues CID#34544 + CID#34545

Switch to use a function that ensures the buffer is null terminated.

Note that in these cases it is using a time format string that would always be less than the buffer size anyway.

Fix DEM height points on track graphs when height units is in feet and non-zero minimum height.

Offset is already in the current height units,
 so elevation value should be adjusted *after* it is in the appropriate units.

[DOC] Improved introduction.

Only perform man page generation when scrollkeeper is also installed.

[QA] Remove repeated statements in configure.ac

Fix mapnik preferences initialisation to not crash if preferences are not available.

Enable reference documentation generation by disabling optional components.

Otherwise at the linking stage of the scan program, it tries to find terraserver related objects;
 which are no longer in the default build.

Reactivate doc subtree

Previously, in order to build the reference documentation only when needed
(read: only some maintainers and only when they want) a solution was created
to deactivate processing of the whole doc subtree. But the side effect was to
not deliver at all the doc subtree if the correct options are not activated.
With an incomplete delivery, some tools like autoreconf (used by packaging
utilities) fail, bringing packager to patch and patch again the sources.

With this change, the doc subtree is no more ignored and the reference
documentation is processed ony when the corresponding ./configure flags is
activated.

Ensure thumbnails are created when opening JPG files.

Insert parameter for Viking's version in viking.xml

Insertion is done by configure, so the file is renamed as viking.xml.in.
The most annoying with this change is probably to configure the XML editor
to handle xml.in files.

Fix 2 heap-buffer-overflows in trw_layer_download_map_along_track_cb()

1 byte is less than sizeof(gpointer), so we do not have enough memory
for last NULL pointer.

This could be solved by braces, but since glib provides g_malloc_n(),
and this kind of bug can be avoided with it, so.

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

Fix heap-buffer-overflow on Layers -> New GPS Layer

Very typical issue, unfortunately, missing braces and we are
corrupting memory. It is better to use g_malloc_n() to avoid this.

Fixes following AddressSanitizer error:
==17766==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x60b0008db6a0 at pc 0x00000048ba9e bp 0x7ffcf3e532a0 sp 0x7ffcf3e53298
WRITE of size 8 at 0x60b0008db6a0 thread T0
    0 0x48ba9d in gps_layer_inst_init /home/lebedevri/src/viking/src/vikgpslayer.c:386
    1 0x7fa95d549f98 in g_type_create_instance (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x31f98)
    2 0x7fa95d52d866  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x15866)
    3 0x7fa95d52f25c in g_object_newv (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x1725c)
    4 0x7fa95d52f9f3 in g_object_new (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x179f3)
    5 0x48dca7 in vik_gps_layer_new /home/lebedevri/src/viking/src/vikgpslayer.c:659
    6 0x48e1f0 in vik_gps_layer_create /home/lebedevri/src/viking/src/vikgpslayer.c:419
    7 0x45d119 in vik_layer_create /home/lebedevri/src/viking/src/viklayer.c:229
    8 0x460463 in vik_layers_panel_new_layer /home/lebedevri/src/viking/src/viklayerspanel.c:461
    9 0x46982e in menu_addlayer_cb /home/lebedevri/src/viking/src/vikwindow.c:2411
    10 0x7fa95d5282d4 in g_closure_invoke (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x102d4)
    11 0x7fa95d53a03b  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2203b)
    12 0x7fa95d542697 in g_signal_emit_valist (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a697)
    13 0x7fa95d5428fe in g_signal_emit (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a8fe)
    14 0x7fa95f0ec8cf  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x748cf)
    15 0x7fa95d528503  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x10503)
    16 0x7fa95d541fa6 in g_signal_emit_valist (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x29fa6)
    17 0x7fa95d5428fe in g_signal_emit (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a8fe)
    18 0x7fa95f2c0c55 in gtk_widget_activate (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x248c55)
    19 0x7fa95f1bc98c in gtk_menu_shell_activate_item (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x14498c)
    20 0x7fa95f1bcd2a  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x144d2a)
    21 0x7fa95f1aaa7e  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x132a7e)
    22 0x7fa95d5282d4 in g_closure_invoke (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x102d4)
    23 0x7fa95d539f31  (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x21f31)
    24 0x7fa95d5421a4 in g_signal_emit_valist (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a1a4)
    25 0x7fa95d5428fe in g_signal_emit (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x2a8fe)
    26 0x7fa95f2c1ecb  (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x249ecb)
    27 0x7fa95f1a91c3 in gtk_propagate_event (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x1311c3)
    28 0x7fa95f1a965a in gtk_main_do_event (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x13165a)
    29 0x7fa95ee1abbb  (/usr/lib/x86_64-linux-gnu/libgdk-x11-2.0.so.0+0x5abbb)
    30 0x7fa95d251c3c in g_main_context_dispatch (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x49c3c)
    31 0x7fa95d251f1f  (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x49f1f)
    32 0x7fa95d252241 in g_main_loop_run (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x4a241)
    33 0x7fa95f1a85d6 in gtk_main (/usr/lib/x86_64-linux-gnu/libgtk-x11-2.0.so.0+0x1305d6)
    34 0x41829c in main /home/lebedevri/src/viking/src/main.c:259
    35 0x7fa959a69b44 in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x21b44)
    36 0x417b78  (/home/lebedevri/src/viking/src/viking+0x417b78)

0x60b0008db6a1 is located 0 bytes to the right of 97-byte region [0x60b0008db640,0x60b0008db6a1)
allocated by thread T0 here:
    0 0x7fa95f96437a in malloc (/usr/lib/x86_64-linux-gnu/libasan.so.2+0x9437a)
    1 0x7fa95d257799 in g_malloc (/lib/x86_64-linux-gnu/libglib-2.0.so.0+0x4f799)
    2 0x7fa95d549f98 in g_type_create_instance (/usr/lib/x86_64-linux-gnu/libgobject-2.0.so.0+0x31f98)

SUMMARY: AddressSanitizer: heap-buffer-overflow /home/lebedevri/src/viking/src/vikgpslayer.c:386 gps_layer_inst_init
Shadow bytes around the buggy address:
..
  0x0c16801136c0: fa fa fa fa fa fa fa fa 00 00 00 00 00 00 00 00
=>0x0c16801136d0: 00 00 00 00[01]fa fa fa fa fa fa fa fa fa 00 00
..
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
==17766==ABORTING

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

SF Bugs#121: Additional race condition crash protection.

Subsequent code where variables could point to invalid locations.

Note this is called manually, so much less likely to encounter the race condition.

SF Bugs#121: Fix crash when invalidating previously acquired tiles.

This race condition can most often be triggered by zooming in/out rapidly.

The fault is due to using accessing variables outside of the lock section of the memory cache.
This has been in the code since the initial lock adding commit [in 2008]
 (SHA:9b79169dba7cd4df6badebdda23a370064ebfafd)

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

QA: extract effective code from g_assert

Typos spotted by coverity: https://scan.coverity.com/projects/viking

DOC: Fix typo

DOC: fixed typo

Fixed typo on the bzip2 option's name in configure

To fix this error, there were two solutions:
- change the name of the option
- change the description
I decided to change the name of the option.

[QA] Extra info on magic usage.

[QA] Extra info when using bzip2.

SF BugsZZ#123: Fix bzip2 decompression on Windows.

Need to specify opening as a binary file otherwise BZ2 read fails on Windows.

Also remember to cleanup resources - particularly closing the file.
Otherwise on Windows the subsequent remove + rename file operations fail as the file is still open.

Fix Geocaches acquiring with latest geo-* software

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

Mapnik3 support.

C++ standard required is now C++11.
New location for plugins.
Revised code for accessing Mapnik parameters and some types/functions have been renamed.

More correct menu icon

Don't attempt download if temporary file can't be opened.

[QA] Fix spelling in a comment and add function comment

Ensure layer visibility is retained when copying layers.

SF Bugs#121: Provisional fix

Protection against storing corrupt pixbufs into the mapcache may help prevent crashes seen in #121.

[WINDOWS] Fix reading zip files on Windows due to compiler alignment defaults changing.

Read zip files contents which are stored with no compression.

[QA] Protection from large malloc failures

Fix logic for comparing timestamps

[QA] Use correct type in sizeof function for memory allocation.

Don't retain GTK Dialogs in memory.

Not entirely sure why these were created and then shown/hidden as it doesn't save much time compared to simply creating them on demand.

Latest versions of GTK Dialog are more complicated and seemingly respawns a thread every few seconds (presumably to monitor if files have been created/deleted), irrespective whether the dialog is shown or not.

Thus now destroying it we avoid this unnecessary thread (re)creation overhead.

Re-fix SF Features #82. Simplify usage of restoring the last used directory.

[QA] Don't need copy this string.

[QA] Use map ids

SF Bugs#122: Fix memory cache confusion with multiple "On Disk OSM Tile Format" maps

Ensure use of the correct map id comparison so the string to enable differing cache lookups is set.

Prevent Gtk-Critical warnings after viewing the Track property dialog.

Support all values of GPS Mode fixType.

[QA] Fix typo in (an unused) macro definition.

keep alpha channel on mapnik rendering, scale alpha to alpha of mapnik layer

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

SF Bugs#120: Fix Track drawing bug across the 180th Meridian.

Red Hat Bugzilla – Bug 1210403: Only download and process one Bing attribution list.

If making more than one download request to get the Bing attributions,
 but then updating (and using) a single instance variable from multiple threads,
 results in undefined behaviour (i.e. could lead to crashes).

Thus ensure only one request at a time is made.

Simplify debug output of loaded Bing attributions.

Don't need to print out every stage of the formulation of a single attribute structure.

[QA] Make function names more unique to simplify use of debugger breakpoints.

Make simple GPSBabel filter options use the updated acquire framework options.

These filter options don't have to be shell commands anymore and so should work cross platform.

Rework Acquire framework process function options for maintainability and understanding.

Instead of overloaded parameters, use a structure with fields for the specific processing types and gpsbabel options.

Thus rename functions and parameter variables, update the usage thereof and simplify code as applicable.

Fix stability of some 32 bit builds.

Forcing 64bit types seems to improve reliability of these timestamp operations.

Fix tracks and routes not displayed when downloaded via the GPS Layer.

Need to calculate the bounding box of these, which is performed in the post read function.

Fix availability of the Google routing direction engine

Prevent crashing on direction requests when no engine is available.

[QA] Rename routing engine function as it gives URLs

Fix display of map tile source information when it has extended characters.

Since the dialog text uses Pango markup, need to escape the input string.

[WINDOWS] Remove usage of 'filter with' functions as these don't work in Windows.

Since these functions are bash shell commands they only work on Unix like systems.

[WINDOWS] Fix a missing \ in a potential gpsbabel location.

Update Help to wiki conversion method as the wiki is now in Markdown.

datasources.xml file should be packaged

Fix warning in a test program.

[DOC] Update copyright year

Fix potential broken builds.

Annoyingly g_str_to_ascii() is actually a recent addition to GLIB,
 despite the manual not stating which version it is available in
 (hence I thought it was always available).