[swift2objc] Add support for mutating functions

pkgs/swift2objc/lib/src/ast/declarations/compounds/members/method_declaration.dart

@@ -52,20 +52,20 @@ class MethodDeclaration extends AstNode
         for (final p in params) p.name,
       ].join(':');
 
-  MethodDeclaration({
-    required this.id,
-    required this.name,
-    required this.returnType,
-    required this.params,
-    this.typeParams = const [],
-    this.hasObjCAnnotation = false,
-    this.statements = const [],
-    this.isStatic = false,
-    this.isOverriding = false,
-    this.throws = false,
-    this.async = false,
-    this.mutating = false
-  }) : assert(!isStatic || !isOverriding);
+  MethodDeclaration(
+      {required this.id,
+      required this.name,
+      required this.returnType,
+      required this.params,
+      this.typeParams = const [],
+      this.hasObjCAnnotation = false,
+      this.statements = const [],
+      this.isStatic = false,
+      this.isOverriding = false,
+      this.throws = false,
+      this.async = false,
+      this.mutating = false})
+      : assert(!isStatic || !isOverriding);
 
   @override
   void visit(Visitation visitation) => visitation.visitMethodDeclaration(this);
@@ -77,4 +77,5 @@ class MethodDeclaration extends AstNode
     visitor.visitAll(typeParams);
     visitor.visit(returnType);
   }
+
 }

pkgs/swift2objc/lib/src/parser/parsers/declaration_parsers/parse_function_declaration.dart

@@ -36,32 +36,32 @@ MethodDeclaration parseMethodDeclaration(
   final info =
       parseFunctionInfo(methodSymbolJson['declarationFragments'], symbolgraph);
   return MethodDeclaration(
-    id: parseSymbolId(methodSymbolJson),
-    name: parseSymbolName(methodSymbolJson),
-    returnType: _parseFunctionReturnType(methodSymbolJson, symbolgraph),
-    params: info.params,
-    hasObjCAnnotation: parseSymbolHasObjcAnnotation(methodSymbolJson),
-    isStatic: isStatic,
-    throws: info.throws,
-    async: info.async,
-    mutating: _parseFunctionIsMutating(methodSymbolJson)
-  );
+      id: parseSymbolId(methodSymbolJson),
+      name: parseSymbolName(methodSymbolJson),
+      returnType: _parseFunctionReturnType(methodSymbolJson, symbolgraph),
+      params: info.params,
+      hasObjCAnnotation: parseSymbolHasObjcAnnotation(methodSymbolJson),
+      isStatic: isStatic,
+      throws: info.throws,
+      async: info.async,
+      mutating: info.mutating);
 }
 
 typedef ParsedFunctionInfo = ({
   List<Parameter> params,
   bool throws,
   bool async,
+  bool mutating,
 });
 
 ParsedFunctionInfo parseFunctionInfo(
   Json declarationFragments,
   ParsedSymbolgraph symbolgraph,
 ) {
   // `declarationFragments` describes each part of the function declaration,
-  // things like the `func` keyword, brackets, spaces, etc. We only care about
-  // the parameter fragments and annotations here, and they always appear in
-  // this order:
+  // things like the `func` keyword, brackets, spaces, etc.
+  // For the most part, We only care about the parameter fragments and
+  // annotations here, and they always appear in this order:
   // [
   //   ..., '(',
   //   externalParam, ' ', internalParam, ': ', type..., ', '
@@ -81,6 +81,7 @@ ParsedFunctionInfo parseFunctionInfo(
   );
 
   var tokens = TokenList(declarationFragments);
+
   String? maybeConsume(String kind) {
     if (tokens.isEmpty) return null;
     final spelling = getSpellingForKind(tokens[0], kind);
@@ -140,16 +141,14 @@ ParsedFunctionInfo parseFunctionInfo(
     params: parameters,
     throws: annotations.contains('throws'),
     async: annotations.contains('async'),
+    mutating: declarationFragments
+        .where((j) =>
+            j['kind'].get<String?>() == 'keyword' &&
+            j['spelling'].get<String?>() == 'mutating')
+        .isNotEmpty
   );
 }
 
-bool _parseFunctionIsMutating(Json methodSymbolJson) {
-  return methodSymbolJson['declarationFragments']
-  .where((j) => j['kind'].get<String?>() == 'keyword' 
-    && j['spelling'].get<String?>() == 'mutating')
-  .isNotEmpty;
-}
-
 ReferredType _parseFunctionReturnType(
   Json methodSymbolJson,
   ParsedSymbolgraph symbolgraph,

pkgs/swift2objc/test/unit/parse_function_info_test.dart

@@ -456,6 +456,96 @@ void main() {
       expect(info.throws, isTrue);
       expect(info.async, isTrue);
     });
+
+    test('Mutating Function that throws', () {
+      final json = Json(jsonDecode('''
+        [
+                {
+                    "kind": "keyword",
+                    "spelling": "mutating"
+                },
+                {
+                    "kind": "text",
+                    "spelling": " "
+                },
+                {
+                    "kind": "keyword",
+                    "spelling": "func"
+                },
+                {
+                    "kind": "text",
+                    "spelling": " "
+                },
+                {
+                    "kind": "identifier",
+                    "spelling": "moveBy"
+                },
+                {
+                    "kind": "text",
+                    "spelling": "("
+                },
+                {
+                    "kind": "externalParam",
+                    "spelling": "x"
+                },
+                {
+                    "kind": "text",
+                    "spelling": " "
+                },
+                {
+                    "kind": "internalParam",
+                    "spelling": "deltaX"
+                },
+                {
+                    "kind": "text",
+                    "spelling": ": "
+                },
+                {
+                    "kind": "typeIdentifier",
+                    "spelling": "Double",
+                    "preciseIdentifier": "s:Sd"
+                },
+                {
+                    "kind": "text",
+                    "spelling": ", "
+                },
+                {
+                    "kind": "externalParam",
+                    "spelling": "y"
+                },
+                {
+                    "kind": "text",
+                    "spelling": " "
+                },
+                {
+                    "kind": "internalParam",
+                    "spelling": "deltaY"
+                },
+                {
+                    "kind": "text",
+                    "spelling": ": "
+                },
+                {
+                    "kind": "typeIdentifier",
+                    "spelling": "Double",
+                    "preciseIdentifier": "s:Sd"
+                },
+                {
+                    "kind": "text",
+                    "spelling": ") "
+                },
+                {
+                    "kind": "keyword",
+                    "spelling": "throws"
+                }
+            ]
+        '''));
+
+      final info = parseFunctionInfo(json, emptySymbolgraph);
+
+      expect(info.throws, isTrue);
+      expect(info.mutating, isTrue);
+    });
   });
 
   group('Invalid json', () {