>What does the "ff" abbreviation stand for?

'ff' is widely used in 'page X ff', meaning: 'X and the following
pages'. I used it here for 'line X ff' because the patch programs
cuts out a block starting at line X and ending at X plus some more
lines.

>> TextOut.c:369: error: invalid lvalue in assignment

>I don't see that, and in my version, line 369 is a #ifdef.

I added the '#ifdef' at line 369. The original code was:

Out_FontHeight(o) = Out_Font_Ascent(o) + Out_Font_Descent(o);

'Out_FontHeight(o)' is a macro, defined in TextOutP.h:

#ifdef  USE_XFT 
# define Out_FontHeight(o)  \
    (Out_XftFont(o) ? Out_XftFont(o)->height : o->lineheight)
#define Out_FontAscent(o)   \
    (Out_XftFont(o) ? Out_XftFont(o)->ascent : o->font->max_bounds.ascent)
#define Out_FontDescent(o)  \
    (Out_XftFont(o) ? Out_XftFont(o)->descent : o->font->max_bounds.descent)
#else
# define Out_FontHeight(o) (o->lineheight)
# define Out_FontAscent(o) (o->font->max_bounds.ascent)
# define Out_FontDescent(o) (o->font->max_bounds.descent)
#endif

As I 'USE_XFT', 'gcc-4.x' did not accept the conditional expression:

  (Out_XftFont(o) ? Out_XftFont(o)->height : o->lineheight)

as an 'lvalue in assignment' of:

  (Out_XftFont(o) ? Out_XftFont(o)->height : o->lineheight) =
    Out_Font_Ascent(o) + Out_Font_Descent(o);

>> Xme.c:281: warning: operation on '_Xm_fastPtr' may be undefined
>
>(a) I don't see that
>(b) I don't understand why there would be this difference : the code
>    appears fully equivalent to me, except for the fact that your code
>    always does an XtBackground() call, the original code doesn't.

Accepted: clumsy proposal. Should be replaced with:

bg = (XmIsGadget(w)? XmParentBackground(w): XtBackground(w));

>I applied that patch. Do you have the latest version ?

No, my latest version is 'lesstif-apr-18-06', patched. Will download on
Saturday, apr-28-06.

winfried


Viel oder wenig? Schnell oder langsam? Unbegrenzt surfen + telefonieren
ohne Zeit- und Volumenbegrenzung? DAS TOP ANGEBOT JETZT bei Arcor: günstig
und schnell mit DSL - das All-Inclusive-Paket für clevere Doppel-Sparer,
nur  44,85 €  inkl. DSL- und ISDN-Grundgebühr!
http://www.arcor.de/rd/emf-dsl-2


-------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid0709&bid&3057&dat1642
_______________________________________________
Lesstif-discuss mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/lesstif-discuss

Reply via email to