Page MenuHomePhabricator
Files F15285015

D13862.diff
No OneTemporary
Actions

D13862.diff
View Options

diff --git a/resources/test/diverse_symbols.php b/resources/test/diverse_symbols.php
--- a/resources/test/diverse_symbols.php
+++ b/resources/test/diverse_symbols.php
@@ -14,7 +14,7 @@
function f() {}
-(function() {
+(function () {
// Anonymous function.
});
diff --git a/src/future/exec/__tests__/ExecFutureTestCase.php b/src/future/exec/__tests__/ExecFutureTestCase.php
--- a/src/future/exec/__tests__/ExecFutureTestCase.php
+++ b/src/future/exec/__tests__/ExecFutureTestCase.php
@@ -82,7 +82,7 @@
}
foreach (new FutureIterator($futures) as $future) {
- list ($err) = $future->resolve();
+ list($err) = $future->resolve();
$this->assertTrue($err > 0);
$this->assertTrue($future->getWasKilledByTimeout());
diff --git a/src/lexer/PhutilPHPFragmentLexer.php b/src/lexer/PhutilPHPFragmentLexer.php
--- a/src/lexer/PhutilPHPFragmentLexer.php
+++ b/src/lexer/PhutilPHPFragmentLexer.php
@@ -141,7 +141,10 @@
// if you put a comment between the symbol and the operator, but
// that's a bizarre usage.
array($identifier_ns_pattern.'(?=\s*[\\(])', 'nf'),
- array($identifier_ns_pattern.'(?=\s*::)', 'nc', 'context_attr',
+ array(
+ $identifier_ns_pattern.'(?=\s*::)',
+ 'nc',
+ 'context_attr',
array(
'context' => 'push',
),
@@ -165,17 +168,26 @@
// the attribute or method (e.g., "X::C" or "X::f()").
'context_attr' => array_merge($nonsemantic_rules, array(
array('::', 'o'),
- array($identifier_pattern.'(?=\s*[\\(])', 'nf', '!pop',
+ array(
+ $identifier_pattern.'(?=\s*[\\(])',
+ 'nf',
+ '!pop',
array(
'context' => 'pop',
),
),
- array($identifier_pattern, 'na', '!pop',
+ array(
+ $identifier_pattern,
+ 'na',
+ '!pop',
array(
'context' => 'pop',
),
),
- array('', null, '!pop',
+ array(
+ '',
+ null,
+ '!pop',
array(
'context' => 'discard',
),
diff --git a/src/markup/engine/remarkup/blockrule/PhutilRemarkupBlockRule.php b/src/markup/engine/remarkup/blockrule/PhutilRemarkupBlockRule.php
--- a/src/markup/engine/remarkup/blockrule/PhutilRemarkupBlockRule.php
+++ b/src/markup/engine/remarkup/blockrule/PhutilRemarkupBlockRule.php
@@ -132,7 +132,7 @@
background: #d3d3d3;
margin: 12px 0;',
);
- $cell_attributes = array (
+ $cell_attributes = array(
'style' => 'background: #ffffff;
padding: 3px 6px;',
);
diff --git a/src/markup/engine/remarkup/blockrule/PhutilRemarkupCodeBlockRule.php b/src/markup/engine/remarkup/blockrule/PhutilRemarkupCodeBlockRule.php
--- a/src/markup/engine/remarkup/blockrule/PhutilRemarkupCodeBlockRule.php
+++ b/src/markup/engine/remarkup/blockrule/PhutilRemarkupCodeBlockRule.php
@@ -132,7 +132,7 @@
$name_header = null;
if ($this->getEngine()->isHTMLMailMode()) {
- $header_attributes = array (
+ $header_attributes = array(
'style' => 'padding: 6px 8px;
background: #fdf5d4;
color: rgba(0,0,0,.75);
diff --git a/src/parser/argument/__tests__/PhutilArgumentParserTestCase.php b/src/parser/argument/__tests__/PhutilArgumentParserTestCase.php
--- a/src/parser/argument/__tests__/PhutilArgumentParserTestCase.php
+++ b/src/parser/argument/__tests__/PhutilArgumentParserTestCase.php
@@ -348,7 +348,13 @@
$args = new PhutilArgumentParser(
array(
- 'bin', '--a', 'a', '--b=b', '-c', 'c', '-d=d',
+ 'bin',
+ '--a',
+ 'a',
+ '--b=b',
+ '-c',
+ 'c',
+ '-d=d',
));
$args->parseFull($specs);
@@ -368,7 +374,10 @@
$args = new PhutilArgumentParser(
array(
- 'bin', '-', '--file', '-',
+ 'bin',
+ '-',
+ '--file',
+ '-',
));
$args->parsePartial($specs);
@@ -400,7 +409,12 @@
);
$args = new PhutilArgumentParser(array(
- 'bin', '--eat', 'apple', '--eat', 'pear', '--eat=orange',
+ 'bin',
+ '--eat',
+ 'apple',
+ '--eat',
+ 'pear',
+ '--eat=orange',
));
$args->parseFull($specs);
diff --git a/src/utils/__tests__/PhutilUTF8TestCase.php b/src/utils/__tests__/PhutilUTF8TestCase.php
--- a/src/utils/__tests__/PhutilUTF8TestCase.php
+++ b/src/utils/__tests__/PhutilUTF8TestCase.php
@@ -150,7 +150,9 @@
// This behavior is maybe a little bad, but it seems mostly reasonable,
// at least for latin languages.
array(
- 'Derp, supercalafragalisticexpialadoshus', 30, '...',
+ 'Derp, supercalafragalisticexpialadoshus',
+ 30,
+ '...',
'Derp...',
),
@@ -176,27 +178,39 @@
$cases = array(
array(
"o\xCD\xA0o\xCD\xA0o\xCD\xA0o\xCD\xA0o\xCD\xA0",
- 6, "o\xCD\xA0!",
- 6, "o\xCD\xA0o\xCD\xA0!",
- 6, "o\xCD\xA0o\xCD\xA0o\xCD\xA0o\xCD\xA0o\xCD\xA0",
+ 6,
+ "o\xCD\xA0!",
+ 6,
+ "o\xCD\xA0o\xCD\xA0!",
+ 6,
+ "o\xCD\xA0o\xCD\xA0o\xCD\xA0o\xCD\xA0o\xCD\xA0",
),
array(
"X\xCD\xA0\xCD\xA0\xCD\xA0Y",
- 6, '!',
- 6, "X\xCD\xA0\xCD\xA0\xCD\xA0Y",
- 6, "X\xCD\xA0\xCD\xA0\xCD\xA0Y",
+ 6,
+ '!',
+ 6,
+ "X\xCD\xA0\xCD\xA0\xCD\xA0Y",
+ 6,
+ "X\xCD\xA0\xCD\xA0\xCD\xA0Y",
),
array(
"X\xCD\xA0\xCD\xA0\xCD\xA0YZ",
- 6, '!',
- 5, "X\xCD\xA0\xCD\xA0\xCD\xA0!",
- 2, "X\xCD\xA0\xCD\xA0\xCD\xA0!",
+ 6,
+ '!',
+ 5,
+ "X\xCD\xA0\xCD\xA0\xCD\xA0!",
+ 2,
+ "X\xCD\xA0\xCD\xA0\xCD\xA0!",
),
array(
"\xE2\x98\x83\xE2\x98\x83\xE2\x98\x83\xE2\x98\x83",
- 4, "\xE2\x98\x83!",
- 3, "\xE2\x98\x83\xE2\x98\x83!",
- 3, "\xE2\x98\x83\xE2\x98\x83!",
+ 4,
+ "\xE2\x98\x83!",
+ 3,
+ "\xE2\x98\x83\xE2\x98\x83!",
+ 3,
+ "\xE2\x98\x83\xE2\x98\x83!",
),
);

File Metadata

Mime Type
text/plain
Expires
Wed, Mar 5, 11:18 AM (1 d, 12 h ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
7225313
Default Alt Text
D13862.diff (6 KB)

Event Timeline

Phabricator · Built by Phacility