From a07ef040e365ae4482010c213e2728bf178f662b Mon Sep 17 00:00:00 2001 From: Viktor Szakats Date: Tue, 20 Nov 2012 16:07:26 +0000 Subject: [PATCH] 2012-11-20 17:05 UTC+0100 Viktor Szakats (harbour syenar.net) * contrib/hbwin/tests/testsvc.prg + added some lines to test parameter passing * include/hbdefs.h * deprecated HB_TCHAR_*() macros with HB_LEGACY_LEVEL4 Everyone pls move on to Str API calls and hbwinuni.h macros. * include/hbexpra.c * include/hbexprb.c * include/hbwmain.c * applied uncrustify fixes --- harbour/ChangeLog | 14 ++ harbour/contrib/hbwin/tests/testsvc.prg | 7 +- harbour/include/hbdefs.h | 2 +- harbour/include/hbexpra.c | 73 +++++----- harbour/include/hbexprb.c | 182 ++++++++++++------------ harbour/include/hbwmain.c | 10 +- 6 files changed, 152 insertions(+), 136 deletions(-) diff --git a/harbour/ChangeLog b/harbour/ChangeLog index c72612b71a..a7e956d72e 100644 --- a/harbour/ChangeLog +++ b/harbour/ChangeLog @@ -10,6 +10,20 @@ * Change, ! Fix, % Optimization, + Addition, - Removal, ; Comment */ +2012-11-20 17:05 UTC+0100 Viktor Szakats (harbour syenar.net) + * contrib/hbwin/tests/testsvc.prg + + added some lines to test parameter passing + + * include/hbdefs.h + * deprecated HB_TCHAR_*() macros with HB_LEGACY_LEVEL4 + Everyone pls move on to Str API calls and hbwinuni.h + macros. + + * include/hbexpra.c + * include/hbexprb.c + * include/hbwmain.c + * applied uncrustify fixes + 2012-11-20 16:32 UTC+0100 Viktor Szakats (harbour syenar.net) * contrib/hbct/tests/*.prg * contrib/hbnetio/tests/*.prg diff --git a/harbour/contrib/hbwin/tests/testsvc.prg b/harbour/contrib/hbwin/tests/testsvc.prg index 77fc133b69..69c2d0a0d2 100644 --- a/harbour/contrib/hbwin/tests/testsvc.prg +++ b/harbour/contrib/hbwin/tests/testsvc.prg @@ -109,13 +109,18 @@ PROCEDURE Main( cMode ) #include "fileio.ch" -PROCEDURE SrvMain() +PROCEDURE SrvMain( cParam1, cParam2 ) LOCAL n := 0 LOCAL fhnd := hb_FCreate( hb_DirBase() + "testsvc.out", FC_NORMAL, FO_DENYNONE + FO_WRITE ) LOCAL cParam + hb_default( @cParam1, "" ) + hb_default( @cParam2, "" ) + FWrite( fhnd, "Startup" + hb_eol() ) + FWrite( fhnd, "|" + hb_CmdLine() + "|" + hb_eol() ) + FWrite( fhnd, "|" + cParam1 + "|" + cParam2 + "|" + hb_eol() ) FOR EACH cParam IN hb_AParams() FWrite( fhnd, "Parameter " + hb_ntos( cParam:__enumIndex() ) + " >" + cParam + "<" + hb_eol() ) diff --git a/harbour/include/hbdefs.h b/harbour/include/hbdefs.h index 6f8a283626..a77b7d21bc 100644 --- a/harbour/include/hbdefs.h +++ b/harbour/include/hbdefs.h @@ -1558,7 +1558,7 @@ typedef HB_U32 HB_FATTR; #endif -#if defined( HB_OS_WIN ) +#if defined( HB_OS_WIN ) && defined( HB_LEGACY_LEVEL4 ) #if defined( UNICODE ) #define HB_TCHAR_COPYTO(d,s,l) hb_mbntowccpy(d,s,l) #define HB_TCHAR_COPYFROM(d,s,l) hb_wcntombcpy(d,s,l) diff --git a/harbour/include/hbexpra.c b/harbour/include/hbexpra.c index 1d22a02f51..ad6e0ff671 100644 --- a/harbour/include/hbexpra.c +++ b/harbour/include/hbexpra.c @@ -78,7 +78,7 @@ HB_SIZE hb_compExprListEval2( HB_COMP_DECL, HB_EXPR_PTR pExpr1, HB_EXPR_PTR pExp { HB_SIZE nLen = 0; - if( !pEval ) + if( ! pEval ) return nLen; if( ( pExpr1->ExprType == HB_ET_LIST || pExpr1->ExprType == HB_ET_ARGLIST ) @@ -143,9 +143,9 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM * e.g. &MyVar() */ - HB_TRACE(HB_TR_DEBUG, ("hb_compExprNewFunCall(%s)", pName->value.asSymbol.name)); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprNewFunCall(%s)", pName->value.asSymbol.name ) ); -#if !defined( HB_MACRO_SUPPORT ) && defined( HB_USE_ENUM_FUNCTIONS ) +#if ! defined( HB_MACRO_SUPPORT ) && defined( HB_USE_ENUM_FUNCTIONS ) { int iLen = strlen( pName->value.asSymbol.name ); if( iLen >= 10 && i <= 12 && memcmp( "HB_ENUM", pName->value.asSymbol.name, 7 ) == 0 ) @@ -201,7 +201,7 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM } } } - else + else #endif if( pName->value.asSymbol.funcid == HB_F_EVAL && hb_compExprParamListLen( pParms ) != 0 ) @@ -216,7 +216,7 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM hb_compExprNewArgList( pParms->value.asList.pExprList->pNext, HB_COMP_PARAM ) ); -#if !defined( HB_MACRO_SUPPORT ) +#if ! defined( HB_MACRO_SUPPORT ) /* force reduction */ pEval->nLength = 1; #endif @@ -257,7 +257,7 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM _GET_( a[1], "a[1]", , , ) into: __GETA( {||a }, "a", , , , { 1 } ) - */ + */ HB_EXPR_PTR pIndex, pVar; HB_EXPR_PTR pBase; @@ -328,7 +328,7 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM if( pVar->ExprType == HB_ET_STRING ) { HB_SIZE i = 0; - char *szVar = pVar->value.asString.string; + char * szVar = pVar->value.asString.string; /* NOTE: Clipper strips a string at the first '[' character too */ @@ -336,7 +336,7 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM { if( szVar[ i ] == '[' ) { - if( !pVar->value.asString.dealloc ) + if( ! pVar->value.asString.dealloc ) { szVar = pVar->value.asString.string = ( char * ) hb_xmemdup( pVar->value.asString.string, i + 1 ); @@ -408,8 +408,9 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM } } else - { /* @ 0,0 GET &(var) - */ + { + /* @ 0,0 GET &(var) + */ #if defined( HB_MACRO_SUPPORT ) hb_macroError( EG_SYNTAX, HB_COMP_PARAM ); #else @@ -425,7 +426,7 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM pNext = pArg->pNext; pArg->pNext = NULL; /* replace first argument with a set/get codeblock */ -#if !defined( HB_MACRO_SUPPORT ) +#if ! defined( HB_MACRO_SUPPORT ) if( pArg->ExprType == HB_ET_VARIABLE ) { if( hb_compVariableFind( HB_COMP_PARAM, pArg->value.asSymbol.name, NULL, NULL ) ) @@ -456,7 +457,6 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM &pName->value.asSymbol.flags ); } } - else if( pName->ExprType == HB_ET_MACRO ) { /* Signal that macro compiler have to generate a pcode that will @@ -464,7 +464,7 @@ HB_EXPR_PTR hb_compExprNewFunCall( HB_EXPR_PTR pName, HB_EXPR_PTR pParms, HB_COM */ pName->value.asMacro.SubType = HB_ET_MACRO_SYMBOL; - HB_TRACE(HB_TR_DEBUG, ("hb_compExprNewFunCall(&)")); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprNewFunCall(&)" ) ); } pExpr = HB_COMP_EXPR_NEW( HB_ET_FUNCALL ); @@ -488,9 +488,9 @@ HB_EXPR_PTR hb_compExprNewArrayAt( HB_EXPR_PTR pArray, HB_EXPR_PTR pIndex, HB_CO HB_EXPR_PTR pExpr; #ifdef HB_MACRO_SUPPORT - HB_TRACE(HB_TR_DEBUG, ("hb_macroExprNewArrayAt()")); + HB_TRACE( HB_TR_DEBUG, ( "hb_macroExprNewArrayAt()" ) ); #else - HB_TRACE(HB_TR_DEBUG, ("hb_compExprNewArrayAt()")); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprNewArrayAt()" ) ); #endif pExpr = HB_COMP_EXPR_NEW( HB_ET_ARRAYAT ); @@ -517,11 +517,12 @@ static const char * s_szStaticFun[] = { "HB_MUTEXCREATE" }; -#define STATIC_FUNCTIONS ( sizeof( s_szStaticFun ) / sizeof( char * ) ) +#define STATIC_FUNCTIONS ( sizeof( s_szStaticFun ) / sizeof( char * ) ) static HB_BOOL hb_compStaticFunction( const char * szName ) { unsigned int ui; + for( ui = 0; ui < STATIC_FUNCTIONS; ++ui ) { if( strcmp( szName, s_szStaticFun[ ui ] ) == 0 ) @@ -535,11 +536,11 @@ static void hb_compExprCheckStaticInitializer( HB_EXPR_PTR pLeftExpr, HB_EXPR_PT { if( ( pRightExpr->ExprType > HB_ET_FUNREF || pRightExpr->ExprType == HB_ET_SELF ) && - !( pRightExpr->ExprType == HB_ET_FUNCALL && - pRightExpr->value.asFunCall.pFunName->ExprType == HB_ET_FUNNAME && - hb_compStaticFunction( pRightExpr->value.asFunCall.pFunName-> - value.asSymbol.name ) && - hb_compExprParamListLen( pRightExpr->value.asFunCall.pParms ) == 0 ) ) + ! ( pRightExpr->ExprType == HB_ET_FUNCALL && + pRightExpr->value.asFunCall.pFunName->ExprType == HB_ET_FUNNAME && + hb_compStaticFunction( pRightExpr->value.asFunCall.pFunName-> + value.asSymbol.name ) && + hb_compExprParamListLen( pRightExpr->value.asFunCall.pParms ) == 0 ) ) { /* Illegal initializer for static variable (not a constant value) */ @@ -553,7 +554,7 @@ static void hb_compExprCheckStaticListInitializers( HB_EXPR_PTR pLeftExpr, HB_EX while( *pExpr ) { - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) { /* When -kc switch is used expression list is not stripped * in reduce operation @@ -567,8 +568,8 @@ static void hb_compExprCheckStaticListInitializers( HB_EXPR_PTR pLeftExpr, HB_EX ( *pExpr )->pNext = pNext; /* restore the link to next expression */ } - if( (*pExpr)->ExprType == HB_ET_ARRAY || - (*pExpr)->ExprType == HB_ET_HASH ) + if( ( *pExpr )->ExprType == HB_ET_ARRAY || + ( *pExpr )->ExprType == HB_ET_HASH ) { hb_compExprCheckStaticListInitializers( pLeftExpr, *pExpr, HB_COMP_PARAM ); } @@ -591,7 +592,7 @@ HB_EXPR_PTR hb_compExprAssignStatic( HB_EXPR_PTR pLeftExpr, HB_EXPR_PTR pRightEx { HB_EXPR_PTR pExpr; - HB_TRACE(HB_TR_DEBUG, ("hb_compExprAssignStatic()")); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprAssignStatic()" ) ); pExpr = HB_COMP_EXPR_NEW( HB_EO_ASSIGN ); @@ -601,17 +602,17 @@ HB_EXPR_PTR hb_compExprAssignStatic( HB_EXPR_PTR pLeftExpr, HB_EXPR_PTR pRightEx /* When -kc switch is used expression list is not stripped * in reduce operation */ - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pRightExpr = hb_compExprListStrip( pRightExpr, HB_COMP_PARAM ); pExpr->value.asOperator.pRight = pRightExpr; if( pRightExpr->ExprType == HB_ET_ARGLIST ) { - /* HB_ET_ARGLIST is used in case of STATIC var[dim1, dim2, dimN] - * was used - we have to check if all array dimensions are - * constant values - */ + /* HB_ET_ARGLIST is used in case of STATIC var[dim1, dim2, dimN] + * was used - we have to check if all array dimensions are + * constant values + */ hb_compExprCheckStaticListInitializers( pLeftExpr, pRightExpr, HB_COMP_PARAM ); } else if( pRightExpr->ExprType == HB_ET_ARRAY ) @@ -640,7 +641,7 @@ HB_EXPR_PTR hb_compExprAssignStatic( HB_EXPR_PTR pLeftExpr, HB_EXPR_PTR pRightEx HB_EXPR_PTR hb_compExprSetCodeblockBody( HB_EXPR_PTR pExpr, HB_BYTE * pCode, HB_SIZE nLen ) { - HB_TRACE(HB_TR_DEBUG, ("hb_compExprSetCodeblockBody(%p,%p,%" HB_PFS "u)", pExpr, pCode, nLen)); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprSetCodeblockBody(%p,%p,%" HB_PFS "u)", pExpr, pCode, nLen ) ); pExpr->value.asCodeblock.string = ( char * ) hb_xgrab( nLen + 1 ); memcpy( pExpr->value.asCodeblock.string, pCode, nLen ); @@ -660,7 +661,7 @@ HB_EXPR_PTR hb_compExprSetCodeblockBody( HB_EXPR_PTR pExpr, HB_BYTE * pCode, HB_ */ HB_EXPR_PTR hb_macroExprGenPush( HB_EXPR_PTR pExpr, HB_COMP_DECL ) { - HB_TRACE(HB_TR_DEBUG, ("hb_macroExprGenPush(%i)", pExpr->ExprType)); + HB_TRACE( HB_TR_DEBUG, ( "hb_macroExprGenPush(%i)", pExpr->ExprType ) ); pExpr = HB_EXPR_USE( pExpr, HB_EA_REDUCE ); return HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); @@ -670,7 +671,7 @@ HB_EXPR_PTR hb_macroExprGenPush( HB_EXPR_PTR pExpr, HB_COMP_DECL ) */ HB_EXPR_PTR hb_macroExprGenPop( HB_EXPR_PTR pExpr, HB_COMP_DECL ) { - HB_TRACE(HB_TR_DEBUG, ("hb_macroExprGenPop(%i)", pExpr->ExprType)); + HB_TRACE( HB_TR_DEBUG, ( "hb_macroExprGenPop(%i)", pExpr->ExprType ) ); pExpr = HB_EXPR_USE( pExpr, HB_EA_REDUCE ); HB_EXPR_USE( pExpr, HB_EA_LVALUE ); @@ -684,7 +685,7 @@ HB_EXPR_PTR hb_macroExprGenPop( HB_EXPR_PTR pExpr, HB_COMP_DECL ) */ HB_EXPR_PTR hb_compExprGenPush( HB_EXPR_PTR pExpr, HB_COMP_DECL ) { - HB_TRACE(HB_TR_DEBUG, ("hb_compExprGenPush(%i)", pExpr->ExprType)); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprGenPush(%i)", pExpr->ExprType ) ); pExpr = HB_EXPR_USE( pExpr, HB_EA_REDUCE ); return HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); @@ -694,7 +695,7 @@ HB_EXPR_PTR hb_compExprGenPush( HB_EXPR_PTR pExpr, HB_COMP_DECL ) */ HB_EXPR_PTR hb_compExprGenPop( HB_EXPR_PTR pExpr, HB_COMP_DECL ) { - HB_TRACE(HB_TR_DEBUG, ("hb_compExprGenPop(%i)", pExpr->ExprType)); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprGenPop(%i)", pExpr->ExprType ) ); HB_EXPR_USE( pExpr, HB_EA_LVALUE ); return HB_EXPR_USE( pExpr, HB_EA_POP_PCODE ); @@ -705,7 +706,7 @@ HB_EXPR_PTR hb_compExprGenPop( HB_EXPR_PTR pExpr, HB_COMP_DECL ) */ HB_EXPR_PTR hb_compExprGenStatement( HB_EXPR_PTR pExpr, HB_COMP_DECL ) { - HB_TRACE(HB_TR_DEBUG, ("hb_compExprGenStatement(%p)", pExpr)); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprGenStatement(%p)", pExpr ) ); if( pExpr ) { if( pExpr->ExprType == HB_EO_EQUAL ) diff --git a/harbour/include/hbexprb.c b/harbour/include/hbexprb.c index 7c6f277ae8..15617f769c 100644 --- a/harbour/include/hbexprb.c +++ b/harbour/include/hbexprb.c @@ -53,7 +53,7 @@ #include "hbcomp.h" #include "hbmacro.ch" -#if !defined( HB_HASH_USES_ARRAY_INDEXES ) +#if ! defined( HB_HASH_USES_ARRAY_INDEXES ) # define HB_HASH_USES_ARRAY_INDEXES #endif @@ -126,7 +126,7 @@ static HB_EXPR_FUNC( hb_compExprUseIN ); static HB_EXPR_FUNC( hb_compExprUseNegate ); /* other helper functions -*/ + */ #if defined( HB_MACRO_SUPPORT ) static void hb_compExprCodeblockPush( HB_EXPR_PTR, HB_COMP_DECL ); #else @@ -145,7 +145,7 @@ static void hb_compExprUsePreOp( HB_EXPR_PTR pSelf, HB_BYTE bOper, HB_COMP_DECL static void hb_compExprUseAliasMacro( HB_EXPR_PTR pAliasedVar, HB_BYTE bAction, HB_COMP_DECL ); static HB_EXPR_PTR hb_compExprReduceList( HB_EXPR_PTR pExpr, HB_COMP_DECL ); static HB_EXPR_PTR hb_compExprReduceAliasString( HB_EXPR_PTR pExpr, HB_EXPR_PTR pAlias, HB_COMP_DECL ); -static HB_BOOL hb_compExprIsMemvarAlias( const char *szAlias ); +static HB_BOOL hb_compExprIsMemvarAlias( const char * szAlias ); const HB_EXPR_FUNC_PTR hb_comp_ExprTable[ HB_EXPR_COUNT ] = { @@ -178,19 +178,19 @@ const HB_EXPR_FUNC_PTR hb_comp_ExprTable[ HB_EXPR_COUNT ] = { hb_compExprUseAlias, hb_compExprUseRTVariable, hb_compExprUseVariable, - hb_compExprUsePostInc, /* post-operators -> lowest precedence */ + hb_compExprUsePostInc, /* post-operators -> lowest precedence */ hb_compExprUsePostDec, - hb_compExprUseAssign, /* assigments */ + hb_compExprUseAssign, /* assigments */ hb_compExprUsePlusEq, hb_compExprUseMinusEq, hb_compExprUseMultEq, hb_compExprUseDivEq, hb_compExprUseModEq, hb_compExprUseExpEq, - hb_compExprUseOr, /* logical operators */ + hb_compExprUseOr, /* logical operators */ hb_compExprUseAnd, hb_compExprUseNot, - hb_compExprUseEqual, /* relational operators */ + hb_compExprUseEqual, /* relational operators */ hb_compExprUseEQ, hb_compExprUseNE, hb_compExprUseIN, @@ -198,9 +198,9 @@ const HB_EXPR_FUNC_PTR hb_comp_ExprTable[ HB_EXPR_COUNT ] = { hb_compExprUseGT, hb_compExprUseLE, hb_compExprUseGE, - hb_compExprUsePlus, /* addition */ + hb_compExprUsePlus, /* addition */ hb_compExprUseMinus, - hb_compExprUseMult, /* multiple */ + hb_compExprUseMult, /* multiple */ hb_compExprUseDiv, hb_compExprUseMod, hb_compExprUsePower, @@ -314,7 +314,7 @@ static HB_EXPR_FUNC( hb_compExprUseDate ) break; case HB_EA_ARRAY_INDEX: #ifdef HB_HASH_USES_ARRAY_INDEXES - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) #endif hb_compErrorIndex( HB_COMP_PARAM, pSelf ); /* Date cannot be used as index element */ break; @@ -349,7 +349,7 @@ static HB_EXPR_FUNC( hb_compExprUseTimeStamp ) break; case HB_EA_ARRAY_INDEX: #ifdef HB_HASH_USES_ARRAY_INDEXES - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) #endif hb_compErrorIndex( HB_COMP_PARAM, pSelf ); /* timestamp cannot be used as index element */ break; @@ -385,7 +385,7 @@ static HB_EXPR_FUNC( hb_compExprUseString ) break; case HB_EA_ARRAY_INDEX: #ifdef HB_HASH_USES_ARRAY_INDEXES - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) #endif hb_compErrorIndex( HB_COMP_PARAM, pSelf ); /* string cannot be used as index element */ break; @@ -393,9 +393,8 @@ static HB_EXPR_FUNC( hb_compExprUseString ) hb_compErrorLValue( HB_COMP_PARAM, pSelf ); break; case HB_EA_PUSH_PCODE: - { #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_MACROTEXT ) + if( ! HB_SUPPORT_MACROTEXT ) HB_GEN_FUNC2( PushString, pSelf->value.asString.string, pSelf->nLength + 1 ); else @@ -412,7 +411,6 @@ static HB_EXPR_FUNC( hb_compExprUseString ) } #endif break; - } case HB_EA_POP_PCODE: break; @@ -448,21 +446,19 @@ static HB_EXPR_FUNC( hb_compExprUseCodeblock ) hb_compErrorLValue( HB_COMP_PARAM, pSelf ); break; case HB_EA_PUSH_PCODE: - { #if defined( HB_MACRO_SUPPORT ) hb_compExprCodeblockPush( pSelf, HB_COMP_PARAM ); #else if( pSelf->value.asCodeblock.flags & HB_BLOCK_EXT ) hb_compExprCodeblockExtPush( pSelf, HB_COMP_PARAM ); else if( ( pSelf->value.asCodeblock.flags & HB_BLOCK_MACROVAR ) && - !( pSelf->value.asCodeblock.flags & HB_BLOCK_VPARAMS ) ) + ! ( pSelf->value.asCodeblock.flags & HB_BLOCK_VPARAMS ) ) /* early evaluation of a macro */ hb_compExprCodeblockEarly( pSelf, HB_COMP_PARAM ); else hb_compExprCodeblockPush( pSelf, 0, HB_COMP_PARAM ); #endif break; - } case HB_EA_POP_PCODE: break; case HB_EA_PUSH_POP: @@ -590,7 +586,7 @@ static HB_EXPR_FUNC( hb_compExprUseArray ) { HB_BOOL fArgsList = pSelf->ExprType == HB_ET_MACROARGLIST; - if( !fArgsList ) + if( ! fArgsList ) { /* Note: direct type change */ pSelf->ExprType = HB_ET_ARGLIST; @@ -637,7 +633,7 @@ static HB_EXPR_FUNC( hb_compExprUseArray ) { HB_EXPR_PTR pElem = pSelf->value.asList.pExprList; /* Delete all elements of the array - */ + */ while( pElem ) { HB_EXPR_PTR pNext = pElem->pNext; @@ -708,7 +704,7 @@ static HB_EXPR_FUNC( hb_compExprUseHash ) { HB_EXPR_PTR pElem = pSelf->value.asList.pExprList; /* Delete all elements of the hash array - */ + */ while( pElem ) { HB_EXPR_PTR pNext = pElem->pNext; @@ -840,7 +836,7 @@ static HB_EXPR_FUNC( hb_compExprUseRef ) else if( pExp->ExprType == HB_ET_SEND ) { /* HB_EXPR_PTR pSend = pExp->value.asMessage.pObject; - if( !pSend || pSend->ExprType == HB_ET_VARIABLE ) */ + if( ! pSend || pSend->ExprType == HB_ET_VARIABLE ) */ { hb_compExprPushSendPop( pExp, HB_COMP_PARAM ); HB_GEN_FUNC1( PCode1, HB_P_PUSHOVARREF ); @@ -861,7 +857,8 @@ static HB_EXPR_FUNC( hb_compExprUseRef ) if( pExp->value.asAlias.pVar->ExprType == HB_ET_VARIABLE && pExp->value.asAlias.pAlias->ExprType == HB_ET_ALIAS && hb_compExprIsMemvarAlias( pExp->value.asAlias.pAlias->value.asSymbol.name ) ) - { /* @M-> @MEMVAR-> or @MEMVA-> or @MEMV-> */ + { + /* @M-> @MEMVAR-> or @MEMVA-> or @MEMV-> */ HB_GEN_FUNC1( PushMemvarRef, pExp->value.asAlias.pVar->value.asSymbol.name ); break; } @@ -918,17 +915,17 @@ static HB_EXPR_FUNC( hb_compExprUseIIF ) case HB_EA_PUSH_PCODE: { /* this is called if all three parts of IIF expression should be generated - */ + */ HB_ISIZ nPosFalse, nPosEnd; HB_EXPR_PTR pExpr = pSelf->value.asList.pExprList; HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); nPosFalse = HB_GEN_FUNC1( JumpFalse, 0 ); - pExpr =pExpr->pNext; + pExpr = pExpr->pNext; HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); nPosEnd = HB_GEN_FUNC1( Jump, 0 ); - pExpr =pExpr->pNext; + pExpr = pExpr->pNext; HB_GEN_FUNC1( JumpHere, nPosFalse ); HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); @@ -938,17 +935,17 @@ static HB_EXPR_FUNC( hb_compExprUseIIF ) case HB_EA_POP_PCODE: { /* this is called if all three parts of IIF expression should be generated - */ + */ HB_ISIZ nPosFalse, nPosEnd; HB_EXPR_PTR pExpr = pSelf->value.asList.pExprList; HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); nPosFalse = HB_GEN_FUNC1( JumpFalse, 0 ); - pExpr =pExpr->pNext; + pExpr = pExpr->pNext; HB_EXPR_USE( pExpr, HB_EA_POP_PCODE ); nPosEnd = HB_GEN_FUNC1( Jump, 0 ); - pExpr =pExpr->pNext; + pExpr = pExpr->pNext; HB_GEN_FUNC1( JumpHere, nPosFalse ); HB_EXPR_USE( pExpr, HB_EA_POP_PCODE ); @@ -968,12 +965,12 @@ static HB_EXPR_FUNC( hb_compExprUseIIF ) HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); nPosFalse = HB_GEN_FUNC1( JumpFalse, 0 ); - pExpr =pExpr->pNext; + pExpr = pExpr->pNext; /* do not generate warning about meaningless use of expression NIL */ if( pExpr->ExprType != HB_ET_NIL ) HB_EXPR_USE( pExpr, HB_EA_PUSH_POP ); - pExpr =pExpr->pNext; + pExpr = pExpr->pNext; #if defined( HB_MACRO_SUPPORT ) if( HB_PCODE_DATA->nPCodePos == nPosFalse + 3 ) @@ -1159,9 +1156,9 @@ static HB_EXPR_FUNC( hb_compExprUseArgList ) if( pSelf->value.asList.reference ) { #if defined( HB_MACRO_SUPPORT ) - if( !HB_PCODE_DATA->fVParams ) + if( ! HB_PCODE_DATA->fVParams ) #else - if( !HB_COMP_PARAM->functions.pLast->fVParams ) + if( ! HB_COMP_PARAM->functions.pLast->fVParams ) #endif { hb_compErrorVParams( HB_COMP_PARAM, @@ -1291,7 +1288,7 @@ static HB_EXPR_FUNC( hb_compExprUseArrayAt ) */ if( pSelf->value.asList.pExprList->ExprType == HB_ET_VARIABLE ) { -#if !defined( HB_MACRO_SUPPORT ) +#if ! defined( HB_MACRO_SUPPORT ) int iScope; hb_compVariableFind( HB_COMP_PARAM, pSelf->value.asList.pExprList->value.asSymbol.name, NULL, &iScope ); if( iScope == HB_VS_UNDECLARED ) @@ -1327,9 +1324,9 @@ static HB_EXPR_FUNC( hb_compExprUseArrayAt ) if( pExpr->ExprType == HB_ET_MACROARGLIST ) /* restore original expression type */ pExpr->ExprType = HB_ET_ARRAY; - else if( !HB_IS_VALID_INDEX( nIndex, nSize ) ) + else if( ! HB_IS_VALID_INDEX( nIndex, nSize ) ) { - if( !HB_SUPPORT_ARRSTR ) + if( ! HB_SUPPORT_ARRSTR ) hb_compErrorBound( HB_COMP_PARAM, pIdx ); } else @@ -1354,7 +1351,7 @@ static HB_EXPR_FUNC( hb_compExprUseArrayAt ) HB_COMP_EXPR_FREE( pSelf ); pSelf = pNew; } - else if( !HB_SUPPORT_ARRSTR ) + else if( ! HB_SUPPORT_ARRSTR ) hb_compErrorBound( HB_COMP_PARAM, pIdx ); } } @@ -1372,7 +1369,7 @@ static HB_EXPR_FUNC( hb_compExprUseArrayAt ) hb_compErrorBound( HB_COMP_PARAM, pIdx ); } #endif - else if( !HB_SUPPORT_ARRSTR ) + else if( ! HB_SUPPORT_ARRSTR ) { HB_EXPR_USE( pExpr, HB_EA_ARRAY_AT ); } @@ -1438,7 +1435,7 @@ static HB_EXPR_FUNC( hb_compExprUseArrayAt ) HB_GEN_FUNC1( PCode1, HB_P_PUSHOVARREF ); } else if( pList->ExprType == HB_ET_ARRAYAT && - !pList->value.asList.reference ) + ! pList->value.asList.reference ) { pList->value.asList.reference = HB_TRUE; HB_EXPR_USE( pList, HB_EA_PUSH_PCODE ); @@ -1520,7 +1517,7 @@ static HB_EXPR_FUNC( hb_compExprUseArrayAt ) HB_GEN_FUNC1( PCode1, HB_P_PUSHOVARREF ); } else if( pList->ExprType == HB_ET_ARRAYAT && - !pList->value.asList.reference ) + ! pList->value.asList.reference ) { pList->value.asList.reference = HB_TRUE; HB_EXPR_USE( pList, HB_EA_PUSH_PCODE ); @@ -2214,7 +2211,7 @@ static HB_EXPR_FUNC( hb_compExprUseFunCall ) */ static HB_EXPR_FUNC( hb_compExprUseAliasVar ) { - HB_TRACE(HB_TR_DEBUG, ("hb_compExprUseAliasVar()")); + HB_TRACE( HB_TR_DEBUG, ( "hb_compExprUseAliasVar()" ) ); switch( iMessage ) { @@ -2266,7 +2263,7 @@ static HB_EXPR_FUNC( hb_compExprUseAliasVar ) * * NOTE: HB_TRUE = push also alias */ - HB_GEN_FUNC4( PushAliasedVar, pSelf->value.asAlias.pVar->value.asSymbol.name, HB_TRUE, pAlias->value.asSymbol.name, 0 ); + HB_GEN_FUNC4( PushAliasedVar, pSelf->value.asAlias.pVar->value.asSymbol.name, HB_TRUE, pAlias->value.asSymbol.name, 0 ); } else if( pAlias->ExprType == HB_ET_NUMERIC ) { @@ -2528,13 +2525,13 @@ static HB_EXPR_FUNC( hb_compExprUseVariable ) case HB_EA_PUSH_PCODE: #if defined( HB_MACRO_SUPPORT ) /* NOTE: When the following syntax is used: - * ( any_expr )->&var2 - * then macro compiler is compiling the right side of alias - * operator only (if 'any_expr' is not a string) - an alias value - * is placed on the eval stack before macro compilation. - * The HB_MACRO_GEN_ALIASED flag is used to signal that we have to - * genearate alias aware pcode even if we known a variable part only. - */ + * ( any_expr )->&var2 + * then macro compiler is compiling the right side of alias + * operator only (if 'any_expr' is not a string) - an alias value + * is placed on the eval stack before macro compilation. + * The HB_MACRO_GEN_ALIASED flag is used to signal that we have to + * genearate alias aware pcode even if we known a variable part only. + */ if( HB_MACRO_DATA->Flags & HB_MACRO_GEN_ALIASED ) HB_GEN_FUNC4( PushAliasedVar, pSelf->value.asSymbol.name, HB_FALSE, NULL, 0 ); else @@ -2576,7 +2573,7 @@ static HB_EXPR_FUNC( hb_compExprUseSetGet ) pSelf->value.asSetGet.pVar = HB_EXPR_USE( pSelf->value.asSetGet.pVar, HB_EA_REDUCE ); pSelf->value.asSetGet.pExpr = HB_EXPR_USE( pSelf->value.asSetGet.pExpr, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asSetGet.pVar = hb_compExprListStrip( pSelf->value.asSetGet.pVar, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asSetGet.pVar, HB_EA_LVALUE ); @@ -2730,7 +2727,7 @@ static HB_EXPR_FUNC( hb_compExprUseSend ) HB_GEN_FUNC2( PCode2, HB_P_SWAP, 1 ); } HB_GEN_FUNC2( PCode2, HB_P_SENDSHORT, 1 ); - if( !pSelf->value.asMessage.pParms ) + if( ! pSelf->value.asMessage.pParms ) HB_GEN_FUNC1( PCode1, HB_P_POP ); break; @@ -2749,14 +2746,12 @@ static HB_EXPR_FUNC( hb_compExprUseSend ) break; case HB_EA_DELETE: - { - if( pSelf->value.asMessage.pObject ) - HB_COMP_EXPR_FREE( pSelf->value.asMessage.pObject ); - if( pSelf->value.asMessage.pParms ) - HB_COMP_EXPR_FREE( pSelf->value.asMessage.pParms ); - if( pSelf->value.asMessage.pMessage ) - HB_COMP_EXPR_FREE( pSelf->value.asMessage.pMessage ); - } + if( pSelf->value.asMessage.pObject ) + HB_COMP_EXPR_FREE( pSelf->value.asMessage.pObject ); + if( pSelf->value.asMessage.pParms ) + HB_COMP_EXPR_FREE( pSelf->value.asMessage.pParms ); + if( pSelf->value.asMessage.pMessage ) + HB_COMP_EXPR_FREE( pSelf->value.asMessage.pMessage ); break; } return pSelf; @@ -2769,7 +2764,7 @@ static HB_EXPR_FUNC( hb_compExprUsePostInc ) case HB_EA_REDUCE: pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -2811,7 +2806,7 @@ static HB_EXPR_FUNC( hb_compExprUsePostDec ) case HB_EA_REDUCE: pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -2855,7 +2850,7 @@ static HB_EXPR_FUNC( hb_compExprUseAssign ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -2927,7 +2922,7 @@ static HB_EXPR_FUNC( hb_compExprUseAssign ) HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_PUSH_PCODE ); /* QUESTION: Can we replace DUPLICATE+POP with a single PUT opcode - */ + */ HB_GEN_FUNC1( PCode1, HB_P_PUSHUNREF ); HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_POP_PCODE ); } @@ -2974,7 +2969,7 @@ static HB_EXPR_FUNC( hb_compExprUsePlusEq ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -3018,7 +3013,7 @@ static HB_EXPR_FUNC( hb_compExprUseMinusEq ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -3062,7 +3057,7 @@ static HB_EXPR_FUNC( hb_compExprUseMultEq ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -3106,7 +3101,7 @@ static HB_EXPR_FUNC( hb_compExprUseDivEq ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -3150,7 +3145,7 @@ static HB_EXPR_FUNC( hb_compExprUseModEq ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -3194,7 +3189,7 @@ static HB_EXPR_FUNC( hb_compExprUseExpEq ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -3969,8 +3964,8 @@ static HB_EXPR_FUNC( hb_compExprUsePlus ) break; } else if( pLeft->value.asNum.NumType == HB_ET_LONG ? - pLeft->value.asNum.val.l == -1 : - pLeft->value.asNum.val.d == -1 ) + pLeft->value.asNum.val.l == -1 : + pLeft->value.asNum.val.d == -1 ) { HB_EXPR_USE( pRight, HB_EA_PUSH_PCODE ); HB_GEN_FUNC1( PCode1, HB_P_DEC ); @@ -3988,8 +3983,8 @@ static HB_EXPR_FUNC( hb_compExprUsePlus ) break; } else if( pRight->value.asNum.NumType == HB_ET_LONG ? - pRight->value.asNum.val.l == -1 : - pRight->value.asNum.val.d == -1 ) + pRight->value.asNum.val.l == -1 : + pRight->value.asNum.val.d == -1 ) { HB_EXPR_USE( pLeft, HB_EA_PUSH_PCODE ); HB_GEN_FUNC1( PCode1, HB_P_DEC ); @@ -4068,8 +4063,8 @@ static HB_EXPR_FUNC( hb_compExprUseMinus ) break; } else if( pRight->value.asNum.NumType == HB_ET_LONG ? - pRight->value.asNum.val.l == -1 : - pRight->value.asNum.val.d == -1 ) + pRight->value.asNum.val.l == -1 : + pRight->value.asNum.val.d == -1 ) { HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_PUSH_PCODE ); HB_GEN_FUNC1( PCode1, HB_P_INC ); @@ -4233,7 +4228,7 @@ static HB_EXPR_FUNC( hb_compExprUseMod ) case HB_EA_REDUCE: pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); - pSelf =hb_compExprReduceMod( pSelf, HB_COMP_PARAM ); + pSelf = hb_compExprReduceMod( pSelf, HB_COMP_PARAM ); break; case HB_EA_ARRAY_AT: @@ -4291,7 +4286,7 @@ static HB_EXPR_FUNC( hb_compExprUsePower ) pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); pSelf->value.asOperator.pRight = HB_EXPR_USE( pSelf->value.asOperator.pRight, HB_EA_REDUCE ); if( HB_SUPPORT_HARBOUR ) /* Clipper doesn't optimize it */ - pSelf =hb_compExprReducePower( pSelf, HB_COMP_PARAM ); + pSelf = hb_compExprReducePower( pSelf, HB_COMP_PARAM ); break; case HB_EA_ARRAY_AT: @@ -4403,7 +4398,7 @@ static HB_EXPR_FUNC( hb_compExprUsePreInc ) case HB_EA_REDUCE: pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -4447,7 +4442,7 @@ static HB_EXPR_FUNC( hb_compExprUsePreDec ) case HB_EA_REDUCE: pSelf->value.asOperator.pLeft = HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_REDUCE ); #if ! defined( HB_MACRO_SUPPORT ) - if( !HB_SUPPORT_HARBOUR ) + if( ! HB_SUPPORT_HARBOUR ) pSelf->value.asOperator.pLeft = hb_compExprListStrip( pSelf->value.asOperator.pLeft, HB_COMP_PARAM ); #endif HB_EXPR_USE( pSelf->value.asOperator.pLeft, HB_EA_LVALUE ); @@ -4488,7 +4483,7 @@ static HB_EXPR_FUNC( hb_compExprUsePreDec ) /* This generates a push pcode for a codeblock (with no macro expression or with late evaluation of a macro) -*/ + */ #if defined( HB_MACRO_SUPPORT ) static void hb_compExprCodeblockPush( HB_EXPR_PTR pSelf, HB_COMP_DECL ) #else @@ -4518,7 +4513,7 @@ static HB_BOOL hb_compExprCodeblockPush( HB_EXPR_PTR pSelf, int iEarlyEvalPass, while( pVar ) { hb_compVariableAdd( HB_COMP_PARAM, pVar->szName, hb_compVarTypeNew( HB_COMP_PARAM, pVar->bType, NULL ) ); - pVar =pVar->pNext; + pVar = pVar->pNext; } } @@ -4549,7 +4544,7 @@ static HB_BOOL hb_compExprCodeblockPush( HB_EXPR_PTR pSelf, int iEarlyEvalPass, if( ( pSelf->value.asCodeblock.flags & HB_BLOCK_REDUCE ) != 0 || HB_SUPPORT_HARBOUR ) { - * pPrev = pExpr = HB_EXPR_USE( pExpr, HB_EA_REDUCE ); + *pPrev = pExpr = HB_EXPR_USE( pExpr, HB_EA_REDUCE ); pExpr->pNext = pNext; /* restore the link to next expression */ } @@ -4573,8 +4568,8 @@ static HB_BOOL hb_compExprCodeblockPush( HB_EXPR_PTR pSelf, int iEarlyEvalPass, else HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); #endif - pPrev = &pExpr->pNext; - pExpr = pNext; + pPrev = &pExpr->pNext; + pExpr = pNext; } #if defined( HB_MACRO_SUPPORT ) hb_macroCodeBlockEnd( HB_COMP_PARAM ); @@ -4593,8 +4588,8 @@ static HB_BOOL hb_compExprCodeblockPush( HB_EXPR_PTR pSelf, int iEarlyEvalPass, } /* This generates a push pcode for early evaluation of a macro -*/ -#if !defined( HB_MACRO_SUPPORT ) + */ +#if ! defined( HB_MACRO_SUPPORT ) static void hb_compExprCodeblockExtPush( HB_EXPR_PTR pSelf, HB_COMP_DECL ) { hb_compGenPCodeN( ( HB_BYTE * ) pSelf->value.asCodeblock.string, @@ -4629,7 +4624,7 @@ static void hb_compExprCodeblockEarly( HB_EXPR_PTR pSelf, HB_COMP_DECL ) /* everything else is macro compiled at runtime * {|| &variable+1} => &( '{|| &variable+1}' ) */ - if( !hb_compExprCodeblockPush( pSelf, 1, HB_COMP_PARAM ) ) + if( ! hb_compExprCodeblockPush( pSelf, 1, HB_COMP_PARAM ) ) { HB_BOOL fMacroText = ( HB_COMP_PARAM->supported & HB_COMPFLAG_MACROTEXT ) != 0; HB_COMP_PARAM->supported |= HB_COMPFLAG_MACROTEXT; @@ -4638,7 +4633,7 @@ static void hb_compExprCodeblockEarly( HB_EXPR_PTR pSelf, HB_COMP_DECL ) HB_EXPR_USE( pExpr, HB_EA_PUSH_PCODE ); HB_COMP_EXPR_FREE( pExpr ); hb_compCodeBlockStop( HB_COMP_PARAM ); - if( !fMacroText ) + if( ! fMacroText ) HB_COMP_PARAM->supported &= ~HB_COMPFLAG_MACROTEXT; } } @@ -5161,7 +5156,7 @@ static void hb_compExprPushPreOp( HB_EXPR_PTR pSelf, HB_BYTE bOper, HB_COMP_DECL return; } #endif -#if !defined( HB_MACRO_SUPPORT ) +#if ! defined( HB_MACRO_SUPPORT ) else if( pSelf->value.asOperator.pLeft->ExprType == HB_ET_VARIABLE ) { int iVar, iScope; @@ -5276,7 +5271,7 @@ static void hb_compExprPushPostOp( HB_EXPR_PTR pSelf, HB_BYTE bOper, HB_COMP_DEC return; } #endif -#if !defined( HB_MACRO_SUPPORT ) +#if ! defined( HB_MACRO_SUPPORT ) else if( pSelf->value.asOperator.pLeft->ExprType == HB_ET_VARIABLE ) { int iVar, iScope; @@ -5376,7 +5371,7 @@ static void hb_compExprUsePreOp( HB_EXPR_PTR pSelf, HB_BYTE bOper, HB_COMP_DECL return; } #endif -#if !defined( HB_MACRO_SUPPORT ) +#if ! defined( HB_MACRO_SUPPORT ) else if( pSelf->value.asOperator.pLeft->ExprType == HB_ET_VARIABLE ) { int iVar, iScope; @@ -5507,7 +5502,7 @@ static HB_EXPR_PTR hb_compExprReduceAliasString( HB_EXPR_PTR pExpr, HB_EXPR_PTR while( nLen ) { char c = szAlias[ nLen - 1 ]; - if( !HB_ISNEXTIDCHAR( c ) ) + if( ! HB_ISNEXTIDCHAR( c ) ) break; if( HB_ISLOWER( c ) ) fLower = HB_TRUE; @@ -5534,9 +5529,10 @@ static HB_EXPR_PTR hb_compExprReduceAliasString( HB_EXPR_PTR pExpr, HB_EXPR_PTR return pExpr; } -static HB_BOOL hb_compExprIsMemvarAlias( const char *szAlias ) +static HB_BOOL hb_compExprIsMemvarAlias( const char * szAlias ) { int iLen = ( int ) strlen( szAlias ); + /* @M-> @MEMVAR-> or @MEMVA-> or @MEMV-> */ return ( iLen == 1 || ( iLen >= 4 && iLen <= 6 ) ) && memcmp( szAlias, "MEMVAR", iLen ) == 0; diff --git a/harbour/include/hbwmain.c b/harbour/include/hbwmain.c index b64c60b738..dbb569dfe8 100644 --- a/harbour/include/hbwmain.c +++ b/harbour/include/hbwmain.c @@ -53,9 +53,9 @@ #include #if defined( HB_OS_WIN_CE ) -# define HB_LPSTR LPWSTR +# define HB_LPSTR LPWSTR #else -# define HB_LPSTR LPSTR +# define HB_LPSTR LPSTR #endif int WINAPI WinMain( HINSTANCE hInstance, /* handle to current instance */ @@ -69,7 +69,7 @@ int WINAPI WinMain( HINSTANCE hInstance, /* handle to current instance */ HB_SYMBOL_UNUSED( lpCmdLine ); - /* HB_TRACE(HB_TR_DEBUG, ("WinMain(%p, %p, %s, %d)", hInstance, hPrevInstance, lpCmdLine, iCmdShow)); */ + /* HB_TRACE( HB_TR_DEBUG, ("WinMain(%p, %p, %s, %d)", hInstance, hPrevInstance, lpCmdLine, iCmdShow ) ); */ hb_winmainArgInit( hInstance, hPrevInstance, iCmdShow ); @@ -108,9 +108,9 @@ int WINAPI WinMain( HINSTANCE hInstance, /* handle to current instance */ { if( pArg == NULL ) pArg = pDst; - fQuoted = !fQuoted; + fQuoted = ! fQuoted; } - else if( fQuoted || !HB_ISSPACE( *pSrc ) ) + else if( fQuoted || ! HB_ISSPACE( *pSrc ) ) { if( pArg == NULL ) pArg = pDst;