Re: patch for fonts on Mac OS X

[Ethan M. <merritt@u.washington.edu>]

On Monday, January 09, 2012 09:40:26 am Mojca Miklavec wrote:
> 1.) in term.h:
> 
> /* Apple Macintosh */
> #ifdef _Macintosh
> # include "mac.trm"
> #endif
> 
> 1.a) Can somebody please explain me where any of the functions like
> MAC_text are defined? It seems to me as if term/mac.trm does nothing.
> I must be blind, but I don't find any function implemented anywhere.
> 
> 1.b) Obviously that doesn't hurt anyone since _Macintosh isn't defined
> anywhere but on ancient machines. But what's the point in keeping it
> if mac.trm is empty anyway? (I suspect that there was some other
> mac-specific source somewhere that got lost in years.)

Good point.  We could do code archeology, but obviously no one is
using the current mac.trm.  Out it goes!

> 2.) in plot.c:
> 
> #if defined(_Windows) || defined(_Macintosh)
> int
> gnu_main(int argc, char **argv)
> #else
> int
> main(int argc, char **argv)
> #endif
> 
> I don't understand what this does, but gnu_main is probably not needed
> on Apple (modern Mac OS X), so "|| defined(_Macintosh)" may probably
> go away if there is no code to support the old macintosh anyway.

It's a compiler flag, not an OS flag.  
Given that the next line says "gnu_main", I imagine that it was an identifier
defined by some version of gcc.   Whether the current version of gcc used on
OSX has that same flag defined or not, I don't know.

	Ethan





-- 
Ethan A Merritt
Biomolecular Structure Center,  K-428 Health Sciences Bldg
University of Washington, Seattle 98195-7742