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
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 2eea35a..a55fc01 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 @@
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
