On Sun, 09 Nov 2008, Szak�ts Viktor wrote:

Hi Viktor,
> Thanks Przemek, I'm not sure to enable it, because in C
> it's better practice to always check for existence,
> otherwise some compilers will throw warnings.
> Let's wait for more votes.

OK.

>> Not all C compilers accept it.
>> It means that we have to use some arbitrary value for
>> all undefined identifiers. Usually 0. If it's OK for
>> you then it's enough to change line 4609 in ppcore.c:
>>   if( !HB_PP_TOKEN_ISEOC( pToken ) || fUndef )
>> to:
>>   if( !HB_PP_TOKEN_ISEOC( pToken ) )

If we decide to change it then probably it will be better to
make sth like:

   if( fUndef )
      lValue = 0;
   else if( !HB_PP_TOKEN_ISEOC( pToken ) )
      fError = TRUE;

In such for code like:

   #if UNEXISTING_DEFINE == 0
      #stdout "UNEXISTING_DEFINE is 0"
   #endif

#stdout will not be compiled

best regards,
Przemek
_______________________________________________
Harbour mailing list
Harbour@harbour-project.org
http://lists.harbour-project.org/mailman/listinfo/harbour

Reply via email to