RoslynDOM icon indicating copy to clipboard operation
RoslynDOM copied to clipboard

SameIntent incorrect if you add a new PublicAnnotation

Open KathleenDollard opened this issue 10 years ago • 0 comments

  var csharpCode = @"
        using Foo;

        //[[ kad_Test3(val1 = ""Fred"", val2 : 42) ]]
        public class MyClass
        { }
        ";
        var root = RDomCSharpFactory.Factory.GetRootFromString(csharpCode);
        var root2 = root.Copy();
        var class1 =root.Classes.First();
        var class2 = root2.Classes.First();
         class2.PublicAnnotations.AddValue("kad_Test4", "George");
        Assert.IsFalse(class1.PublicAnnotations.SameIntent(class2.PublicAnnotations, false));

Fails because the foreach in PublicAnnotations.SameIntent only traverses first array

KathleenDollard avatar Jul 25 '14 16:07 KathleenDollard