Changeset 1582 in svn


Ignore:
Timestamp:
2015-04-18T07:26:12Z (4 years ago)
Author:
Xuefer
Message:

Decompiler: code style clean up

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/lib/Decompiler.class.php

    r1530 r1582  
    2727                    }
    2828                }
    29                 $args[] = 'array(' . implode(', ', $array) . ')';
     29                $args[] = "array(" . implode(', ', $array) . ')';
    3030            }
    3131            else {
    … …  
    439439        }
    440440        $max = call_user_func_array('max', $keys);
    441         $list = 'list(';
    442         for ($i = 0; $i <= $max; $i ++) {
     441        $list = "list(";
     442        for ($i = 0; $i <= $max; $i++) {
    443443            if ($i) {
    444444                $list .= ', ';
    … …  
    506506            }
    507507            if (is_array($value) || is_a($value, 'Decompiler_Array')) {
    508                 $multiline ++;
     508                $multiline++;
    509509            }
    510510        }
    … …  
    537537            $exp .= $value;
    538538
    539             $i ++;
     539            $i++;
    540540        }
    541541        if ($multiline) {
    … …  
    602602        if ($this->test) {
    603603            $content = file_get_contents(__FILE__);
    604             for ($i = 0; $opname = xcache_get_opcode($i); $i ++) {
     604            for ($i = 0; $opname = xcache_get_opcode($i); $i++) {
    605605                if (!preg_match("/\\bXC_" . $opname . "\\b(?!')/", $content)) {
    606606                    echo "not recognized opcode ", $opname, "\n";
    … …  
    700700        $indent = $EX['indent'];
    701701        $curticks = 0;
    702         for ($i = $range[0]; $i <= $range[1]; $i ++) {
     702        for ($i = $range[0]; $i <= $range[1]; $i++) {
    703703            $op = $EX['opcodes'][$i];
    704704            if (isset($op['gofrom'])) {
    … …  
    786786    }
    787787    // }}}
    788     function fixOpCode(&$opcodes, $removeTailing = false, $defaultReturnValue = null) // {{{
     788    function fixOpCode($opcodes, $removeTailing = false, $defaultReturnValue = null) // {{{
    789789    {
    790790        $last = count($opcodes) - 1;
    791         for ($i = 0; $i <= $last; $i ++) {
     791        for ($i = 0; $i <= $last; $i++) {
    792792            if (function_exists('xcache_get_fixed_opcode')) {
    793793                $opcodes[$i]['opcode'] = xcache_get_fixed_opcode($opcodes[$i]['opcode'], $i);
    … …  
    837837            }
    838838        }
     839        return $opcodes;
    839840    }
    840841    // }}}
    … …  
    11121113                $this->dasmBasicBlock($EX, array($catchFirst, $catchOpLine));
    11131114                $catchOp = &$opcodes[$catchOpLine];
    1114                 echo $indent, 'catch ('
     1115                echo $indent, "catch ("
    11151116                        , $this->stripNamespace(isset($catchOp['op1']['constant']) ? $catchOp['op1']['constant'] : str($this->getOpVal($catchOp['op1'], $EX)))
    11161117                        , ' '
    … …  
    12771278            $body = ob_get_clean();
    12781279
    1279             echo $indent, 'while (', str($this->getOpVal($firstJmpOp['op1'], $EX)), ") {", PHP_EOL;
     1280            echo $indent, "while (", str($this->getOpVal($firstJmpOp['op1'], $EX)), ") {", PHP_EOL;
    12801281            echo $body;
    12811282            echo $indent, '}', PHP_EOL;
    … …  
    13071308            }
    13081309
    1309             echo $indent, 'foreach (', str($firstJmpOp['fe_src'], $EX), " as $as) {", PHP_EOL;
     1310            echo $indent, "foreach (", str($firstJmpOp['fe_src'], $EX), " as $as) {", PHP_EOL;
    13101311            echo $body;
    13111312            echo $indent, '}', PHP_EOL;
    … …  
    13791380        $opcodes = &$op_array['opcodes'];
    13801381        $last = count($opcodes) - 1;
    1381         for ($i = 0; $i <= $last; $i ++) {
     1382        for ($i = 0; $i <= $last; $i++) {
    13821383            $op = &$opcodes[$i];
    13831384            $op['line'] = $i;
    … …  
    14791480    function &dop_array($op_array, $indent = '') // {{{
    14801481    {
    1481         $this->fixOpCode($op_array['opcodes'], true, $indent == '' ? 1 : null);
     1482        $op_array['opcodes'] = $this->fixOpCode($op_array['opcodes'], true, $indent == '' ? 1 : null);
    14821483        $this->buildJmpInfo($op_array);
    14831484
    … …  
    14871488        $nextbbs = array();
    14881489        $starti = 0;
    1489         for ($i = 1; $i <= $last; $i ++) {
     1490        for ($i = 1; $i <= $last; $i++) {
    14901491            if (isset($opcodes[$i]['jmpins'])
    14911492             || isset($opcodes[$i - 1]['jmpouts'])) {
    … …  
    15501551        $currentSourceLine = null;
    15511552
    1552         for ($i = $range[0]; $i <= $range[1]; $i ++, unsetArray($EX['value2constant'], $currentSourceLine)) {
     1553        for ($i = $range[0]; $i <= $range[1]; $i++, unsetArray($EX['value2constant'], $currentSourceLine)) {
    15531554            // {{{ prepair
    15541555            $op = &$opcodes[$i];
    … …  
    15691570
    15701571            if ($opname == 'UNDEF' || !isset($opname)) {
    1571                 echo 'UNDEF OP:';
     1572                echo '// UNDEF OP:';
    15721573                $this->dumpop($op, $EX);
    15731574                continue;
    … …  
    18231824                        case ZEND_FETCH_STATIC:
    18241825                            $statics = &$EX['op_array']['static_variables'];
     1826                            $name = unquoteName($src);
    18251827                            if ((xcache_get_type($statics[$name]) & IS_LEXICAL_REF)) {
    18261828                                $EX['uses'][] = '&' . str($lvalue);
    … …  
    18301832
    18311833                            $resvar = 'static ' . $lvalue;
    1832                             $name = unquoteName($src);
    18331834                            if (isset($statics[$name])) {
    18341835                                $var = $statics[$name];
    … …  
    20152016                break;
    20162017                // }}}
    2017             case XC_DECLARE_CLASS: 
     2018            case XC_DECLARE_CLASS:
    20182019            case XC_DECLARE_INHERITED_CLASS:
    20192020            case XC_DECLARE_INHERITED_CLASS_DELAYED: // {{{
    20202021                $key = $op1['constant'];
    20212022                if (!isset($this->dc['class_table'][$key])) {
    2022                     echo 'class not found: ', $key, 'existing classes are:', "\n";
     2023                    echo "class not found: ", $key, "\nexisting classes are:\n";
    20232024                    var_dump(array_keys($this->dc['class_table']));
    20242025                    exit;
    … …  
    22872288
    22882289            case XC_BEGIN_SILENCE: // {{{
    2289                 $EX['silence'] ++;
     2290                $EX['silence']++;
    22902291                break;
    22912292                // }}}
    22922293            case XC_END_SILENCE: // {{{
    2293                 $EX['silence'] --;
     2294                $EX['silence']--;
    22942295                $lastresvar = '@' . str($lastresvar, $EX);
    22952296                break;
    … …  
    24092410    {
    24102411        $args = array();
    2411         for ($i = 0; $i < $n; $i ++) {
     2412        for ($i = 0; $i < $n; $i++) {
    24122413            $a = array_pop($EX['argstack']);
    24132414            if (is_array($a)) {
    … …  
    24732474    {
    24742475        for ($i = $range[0]; $i <= $range[1]; ++$i) {
    2475             echo $EX['indent'], $i, "\t"; $this->dumpop($EX['opcodes'][$i], $EX);
     2476            echo $EX['indent'], $i, "\t";
     2477            $this->dumpop($EX['opcodes'][$i], $EX);
    24762478        }
    24772479        echo $EX['indent'], "==", PHP_EOL;
    … …  
    24942496
    24952497        $refrest = false;
    2496         for ($i = 0; $i < $c; $i ++) {
     2498        for ($i = 0; $i < $c; $i++) {
    24972499            if ($i) {
    24982500                echo ', ';
    … …  
    25492551    {
    25502552        if ($EX['uses']) {
    2551             echo ' use(', implode(', ', $EX['uses']), ')';
     2553            echo " use(", implode(', ', $EX['uses']), ')';
    25522554        }
    25532555    }
    … …  
    27592761
    27602762                        switch ($opa['fn_flags'] & ZEND_ACC_PPP_MASK) {
    2761                             case ZEND_ACC_PUBLIC:
    2762                                 $decorations[] = "public";
    2763                                 break;
    2764                             case ZEND_ACC_PRIVATE:
    2765                                 $decorations[] = "private";
    2766                                 break;
    2767                             case ZEND_ACC_PROTECTED:
    2768                                 $decorations[] = "protected";
    2769                                 break;
    2770                             default:
    2771                                 $decorations[] = "<visibility error>";
    2772                                 break;
     2763                        case ZEND_ACC_PUBLIC:
     2764                            $decorations[] = "public";
     2765                            break;
     2766                        case ZEND_ACC_PRIVATE:
     2767                            $decorations[] = "private";
     2768                            break;
     2769                        case ZEND_ACC_PROTECTED:
     2770                            $decorations[] = "protected";
     2771                            break;
     2772                        default:
     2773                            $decorations[] = "<visibility error>";
     2774                            break;
    27732775                        }
    27742776                    }
    … …  
    28542856    function outputUnusedOp() // {{{
    28552857    {
    2856         for ($i = 0; $opname = xcache_get_opcode($i); $i ++) {
    2857             if ($opname == 'UNDEF')  {
     2858        for ($i = 0; $opname = xcache_get_opcode($i); $i++) {
     2859            if ($opname == 'UNDEF') {
    28582860                continue;
    28592861            }
    … …  
    30403042}
    30413043//*/
    3042 foreach (array (
     3044foreach (array(
    30433045    'XC_ADD_INTERFACE' => -1,
    30443046    'XC_ASSIGN_DIM' => -1,
Note: See TracChangeset for help on using the changeset viewer.