Add license notice to new file

GPL license validated with Rob Noris by email.

Replace macros by enum

An enum informs the compiler about the idea of related values.

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

Simplify previous commit

I feel that this new change helps to keep code as simple as possible.

There's no need to give VikTrack in argument as VikTrack can be
retrieve from VikTrwLayer with trackname.
Furthermore, deciding to export track as GPX without new file type
value.

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

Enable individual track to GPX export via the Track sublayer menu.

Internally gpx.[ch] supports writing a track to the specified file, so make this available in the GUI.
Expand export definitions in file.h, use the gpx.h interface as necessary routing via a_file_export function.
Filename used is automatically generated from the track name with a '.gpx' appended if appropriate.
Restructure trw_layer_export function to support passing track information.
Add the new sublayer menu entry to use this feature.

Set 'OK' default for autogenerated dialogs.

Set 'OK' default for find waypoint dialog.

Set 'OK' default for download along track dialog.

Set 'OK' defaults for new track, custom zoom & split time dialogs.

Set 'Cancel' default for merge track dialog. User needs to select items first.

Set 'OK' default for the new waypoint dialog.

Set 'OK' default for goto dialogs.

Set 'OK' default for the Save to Image dialog.

Set 'OK' default for the track properties dialog.

Set 'OK' default for the GPS layer dialog.

Set 'OK' defaults for goto dialogs.

Set 'Cancel' default for the Geo Ref dialog, as user should really specify a map file first.

Set 'Cancel' default for the OSM upload dialog.

Set defaults for geonamesearch dialogs.

Set 'OK' default for the background jobs dialog.

Set 'OK' defaults for acquire dialogs.

Add a perl script to auto generate basic Viking .vik files for directories containing images.

Simply recursively search down the directory tree (from the current location) for suitable image files
 [normally jpg|JPG (probably photographs)] and then extract any location data from the EXIF part.

For each directory this info is output to a file into either Viking (default) or GPX data file formats.
Output filename is waypoints.vik (or waypoints.gpx in GPX mode) unless the -o option is specified.

Enable control of the visibility of the menubar [including keyboard shortcut - F4].
Also enable panic key 'Escape' to restore menubar if hidden and no tool uses it.

Enable control of the visibility of the toolbar [including keyboard shortcut - F3].

Make use of the vikwindow toolbar variable.

Enable control of the visibility of the statusbar [including keyboard shortcut - F12].

Releasing viking 1.0

Releasing viking 1.0.

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

Remove duplicates msgid

This seems to disturb Launchpad.

Merge branch 'i18n-launchpad' into master

Auto-update PO translation files

Document previous translation updates

Import Launchpad updates

Add minimal translator-credits in About dialog

Use $(top_srcdir) to locate AUTHORS file

Seems more conventional.

Add documenters to About dialog

Documenters list is extracted from help. The extraction if made with
xsltproc. If this tool is not available, the documenters list is generated
empty.

Add copyright notice to maintainer's scripts

Improve copyright holders

I tried to explore Git in order to identify copyright related issues.
The result is quite incomplete or incoherent: the changes I made does
not use strict rules. So, I hope I did not irritate anybody.

If you think your name should be listed somewhere, or should be removed,
send the patch.

Add copyright to srtm_continent.c

Add origin comment

Add copyright for geonamessearch.h and correct for geonamessearch.c

git log confirms original author as Hein Ragas.
Thus also remove incorrect Created by statement.

Add copyright for files vikgoto.h & vikgotoxmltool.h

git log confirms original author as Guilhem Bonnefille
Thus also amend the corresponding .c files to remove incorrect Created by statement.

Fix copyright for googlesearch.[ch]

Add copyright for files preferences.c & preferences.h

git blame confirms original author as Evan Battaglia
Statement used is standard boilerplate with suitable date.

Add copyright for dems.c & dems.h

git blame confirms original author is Evan Battaglia.
Use copyright as in related files dem.c & dem.h.

Add copyright for datasources.h.

git blame confirms original author is Evan Battaglia.
Used copyright from similar file datasource_gps.c

Correct DocBook usage

The procedure element is used for enumerating different steps in a procedure.
Simple enumeration is done with itemizedlist element.

Various improvements and tidy ups.

Improve concepts:
. Add what is a Layer
. Mention Statusbar
Better use of 'appname'
Move verbose/debug mode to bottom + add example for a map download
General GUI reference improvements.
Add Begin Track Icon.
Other misc bits as I went along...

Deprecated: only include gtk/gtk.h

Improve depreaction options

* Add options
* Add a cached value

Add built-in map sources in example file

Fix extend track using magic scissors.

Should only need to click once to select the next point to extend the current track, hence need to mark that the first point is known (via the 'magic scissors started' variable).

Merge branch 'master' into i18n-launchpad

Document previous translation updates

Import Launchpad updates

Fix autodownloading while panning

Since commit 1c6a6010 ("Disable autodownloading when dragging the map")
we only trigger an autodownload when starting to pan and not on stop,
which is pretty bad. Also pan_move is never reset, resulting in no
update when panning with keyboard shortcuts after panning with the
mouse.

Signed-off-by: Sven Wegener <sven.wegener@stealer.net>

Prevent crashes when downloading Expedia Maps.

Fix expedia code to use updated vikmapslayer_compat.h, which was modified for:
'Allow reuse of curl connection objects'
SHA: 825413bac81e7234ed27a8ff3343a8295cc393e2

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

Fix Debian #599048: Segmentation fault when using gpsbabel

Ensure cmd_off & extra_off are initialized before use.

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

Fix dem_gradient_colors allocation.

A comma was missing - giving an unparsable colour.

Fix commit 1c6a6010661da96fd105ec42ad2f048df79fabbd

I simply did test in the wrong way.
I added documentation.

Fix compiler warnings in preferences.c

Fix: enforce defensive code

Releasing viking 0.9.96

Fix unecessary change in string

Simply play with length.
Furthermore, this character belongs to a string owned
by other part of software.

Free memory

Add defensive code

Increase buffer length

I think 50 is too small as etag contains 36 characters
and header contains 15 characters.

Fix memcheck error

What I think happens is:
1) We first do a request for a tile with an ETag and apply a custom
header, this gets set into the conn->data->set.headers pointer
2) The header gets freed, but the set.headers pointer is left as a
dangling reference to the memory
3) A subsequent request is generated for a tile without an etag so we do
not overwrite the set.headers pointer and it keeps the old, invalid
value and the HTTP request code tries to reference it.

I believe the associated change should fix it by ensuring the dangling
pointer gets cleared during step (2).

Merge remote branch 'jocelyn/etag'

Disable autodownloading when dragging the map

I recently discovered that a recent commit introduce a significant
change on CPU usage.
The commit is 71eff775d86be02173e28421cea7f7d3f5a8344. It offers a
smooth pan when drag&drop, but it introduces a huge amount of tile
requests.

For exemple, when I drag the map, I quickly observe around 600
requests on local tile cache while around 10 requests would be enough.

The matter is, in order to keep a smooth dragging, the viewport is
requested to redraw completly. While doing this, all displayed maps
need to check their local cache. And this, for each X event.

This fix is a temporary one.
A better solution implies certainly a significant refactoring
of display/download.

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

Fix: Uninitialized color value being used

Avoid uninitialized color value usage

Spotted-by: Jon Burgess <jburgess777@gmail.com>

Defensive code

Add new menu entry in GPS Layer to delete the realtime information.

Same method as deleting the upload and download GPS layer information.

Ensure GPS realtime information deleted when GPS Layer -->Empty All is called.

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>