[ gpsdrive ] WITH_MYDEBUG

[info at 4x4falcon.com]

Hi Hamish,

I have a complete rewrite of all source with the compiler directives

#ifdef MYDEBUG
#ifdef FRIENDSD
#ifdef KISMET
#ifdef NAVIGATE

included so that the program can be built with/without these.

I am not uploading them at the moment as there appears to be consensus for
a new release.

Once that's done then I will start uploading the files.

The MYDEBUG removes all the:

if (mydebug > ...

code.

This helps make the program more responsive (particularly to
touchscreen/mouse clicks) and reduces the binary file by 30k (with MYDEBUG
off).

Cheers
Ross


> Hi,
>
> Revision: 2260
>   trunk/DefineOptions.cmake
> ...
> +option(WITH_MYDEBUG "Build with debug -D support" OFF)
> ...
>
> what's the point of this? AFAICT there is no #ifdef MYDEBUG in the
> code, so adding -DMYDEBUG has no effect. (nor permanent #ifdef DEBUG
> for that matter)
>
> Also I would think that "WITH_*" would usually be reserved for "with
> support for some <packagename>, ie not some general preproc flag.
> so FRIENDSD and NAVIGATE, (+KISMET ?) would not get WITH_.
> ??
>
> and probably not bother with setting -DKISMET until that actually exists
> in the code.
>
>
> just trying to reduce the noise,
> Hamish
> _______________________________________________
> GPSdrive mailing list
> GPSdrive at lists.gpsdrivers.org
> http://lists.gpsdrivers.org/mailman/listinfo/gpsdrive
>