@@ -176,8 +176,7 @@ |
||
176 | 176 | } else { |
177 | 177 | $escapedData .= $ch; |
178 | 178 | } |
179 | - } |
|
180 | - else if ($ii < 128) { |
|
179 | + } else if ($ii < 128) { |
|
181 | 180 | /// @todo shall we replace this with a (supposedly) faster str_replace? |
182 | 181 | /// @todo to be 'print safe', should we encode as well character 127 (DEL) ? |
183 | 182 | switch ($ii) { |
@@ -591,8 +591,9 @@ |
||
591 | 591 | } |
592 | 592 | foreach ($parsVariations as $i => $pars) { |
593 | 593 | $innerCode .= "if (\$paramCount == " . count($pars) . ") \$retval = {$catchWarnings}$realFuncName(" . implode(',', $pars) . ");\n"; |
594 | - if ($i < (count($parsVariations) - 1)) |
|
595 | - $innerCode .= "else\n"; |
|
594 | + if ($i < (count($parsVariations) - 1)) { |
|
595 | + $innerCode .= "else\n"; |
|
596 | + } |
|
596 | 597 | } |
597 | 598 | $innerCode .= "if (is_a(\$retval, '{$namespace}Response')) return \$retval; else\n"; |
598 | 599 | if ($funcDesc['returns'] == Value::$xmlrpcDateTime || $funcDesc['returns'] == Value::$xmlrpcBase64) { |
@@ -109,14 +109,16 @@ |
||
109 | 109 | |
110 | 110 | $this->coverageScriptUrl = 'http://' . $this->args['HTTPSERVER'] . '/' . str_replace('/demo/server/server.php', 'tests/phpunit_coverage.php', $this->args['HTTPURI']); |
111 | 111 | |
112 | - if ($this->args['DEBUG'] == 1) |
|
113 | - ob_start(); |
|
112 | + if ($this->args['DEBUG'] == 1) { |
|
113 | + ob_start(); |
|
114 | + } |
|
114 | 115 | } |
115 | 116 | |
116 | 117 | protected function tear_down() |
117 | 118 | { |
118 | - if ($this->args['DEBUG'] != 1) |
|
119 | - return; |
|
119 | + if ($this->args['DEBUG'] != 1) { |
|
120 | + return; |
|
121 | + } |
|
120 | 122 | $out = ob_get_clean(); |
121 | 123 | $status = $this->getStatus(); |
122 | 124 | if ($status == BaseTestRunner::STATUS_ERROR |