Change argument label syntax to use '=' instead of ':'

For consistency with other forms of value assignment.
'a: b' syntax is now purely for compile-time type constraints.

Author: emillaine

docs/book/functions.md

@@ -41,16 +41,16 @@ void foo(public int b) {
 }
 
 void main() {
-    foo(a: 42); // prints "a"
-    foo(b: 42); // prints "b"
+    foo(a = 42); // prints "a"
+    foo(b = 42); // prints "b"
 }
 ```
 
 Functions can return multiple values using tuples:
 
 ```cs
 (int a, bool b) foo() {
-    return (a: 42, b: true);
+    return (a = 42, b = true);
 }
 
 void main() {

docs/book/introduction.md

@@ -247,7 +247,7 @@ active type of a union, etc. Should follow the "pay only for what you use" princ
 
 __Named arguments__ as an optional way to avoid cryptic call sites like `foo(true, false)` by labeling the arguments with the corresponding
 parameter name, with the compiler checking that the argument labels match the parameter names, e.g.
-`foo(a: true, b: false)`. Consequently, named arguments don't have to be in same order as the parameters. This also
+`foo(verbose = true, ignoreErrors = false)`. Consequently, named arguments don't have to be in same order as the parameters. This also
 enables function overloading on parameter names.
 
 __Defer statement__ to allow declaring arbitrary code, such as resource cleanup functions, to be executed when exiting the

docs/book/structs.md

@@ -25,8 +25,8 @@ struct Person {
 }
 
 void main() {
-    var foo = Person(name: "Foo", age: 42, friend: null);
-    var bar = Person(name: "Bar", age: 42, friend: foo);
+    var foo = Person(name = "Foo", age = 42, friend = null);
+    var bar = Person(name = "Bar", age = 42, friend = foo);
     foo.friend = bar;
 
     foo.moveLeft(); // foo.x is now -0.1

examples/asteroids/missile.cx

@@ -30,6 +30,6 @@ struct Missile {
     }
 
     Vector2 tailPosition() {
-        return (position - Vector2(missileSize, 0)).rotated(origin: position, angle: direction);
+        return (position - Vector2(missileSize, 0)).rotated(origin = position, angle = direction);
     }
 }

examples/asteroids/player.cx

@@ -25,9 +25,9 @@ struct Player {
         var b = Vector2(position.x - playerSize, position.y + playerSize);
         var c = Vector2(position.x + playerSize + playerSize, position.y);
 
-        a = a.rotated(origin: position, angle: direction);
-        b = b.rotated(origin: position, angle: direction);
-        c = c.rotated(origin: position, angle: direction);
+        a = a.rotated(origin = position, angle = direction);
+        b = b.rotated(origin = position, angle = direction);
+        c = c.rotated(origin = position, angle = direction);
 
         for (var i in -1...1) {
             for (var j in -1...1) {

examples/opengl/opengl.cx

@@ -46,7 +46,7 @@ int main()
     glGetProgramiv(program, GL_INFO_LOG_LENGTH, &infoLogLength);
     if (infoLogLength > 0)
     {
-        var infoLog = StringBuffer(uninitializedSize: infoLogLength + 1);
+        var infoLog = StringBuffer(uninitializedSize = infoLogLength + 1);
         glGetProgramInfoLog(program, infoLogLength, null, infoLog.data());
         println(infoLog);
     }
@@ -100,7 +100,7 @@ void LoadShader(GLuint shaderID, string path)
     glGetShaderiv(shaderID, GL_INFO_LOG_LENGTH, &infoLogLength);
     if (infoLogLength > 0)
     {
-        var infoLog = StringBuffer(uninitializedSize: infoLogLength + 1);
+        var infoLog = StringBuffer(uninitializedSize = infoLogLength + 1);
         glGetShaderInfoLog(shaderID, infoLogLength, null, infoLog.data());
         println(infoLog);
     }

src/parser/parse.cpp

@@ -119,7 +119,7 @@ void Parser::parseStmtTerminator(const char* contextInfo) {
 
 /// argument-list ::= '(' ')' | '(' nonempty-argument-list ')'
 /// nonempty-argument-list ::= argument | nonempty-argument-list ',' argument
-/// argument ::= (id ':')? expr
+/// argument ::= (id '=')? expr
 std::vector<NamedValue> Parser::parseArgumentList(bool allowEmpty) {
     parse(Token::LeftParen);
     std::vector<NamedValue> args;
@@ -132,7 +132,7 @@ std::vector<NamedValue> Parser::parseArgumentList(bool allowEmpty) {
     do {
         std::string name;
         Location location = Location();
-        if (lookAhead(1) == Token::Colon) {
+        if (lookAhead(1) == Token::Assignment) {
             auto result = parse(Token::Identifier);
             name = result.getString().str();
             location = result.getLocation();
@@ -275,7 +275,7 @@ ArrayLiteralExpr* Parser::parseArrayLiteral() {
 
 /// tuple-literal ::= '(' tuple-literal-elements ')'
 /// tuple-literal-elements ::= tuple-literal-element | tuple-literal-elements ',' tuple-literal-element
-/// tuple-literal-element ::= (id ':')? expr
+/// tuple-literal-element ::= (id '=')? expr
 /// paren-expr ::= '(' expr ')'
 Expr* Parser::parseTupleLiteralOrParenExpr() {
     ASSERT(currentToken() == Token::LeftParen);
@@ -524,6 +524,7 @@ LambdaExpr* Parser::parseLambdaExpr() {
     return lambda;
 }
 
+// TODO: change to: 'if' expr 'then' expr 'else' expr
 /// if-expr ::= expr '?' expr ':' expr
 IfExpr* Parser::parseIfExpr(Expr* condition) {
     ASSERT(currentToken() == Token::QuestionMark);

std/FileStream.cx

@@ -24,9 +24,9 @@ struct OutputFileStream {
 }
 
 OutputFileStream stdout() {
-    return OutputFileStream(fileDescriptor: 1);
+    return OutputFileStream(fileDescriptor = 1);
 }
 
 OutputFileStream stderr() {
-    return OutputFileStream(fileDescriptor: 2);
+    return OutputFileStream(fileDescriptor = 2);
 }

std/List.cx

@@ -18,7 +18,7 @@ struct List<Element> {
 
     /// Initializes an list containing the elements of the given array.
     List(Element[] elements) {
-        init(capacity: elements.size());
+        init(capacity = elements.size());
 
         // TODO: When array references can be iterated, iterate 'elements' directly.
         // TODO: Add push() overload for pushing all elements from an array reference.
@@ -99,7 +99,7 @@ struct List<Element> {
     /// Ensures that the capacity is large enough to store the given number of elements.
     void reserve(int minimumCapacity) {
         if (minimumCapacity > capacity) {
-            var newBuffer = allocateArray<Element>(size: minimumCapacity);
+            var newBuffer = allocateArray<Element>(size = minimumCapacity);
 
             for (var index in 0..size) {
                 var source = &buffer[index];
@@ -189,7 +189,7 @@ struct List<Element> {
     }
 
     List<Output> map<Output>(Output(Element*) transform) {
-        var output = List<Output>(capacity: this.size);
+        var output = List<Output>(capacity = this.size);
 
         for (var element in this) {
             output.push(transform(element));
@@ -199,7 +199,7 @@ struct List<Element> {
     }
 
     List<Output> map<Output>(Output(Element) transform) {
-        var output = List<Output>(capacity: this.size);
+        var output = List<Output>(capacity = this.size);
 
         for (var element in this) {
             output.push(transform(element));

std/StringBuffer.cx

@@ -14,7 +14,7 @@ struct StringBuffer: Comparable, Hashable, Printable {
     }
 
     StringBuffer(string s) {
-        characters = List(capacity: s.size() + 1);
+        characters = List(capacity = s.size() + 1);
 
         for (var ch in s) {
             characters.push(ch);
@@ -40,7 +40,7 @@ struct StringBuffer: Comparable, Hashable, Printable {
 
     /// Initializes the buffer to contain the given number of uninitialized bytes.
     StringBuffer(public int uninitializedSize) {
-        characters = List(uninitializedSize: uninitializedSize + 1);
+        characters = List(uninitializedSize = uninitializedSize + 1);
         characters[uninitializedSize] = '\0';
     }