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>

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