nanogui: #if FOO vs. #ifdef BAR


Previous by date: 1 Jun 2005 15:13:30 +0100 Re: Pending patches/bug/requests ... list available on the FTP, Alain Volmat
Next by date: 1 Jun 2005 15:13:30 +0100 Re: Pending patches/bug/requests ... list available on the FTP, Jachym Holecek
Previous in thread: 1 Jun 2005 15:13:30 +0100 #if FOO vs. #ifdef BAR, Alexander Neundorf
Next in thread: 1 Jun 2005 15:13:30 +0100 Re: #if FOO vs. #ifdef BAR, Alexander Neundorf

Subject: Re: [nanogui] #if FOO vs. #ifdef BAR
From: Alain Volmat ####@####.####
Date: 1 Jun 2005 15:13:30 +0100
Message-Id: <20050601141323.GB10327@pop.mail.yahoo.com>

Hi

* Alexander Neundorf [Tue, 31 May 2005 at 22:43 +0200]
<quote>
> Hi,
> 
> e.g. in engine/devfont.c you can find the following code:
> 
> #ifdef HAVE_PCF_SUPPORT
> 	if (fontclass == MWLF_CLASS_ANY || fontclass == MWLF_CLASS_PCF) {
> ...
> #endif
> 
> #if HAVE_FREETYPE_SUPPORT
>  	if (fontclass == MWLF_CLASS_ANY || fontclass == MWLF_CLASS_FREETYPE) {
> ...
> 
> 
> which is slightly confusing.
> Which of both is the preferred style ?

I personally prefer #ifdef rather than #if. However what is the opinion of
the maintainer ?

> If somebody will commit, I'll send a patch :-)

Actually there are lots of such case in Nanox source code so if a patch
is written I don't think it should only limitate to the devfont.c file.
But on the other side, patching all similar case of the source code would
make a rather large patch which would become harder to commit. ....
 
> What is actually the current state of cvs access ?

The IP address problem ? It seems to have been fixed. I tried several time
tonight and get everytime directed to the correct IP address.
However no patch can be commited into the CVS until the version 0.91 get
released.

Alain
> 
> Bye
> Alex
> --
> Work: ####@####.#### - http://www.jenoptik-los.de
> Home: ####@####.####                - http://www.kde.org
>       ####@####.####               - http://www.neundorf.net
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: ####@####.####
> For additional commands, e-mail: ####@####.####
> 
</quote>

	

	
		
_____________________________________________________________________________ 
Découvrez le nouveau Yahoo! Mail : 1 Go d'espace de stockage pour vos mails, photos et vidéos ! 
Créez votre Yahoo! Mail sur http://fr.mail.yahoo.com

Previous by date: 1 Jun 2005 15:13:30 +0100 Re: Pending patches/bug/requests ... list available on the FTP, Alain Volmat
Next by date: 1 Jun 2005 15:13:30 +0100 Re: Pending patches/bug/requests ... list available on the FTP, Jachym Holecek
Previous in thread: 1 Jun 2005 15:13:30 +0100 #if FOO vs. #ifdef BAR, Alexander Neundorf
Next in thread: 1 Jun 2005 15:13:30 +0100 Re: #if FOO vs. #ifdef BAR, Alexander Neundorf


Powered by ezmlm-browse 0.20.