diff options
author | 2015-03-31 17:27:01 +0000 | |
---|---|---|
committer | 2015-03-31 20:12:23 +0000 | |
commit | 012f789c249b00f318d1e9e7e3817531f6dc34e4 (patch) | |
tree | cfb2e28173b8db31d737f75c266c50b1e0017135 /src/test/java/com/google/devtools/build/lib/syntax/FunctionTest.java | |
parent | ab0ca1afa5fdb79fdd2d6602c3db20c737fd8491 (diff) |
Reorder arguments to BuiltinFunction-s
By popular demand from other implementers, reorder BuiltinFunction arguments
so that mandatory named-only arguments come befor optional named-only arguments
rather than after. This will make Skylark internals slightly clearer and less
surprising, at the cost of eschewing a tiny optimization.
--
MOS_MIGRATED_REVID=89978554
Diffstat (limited to 'src/test/java/com/google/devtools/build/lib/syntax/FunctionTest.java')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/syntax/FunctionTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/syntax/FunctionTest.java b/src/test/java/com/google/devtools/build/lib/syntax/FunctionTest.java index 2eea35a1d9..a55fc01401 100644 --- a/src/test/java/com/google/devtools/build/lib/syntax/FunctionTest.java +++ b/src/test/java/com/google/devtools/build/lib/syntax/FunctionTest.java @@ -383,7 +383,7 @@ public class FunctionTest extends AbstractEvaluationTestCase { assertEquals("xyce", env.lookup("v2")); assertEquals("xyzf", env.lookup("v3")); UserDefinedFunction foo = (UserDefinedFunction) env.lookup("foo"); - assertEquals("foo(a, b = \"b\", *, d = \"d\", c)", foo.toString()); + assertEquals("foo(a, b = \"b\", *, c, d = \"d\")", foo.toString()); } @Test |