Changeset 6338


Ignore:
Timestamp:
01/19/11 10:33:21 (3 years ago)
Author:
garry.yao
Message:

CONSTANTS for function type instead of magic numbers.

Location:
CKPackager/trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • CKPackager/trunk/_source/includes/scriptcompressor.js

    r6330 r6338  
    379379 
    380380                                // Named function expression affects only inner scope. 
    381                                 if ( fn.functionType == 2 && name.length() ) 
     381                                if ( fn.functionType == fn.FUNCTION_EXPRESSION && name.length() ) 
    382382                                { 
    383383                                        scope.declareName( name ); 
     
    406406 
    407407                                // Remove the "scope downcast" provided by Rhino. 
    408                                 if ( fn.functionType == 2 && name.length() ) 
     408                                if ( fn.functionType == fn.FUNCTION_EXPRESSION && name.length() ) 
    409409                                        funcBody.removeChild( funcBody.getFirstChild() ); 
    410410 
     
    417417 
    418418                                // No trailing semicolon for function expression. 
    419                                 if ( fn.functionType != 2 && name.length() ) 
     419                                if ( fn.functionType != fn.FUNCTION_EXPRESSION && name.length() ) 
    420420                                        out( ';' ); 
    421421                                break; 
     
    837837                                        hasCurly = true; 
    838838 
    839                                 hasCurly =  
    840                                         (  
    841                                                 !block  
    842                                                 ||  
    843                                                 (  
    844                                                         block.getType() == Token.BLOCK  
    845                                                         &&  
    846                                                         (  
    847                                                                 !block.getFirstChild()  
    848                                                                 ||  
    849                                                                 block.getFirstChild().getNext()  
     839                                hasCurly = 
     840                                        ( 
     841                                                !block 
     842                                                || 
     843                                                ( 
     844                                                        block.getType() == Token.BLOCK 
     845                                                        && 
     846                                                        ( 
     847                                                                !block.getFirstChild() 
     848                                                                || 
     849                                                                block.getFirstChild().getNext() 
    850850                                                                || 
    851851                                                                ( 
    852852                                                                        ( 
    853                                                                                 block.getFirstChild().getType() == Token.BLOCK  
     853                                                                                block.getFirstChild().getType() == Token.BLOCK 
    854854                                                                                || 
    855855                                                                                block.getFirstChild().getType() == Token.IFNE 
     
    858858                                                                        block.getNext().getType() == Token.GOTO 
    859859                                                                ) 
    860                                                         )  
    861                                                 )  
     860                                                        ) 
     861                                                ) 
    862862                                        ); 
    863863 
     
    987987                                node = node.getFirstChild(); 
    988988 
    989                                 var hasPrecedence =  
    990                                                 associativityRTL[ node.getType() ]  
     989                                var hasPrecedence = 
     990                                                associativityRTL[ node.getType() ] 
    991991                                                && precedence[ node.getType() ] 
    992992                                                && ( precedence[ Token.HOOK ] <= precedence[ node.getType() ] ); 
    993993 
    994994                                if ( hasPrecedence ) 
    995                                         out( '(' );                                              
     995                                        out( '(' ); 
    996996 
    997997                                writeNode( node ); 
    998998 
    999999                                if ( hasPrecedence ) 
    1000                                         out( ')' );                              
     1000                                        out( ')' ); 
    10011001 
    10021002                                out( '?' ); 
Note: See TracChangeset for help on using the changeset viewer.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy