Changeset 6338


Ignore:
Timestamp:
01/19/2011 10:33:21 AM (4 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