Fix SF#3009431: Prevent Crash in Real Time Tracking GPS Mode & Autodownload Maps.

Protect the GPS layer initiated map drawing update in the same way as the Map layer background download maps draw update works.

Fix display of rounded speed units scale markers in the speed profile.

The integer method only works when the speed unit is in m/s; thus for others show the value to 1 decimal place.

Revert "Make more portable .vik file, as don't save the map cache directory if it's the map cache default directory."

This reverts commit 39aec6c6b87a324981a6e1d028ba345320a444ad.

This commit introduced a behaviour quite confusing: a saved .vik file
cannot be reopened after a change on prefered cache directory.
At least the old way is deterministic.

Merge remote branch 'rnorris/LayerMenuKeyboardAccelerators'

Make more portable .vik file, as don't save the map cache directory if it's the map cache default directory.

The map cache default directory is dependent on the user and OS, however on reading in when it's blank it automatically puts in the map cache default directory.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Remove specific user's map cache directory from test Terraserver.vik file, so that anyone can use it.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Prevent zoom factor string displaying unnecessary .0s when it's a whole number.

Prevent access to undefined data when fgets() returns NULL

If viking.prefs is empty this triggers the following warning at startup in valgrind:

==21725== Conditional jump or move depends on uninitialised value(s)
==21725==    at 0x447354: preferences_load_from_file (preferences.c:147)
==21725==    by 0x447444: a_preferences_get (preferences.c:318)
==21725==    by 0x41019D: a_vik_get_default_lat (globals.c:133)
==21725==    by 0x42B6F3: viewport_init (vikviewport.c:167)
...

This occurs when fgets() returns NULL but the code still tries to
interpret the uninitialized contents of buf.

  Jon

Improve Track Properties speed profile display for low speed difference (e.g. walking) tracks.

The minimum speed difference is far too great so reduce from 20m/s to 5m/s, otherwise the values shown can be very small compared to the overall graph maximum. For tracks with a bigger difference it automatically uses a sensible difference.

Use speed units in display of Track/Waypoint layer draw by velocity config values, but maintain units as metres per second when read from/saved to files.

Fix arithmetic macro definition

cpp macro are only text substitutions.
We have to ensure expansion will do what coder expect.

Add and use defines for conversion of speed units.

Also add and use a feet to meters conversion.

Fix arithmetic macro definition

Add keyboard accelerators for as many as possible View Menu entries.

Remove duplicated line of code in trackpoint drawing.

Fix small memory leak when downloading DEM data via the tool method.

DEM layer properties tidy up.

Remove unused line thickness property (although 'used' in setting up GCs, no actual line drawing takes place so the value is irrelevant).
Change the colour property from a string to a proper GdkGC, so one can use the colour picker to choose the colour. The default is blue.
Use this colour for height's less than then the minimum elevation property (as well as specifically for zero).
Remove the unused single GC.

Add a new debug message

Simplify zoom status when xmpp and ympp are equal

Most of the time xmpp==ympp. So, there is no need to display both factors.

Fix minor source code typo

Fix arithmetic macro definition

cpp macro are only text substitutions.
We have to ensure VIK_METERS_TO_FEET(3+1) will do what coder expect.

Improve grammar/usage for conversion macro function to convert meters into feet. Also prepend these macros with VIK_ as better practice.

Use constant for 'feet in meter'

It's always better to use constant when a value is used more than once.

Use height units in the altitude readout for the cursor position on the status bar.

NB The altitude is only shown for valid DEM layer positions.

Use height units in display of DEM min / max elevation values, but maintain units as metres when read from/saved to files.

Extend set/get param interface to have knowledge of what type of operation it is - i.e. is it related to file i/o or display/cut/copy usage.

This means differing actions can be taken dependent on the usage, such as in determining if conversion operations are required.
Thus when called from file.c the parameter is set to true, and elsewhere it is set to false.

Fix cycle map URL

Uses the OpenCycleMap service
http://www.opencyclemap.org/

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix GType macro usage

Add keyboard accelerator for the Filter extension to the Track/Waypoint layer menu.

Add keyboard accelerators for the Waypoints sublayer menu.

Add keyboard accelerators for the Tracks sublayer menu and improve the text casing.

Add keyboard accelerators for the Track/Waypoints layer menu and improve the text casing.

Add keyboard accelerators for the Maps tools menu and improve the text casing.

Add keyboard accelerators for the Maps layer menu and improve the text casing.

Add keyboard accelerators for the GPS layer menu and improve text casing.

Add keyboard accelerators for the GeoRef layer menu.

Fix compiler warning in file.c

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix compiler warnings in vikgobjectbuilder.c

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix compiler warnings in vikwindow.c

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix compiler warnings in vikmaptype.c

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix compiler warning in download.c

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix compiler warning in degrees_converters.c

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix compiler warnings in babel.c

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Prevent getting stuck in a near infinite loop when using Coord layer, UTM mode and zoomed out to see the whole world.

Restrict internal lat/lon values in coord line drawing so it doesn't go into loops of potentially *massive* iteratations.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix Track Draw by Velocity mode, so that individual track points are coloured (as was the intention).

Now it actually applies the value calculated by the calculate_velocity function to the gc, so it's used in the drawing of the track lines.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Add a message to explain why feature does nothing

Fix previous commit

SHA: 6c83cd7c4ad1b83dd699de1ead995a5a5a053008

Prevent lock up in attempt to download maps along a track in UTM mode.

Method that attempts to get maps between far apart points only appears to work in Mercator (LATLON) mode. Thus only do this when in LATLON mode (i.e. it is skipped when in UTM mode)

Use the standard "Reload" verb

The action is like the action made by browser. And browsers use the
"Reload" verb.

Clarify text on map layer menu

Launchpad Bug #445374: Prevent crash when downloading maps along a track, with Terraserver maps when in Mercator mode.

Simplify code on download action to do when map file is not present

Fix bug where older file was removed when network not present

Fix a minor missing include

It is always really good for a .c file to include its own .h header file.
At least to check that declaration and definition of a function share the
same prototype.

Comment VikSlippyMapSource

Add documentation related to protocol handled by VikSlippyMapSource.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix bashisms.

Change shell to /bin/sh.  Adapt to POSIX shell function syntax.

Add explicit linking against libz and libm

This fixes the following link errors on Fedora 13+

/usr/bin/ld: libviking.a(vikviewport.o): undefined reference to symbol 'tan@@GLIBC_2.2.5'
/usr/bin/ld: note: 'tan@@GLIBC_2.2.5' is defined in DSO /lib64/libm.so.6 so try adding it to the linker command line

/usr/bin/ld: libviking.a(dem.o): undefined reference to symbol 'inflateInit2_'
/usr/bin/ld: note: 'inflateInit2_' is defined in DSO /lib64/libz.so.1 so try adding it to the linker command line

http://fedoraproject.org/w/index.php?title=UnderstandingDSOLinkChange

Fix: warning 'draw-mode' message does not display

I don't know why, but the warning message about a wrong draw mode
does not display. Gtk claimed:
Gtk-CRITICAL **: gtk_widget_get_toplevel: assertion `GTK_IS_WIDGET (widget)' failed

I imagine something is not completly initialised in VikMapsLayer object.
Using VikViewport is just fine and more secure: we know that viewport
already exists.

Fix incorrect limitation on source map id

There is no special reason why id of map source should be stored on 8 bits.

Add warning when requested GObject type is unknown

[TEST] Add Terraserver viking file

Prevent viewport tool selection/edit of trackpoint or waypoint when the layer or sublayer is invisible.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Releasing viking 0.9.95

Merge branch 'i18n-launchpad'

Restore as many translations as possible

Document previous translation updates

Import Launchpad updates

Invalid number of arguments in call to a_http_download_get_u

While trying to build 0.9.91 to 0.9.94 on Gentoo, I received the above error in reference to a call in expedia.c ~line 188. Simple patch calling last argument as null. Below patch is for 0.9.94.

Prevent crash when cut of a GPS track/waypoint layer is attempted via the layers buttons.

c.f. with method used in vik_layers_panel_delete_selected function

Prevent DEM layer colour indexing going negative and can crash on gradient view.

Add some function short cut key definitions to control showing of the center mark and scale viewport features.

NB F8 is already used internally - one can select/unselect the layers/viewport split and use the arrow keys to move the split left/right.

Fix DEM layer to show default sea level.

Allow layer properties dialog to show (and thus use) the intended default miminum elevation.

[DOC] Minor update.

[DOC] Replace Google with OSM capability in man pages.

When manually creating a track, automatically give it a default name.

Now one can just click OK, and get on with creating the track (without needing to type anything).
Of course, one can change the suggested name or rename the track after creation via the layers panel features.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix track background thickness being able to set back to zero.

The default value is zero, and previously if one changed it; it could then not be set back to zero.

Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

[DOC] Tidy Up: First Pass

[DOC] Add contributors from the Wiki(s) history.

Fix internal tile locking

g_list_find() only compares the pointer and not the filename it points
to, use g_list_find_custom() with g_strcmp0() to actually check for the
filename.

Signed-off-by: Sven Wegener <sven.wegener@stealer.net>
Signed-off-by: Guilhem Bonnefille <guilhem.bonnefille@gmail.com>

Fix debug message

Fix top layer not drawn after made invisible and then back to being visible.

Investigated a several of different options of drawing controls (including removing the ability to control the top layers visibility!), but simplest (and most effective IHMO) is just to force any aggregate change to redraw all.

Fix impaired (drawcentermark setting bodged) vik file if saved with the top layer visibility turned off.

Fix syntax errors in help manual

Fix map drag and drop to show the correct whole map

Fix "Go to UTM" dialog title.

Fix first opening of a file which has spaces (or other interesting characters) via the recent files menu.

Previously attempted to operate on an URI, which can fail. Now operates on the local/parsed filename.

Better dialog name when searching for a waypoint.

Improvements for test programs.

Explain how to build.
Extend git ignore for test directory build output.
Improve gpx2gpx to run without the need of a $DISPLAY and to free memory.
Fix compiler warnings.
Add returns for all main programs.

Fix unnamed_tracks / unnamed_waypoints misuse.

Prevent "Gtk-WARNING **: GtkSpinButton: setting an adjustment with non-zero page size is deprecated" messages.

Change max cache time to 1 week.

When caching tiles, it's reasonable for the user to be able to make a
tradeoff as to how long a cache lifetime is reasonable.  Even with OSM
mapnik, it's a reasonable choice to accept tiles that are a week old
rather than incurring network latency.  Arguably terraserver tiles (or
any other source that is not expected to change) should be able to be
set to months.

Releasing viking 0.9.94

Fix POTFILES with new source file

Merge branch 'i18n-launchpad'

Document previous translation updates

Import Launchpad updates

Fix OMF category

Fix Filter->Simplify All Tracks default.

Add knots as a speed unit option.

For all those sailors out there!

[DOC] Fix Gnome's menu entry category for viking