1. In the calculation of the colour used (especially for the height mode) what 
is the point of the '+1' here:
vdl->gcsgradient[(gint)floor((change - vdl->min_elev)/(vdl->max_elev - 
vdl->min_elev)*(DEM_N_GRADIENT_COLORS-2))+1]

If it is removed, it means when you increase the min elevation, elevations 
below this level drawn using the zero index i.e. in blue.
Hence it's then like rising sea levels and a global warming simulator!
Looks much better than the default brown.


2. What do (did?) the properties colour and line thickness supposed to do?
As far as I can make out they are to configure the 'single' gc, but as this gc 
is never actually used to draw anything - they can all be removed as well.
This excess stuff seems to be from when the DEM code was first merged.

I'll write a clean up patch in due course, unless anyone wants it kept in (with 
due reasoning).


Be Seeing You - Rob.
If at first you don't succeed,
then skydiving isn't for you.

                                          
------------------------------------------------------------------------------
This SF.net Dev2Dev email is sponsored by:

Show off your parallel programming skills.
Enter the Intel(R) Threading Challenge 2010.
http://p.sf.net/sfu/intel-thread-sfd
_______________________________________________
Viking-devel mailing list
Viking-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/viking-devel
Viking home page: http://viking.sf.net/

Reply via email to