Merge pull request #9 from skbkontur/d.kop/remove-string-interning

Remove string interning when forming custom field name
skbkontur · Dec 21, 2023 · 5a64ef8 · 5a64ef8
2 parents 4fcc352 + a2d1bf7
commit 5a64ef8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.github/workflows/actions.yml b/.github/workflows/actions.yml
@@ -17,6 +17,9 @@ jobs:
         uses: actions/setup-dotnet@v3
         with:
           global-json-file: global.json
+          dotnet-version: | 
+            3.1.x
+            6.0.x
 
       - name: Install dependencies
         run: dotnet restore ${{ env.SOLUTION_FILE }} --verbosity minimal && dotnet tool restore

diff --git a/Mutators/InternalExpressionExtensions.cs b/Mutators/InternalExpressionExtensions.cs
@@ -102,7 +102,7 @@ public static string CustomFieldName(this Expression node)
         {
             var result = new StringBuilder();
             BuildCustomFieldName(node, result);
-            return string.Intern(result.ToString());
+            return result.ToString();
         }
 
         public static Expression ExtendSelectMany(this Expression expression)
@@ -308,7 +308,7 @@ private static void BuildCustomFieldName(Expression node, StringBuilder result)
                 BuildCustomFieldName(memberExpression.Expression, result);
                 if (result.Length > 0)
                     result.Append("ё");
-                result.Append(string.Intern(memberExpression.Member.Name));
+                result.Append(memberExpression.Member.Name);
                 break;
             case ExpressionType.Call:
                 var methodCallExpression = (MethodCallExpression)node;