Merge pull request #6797 from CyrusNajmabadi/smartTagDuplication

Don't add nested code actions *and* their parent to the smart tag list.

Author: CyrusNajmabadi

src/EditorFeatures/Core/Implementation/Suggestions/SuggestedActionsSourceProvider.cs

@@ -220,6 +220,7 @@ private SuggestedActionSet InlineActions(SuggestedActionSet actionSet)
                         }
 
                         newActions.AddRange(childActionSets[0].Actions);
+                        continue;
                     }
 
                     newActions.Add(action);

src/Workspaces/Core/Portable/CodeActions/CodeAction.cs

@@ -331,9 +331,9 @@ protected override Task<Document> GetChangedDocumentAsync(CancellationToken canc
                 return Task.FromResult<Document>(null);
             }
 
-            private static string ComputeEquivalenceKey(IEnumerable<CodeAction> nestedActions)
+            private static string ComputeEquivalenceKey(ImmutableArray<CodeAction> nestedActions)
             {
-                if (nestedActions == null)
+                if (nestedActions.IsDefault)
                 {
                     return null;
                 }
@@ -343,7 +343,7 @@ private static string ComputeEquivalenceKey(IEnumerable<CodeAction> nestedAction
                 {
                     foreach (var action in nestedActions)
                     {
-                        equivalenceKey.Append(action.EquivalenceKey ?? action.GetHashCode().ToString() + ";");
+                        equivalenceKey.Append((action.EquivalenceKey ?? action.GetHashCode().ToString()) + ";");
                     }
 
                     return equivalenceKey.Length > 0 ? equivalenceKey.ToString() : null;