On Tue, Oct 25, 2005 at 03:45:28PM -0700, Markus Mayer wrote:
> Would it be possible to consider this patch (or a modified version of
> the proposed patch from 2003) for application to the mainline?
> 
> Best regards,
> -Markus

> diff -ru lyx-1.3.6.orig/src/lyxlength.C lyx-1.3.6/src/lyxlength.C
> --- lyx-1.3.6.orig/src/lyxlength.C    2005-01-19 06:49:49.000000000 -0800
> +++ lyx-1.3.6/src/lyxlength.C 2005-10-25 15:33:30.932284000 -0700
> @@ -141,7 +141,7 @@
>       double result = 0.0;
>  
>       switch (unit_) {
> -     case LyXLength::SP:
> +     case LyXLength::SC:

Unfortunately, 'SP' is the canonical name. 

On the other hand, there is no need to have them in ALL_CAPS
which _is_ the traditional namespace for macros.

Maybe  'UnitSP' or such would be a reasonable compromise?

Andre'

Reply via email to