From 4e94c756cba90e11ed849680aa0bade6309f577c Mon Sep 17 00:00:00 2001
From: Sneha Shivakumar <snshivakumar@microsoft.com>
Date: Tue, 27 Sep 2022 17:33:26 -0700
Subject: [PATCH 1/6] Add JsonCloneSettings to disable copy annotations

---
 .../Linq/AnnotationsTests.cs                   | 15 +++++++++++++++
 Src/Newtonsoft.Json/Linq/JArray.cs             |  2 +-
 Src/Newtonsoft.Json/Linq/JConstructor.cs       |  2 +-
 Src/Newtonsoft.Json/Linq/JContainer.cs         |  7 +++++--
 Src/Newtonsoft.Json/Linq/JObject.cs            |  8 ++++----
 Src/Newtonsoft.Json/Linq/JProperty.cs          |  2 +-
 Src/Newtonsoft.Json/Linq/JRaw.cs               |  2 +-
 Src/Newtonsoft.Json/Linq/JToken.cs             | 12 +++++++++++-
 Src/Newtonsoft.Json/Linq/JValue.cs             | 12 ++++++++----
 Src/Newtonsoft.Json/Linq/JsonCloneSettings.cs  | 18 ++++++++++++++++++
 10 files changed, 65 insertions(+), 15 deletions(-)
 create mode 100644 Src/Newtonsoft.Json/Linq/JsonCloneSettings.cs

diff --git a/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs b/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs
index 28af83764..e86f04810 100644
--- a/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs
+++ b/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs
@@ -308,6 +308,21 @@ private void AssertCloneCopy<T>(JToken t, T annotation) where T : class
             Assert.AreEqual(annotation, t.DeepClone().Annotation<T>());
         }
 
+        [Test]
+        public void MultipleAnnotationsAreNotCopiedWithSetting()
+        {
+            Version version = new Version(1, 2, 3, 4);
+
+            JObject o = new JObject();
+            o.AddAnnotation("string!");
+            o.AddAnnotation(version);
+
+            JsonCloneSettings settings = new JsonCloneSettings() { CopyAnnotations = false };
+            JObject o2 = (JObject)o.DeepClone(settings);
+            Assert.IsNull(o2.Annotation<string>());
+            Assert.AreEqual(0, o2.Annotations<Version>().Count());
+        }
+
 #if !NET20
         [Test]
         public void Example()
diff --git a/Src/Newtonsoft.Json/Linq/JArray.cs b/Src/Newtonsoft.Json/Linq/JArray.cs
index 241a56045..5446a9b14 100644
--- a/Src/Newtonsoft.Json/Linq/JArray.cs
+++ b/Src/Newtonsoft.Json/Linq/JArray.cs
@@ -93,7 +93,7 @@ internal override bool DeepEquals(JToken node)
             return (node is JArray t && ContentsEqual(t));
         }
 
-        internal override JToken CloneToken()
+        internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
             return new JArray(this);
         }
diff --git a/Src/Newtonsoft.Json/Linq/JConstructor.cs b/Src/Newtonsoft.Json/Linq/JConstructor.cs
index f2aa86182..4626470ef 100644
--- a/Src/Newtonsoft.Json/Linq/JConstructor.cs
+++ b/Src/Newtonsoft.Json/Linq/JConstructor.cs
@@ -147,7 +147,7 @@ internal override bool DeepEquals(JToken node)
             return (node is JConstructor c && _name == c.Name && ContentsEqual(c));
         }
 
-        internal override JToken CloneToken()
+        internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
             return new JConstructor(this);
         }
diff --git a/Src/Newtonsoft.Json/Linq/JContainer.cs b/Src/Newtonsoft.Json/Linq/JContainer.cs
index f9447cd5d..565f8b3df 100644
--- a/Src/Newtonsoft.Json/Linq/JContainer.cs
+++ b/Src/Newtonsoft.Json/Linq/JContainer.cs
@@ -106,7 +106,7 @@ internal JContainer()
         {
         }
 
-        internal JContainer(JContainer other)
+        internal JContainer(JContainer other, JsonCloneSettings? settings = null)
             : this()
         {
             ValidationUtils.ArgumentNotNull(other, nameof(other));
@@ -118,7 +118,10 @@ internal JContainer(JContainer other)
                 i++;
             }
 
-            CopyAnnotations(this, other);
+            if (settings?.CopyAnnotations ?? true) 
+            {
+                CopyAnnotations(this, other);
+            }
         }
 
         internal void CheckReentrancy()
diff --git a/Src/Newtonsoft.Json/Linq/JObject.cs b/Src/Newtonsoft.Json/Linq/JObject.cs
index 872c8ae7c..96a49f2bf 100644
--- a/Src/Newtonsoft.Json/Linq/JObject.cs
+++ b/Src/Newtonsoft.Json/Linq/JObject.cs
@@ -92,8 +92,8 @@ public JObject()
         /// Initializes a new instance of the <see cref="JObject"/> class from another <see cref="JObject"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JObject"/> object to copy from.</param>
-        public JObject(JObject other)
-            : base(other)
+        public JObject(JObject other, JsonCloneSettings? settings = null)
+            : base(other, settings)
         {
         }
 
@@ -244,9 +244,9 @@ internal void InternalPropertyChanging(JProperty childProperty)
 #endif
         }
 
-        internal override JToken CloneToken()
+        internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
-            return new JObject(this);
+            return new JObject(this, settings);
         }
 
         /// <summary>
diff --git a/Src/Newtonsoft.Json/Linq/JProperty.cs b/Src/Newtonsoft.Json/Linq/JProperty.cs
index 3c738cf52..d54276890 100644
--- a/Src/Newtonsoft.Json/Linq/JProperty.cs
+++ b/Src/Newtonsoft.Json/Linq/JProperty.cs
@@ -282,7 +282,7 @@ internal override bool DeepEquals(JToken node)
             return (node is JProperty t && _name == t.Name && ContentsEqual(t));
         }
 
-        internal override JToken CloneToken()
+        internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
             return new JProperty(this);
         }
diff --git a/Src/Newtonsoft.Json/Linq/JRaw.cs b/Src/Newtonsoft.Json/Linq/JRaw.cs
index 02ca51fd1..6be6cdfa0 100644
--- a/Src/Newtonsoft.Json/Linq/JRaw.cs
+++ b/Src/Newtonsoft.Json/Linq/JRaw.cs
@@ -67,7 +67,7 @@ public static JRaw Create(JsonReader reader)
             }
         }
 
-        internal override JToken CloneToken()
+        internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
             return new JRaw(this);
         }
diff --git a/Src/Newtonsoft.Json/Linq/JToken.cs b/Src/Newtonsoft.Json/Linq/JToken.cs
index 336d748aa..f6bb7739e 100644
--- a/Src/Newtonsoft.Json/Linq/JToken.cs
+++ b/Src/Newtonsoft.Json/Linq/JToken.cs
@@ -131,7 +131,7 @@ public JToken Root
             }
         }
 
-        internal abstract JToken CloneToken();
+        internal abstract JToken CloneToken(JsonCloneSettings? settings = null);
         internal abstract bool DeepEquals(JToken node);
 
         /// <summary>
@@ -2446,6 +2446,16 @@ public JToken DeepClone()
             return CloneToken();
         }
 
+        /// <summary>
+        /// Creates a new instance of the <see cref="JToken"/>. All child tokens are recursively cloned.
+        /// </summary>
+        /// <param name="settings">A flag to indicate whether to clone annotations when performing deep clone on a JToken.</param>
+        /// <returns>A new instance of the <see cref="JToken"/>.</returns>
+        public JToken DeepClone(JsonCloneSettings settings)
+        {
+            return CloneToken(settings);
+        }
+
         /// <summary>
         /// Adds an object to the annotation list of this <see cref="JToken"/>.
         /// </summary>
diff --git a/Src/Newtonsoft.Json/Linq/JValue.cs b/Src/Newtonsoft.Json/Linq/JValue.cs
index 09ed1fd42..5cb60b862 100644
--- a/Src/Newtonsoft.Json/Linq/JValue.cs
+++ b/Src/Newtonsoft.Json/Linq/JValue.cs
@@ -61,10 +61,14 @@ internal JValue(object? value, JTokenType type)
         /// Initializes a new instance of the <see cref="JValue"/> class from another <see cref="JValue"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JValue"/> object to copy from.</param>
-        public JValue(JValue other)
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object that specifies the settings used when cloning JSON.</param>
+        public JValue(JValue other, JsonCloneSettings? settings = null)
             : this(other.Value, other.Type)
         {
-            CopyAnnotations(this, other);
+            if (settings?.CopyAnnotations ?? true) 
+            {
+                CopyAnnotations(this, other);
+            }
         }
 
         /// <summary>
@@ -557,9 +561,9 @@ private static bool Operation(ExpressionType operation, object? objA, object? ob
         }
 #endif
 
-        internal override JToken CloneToken()
+        internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
-            return new JValue(this);
+            return new JValue(this, settings);
         }
 
         /// <summary>
diff --git a/Src/Newtonsoft.Json/Linq/JsonCloneSettings.cs b/Src/Newtonsoft.Json/Linq/JsonCloneSettings.cs
new file mode 100644
index 000000000..cae4507ee
--- /dev/null
+++ b/Src/Newtonsoft.Json/Linq/JsonCloneSettings.cs
@@ -0,0 +1,18 @@
+using System;
+
+namespace Newtonsoft.Json.Linq
+{
+    /// <summary>
+    /// Specifies the settings used when selecting JSON.
+    /// </summary>
+    public class JsonCloneSettings
+    {
+        /// <summary>
+        /// Gets or sets a flag that indicates whether to copy annotations when cloning a JToken.
+        /// </summary>
+        /// <value>
+        /// A flag that indicates whether to copy annotations when cloning a JToken.
+        /// </value>
+        public bool CopyAnnotations { get; set; }
+    }
+}
\ No newline at end of file

From cb5650ceee69edc6efb07eca8c149ab777513298 Mon Sep 17 00:00:00 2001
From: Sneha Shivakumar <snshivakumar@microsoft.com>
Date: Wed, 5 Oct 2022 17:40:00 -0700
Subject: [PATCH 2/6] Fix comment

---
 Src/Newtonsoft.Json/Linq/JObject.cs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/Src/Newtonsoft.Json/Linq/JObject.cs b/Src/Newtonsoft.Json/Linq/JObject.cs
index 96a49f2bf..f6727e1bd 100644
--- a/Src/Newtonsoft.Json/Linq/JObject.cs
+++ b/Src/Newtonsoft.Json/Linq/JObject.cs
@@ -92,6 +92,7 @@ public JObject()
         /// Initializes a new instance of the <see cref="JObject"/> class from another <see cref="JObject"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JObject"/> object to copy from.</param>
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object indicating whether to clone annotations when cloning a JToken.</param>
         public JObject(JObject other, JsonCloneSettings? settings = null)
             : base(other, settings)
         {

From 6b0c863c4d37c5de20ac7ffb228daa73607a4de6 Mon Sep 17 00:00:00 2001
From: Sneha Shivakumar <snshivakumar@microsoft.com>
Date: Wed, 26 Oct 2022 15:26:37 -0700
Subject: [PATCH 3/6] Update all other class constructors to consume new
 settings object.

---
 Src/Newtonsoft.Json/Linq/JArray.cs       | 9 +++++----
 Src/Newtonsoft.Json/Linq/JConstructor.cs | 4 ++--
 Src/Newtonsoft.Json/Linq/JProperty.cs    | 6 +++---
 Src/Newtonsoft.Json/Linq/JRaw.cs         | 6 +++---
 4 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/Src/Newtonsoft.Json/Linq/JArray.cs b/Src/Newtonsoft.Json/Linq/JArray.cs
index 5446a9b14..29a4c692a 100644
--- a/Src/Newtonsoft.Json/Linq/JArray.cs
+++ b/Src/Newtonsoft.Json/Linq/JArray.cs
@@ -65,8 +65,9 @@ public JArray()
         /// Initializes a new instance of the <see cref="JArray"/> class from another <see cref="JArray"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JArray"/> object to copy from.</param>
-        public JArray(JArray other)
-            : base(other)
+        /// <param name="settings">
+        public JArray(JArray other, JsonCloneSettings? settings = null)
+            : base(other, settings)
         {
         }
 
@@ -78,7 +79,7 @@ public JArray(params object[] content)
             : this((object)content)
         {
         }
-
+        
         /// <summary>
         /// Initializes a new instance of the <see cref="JArray"/> class with the specified content.
         /// </summary>
@@ -95,7 +96,7 @@ internal override bool DeepEquals(JToken node)
 
         internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
-            return new JArray(this);
+            return new JArray(this, settings);
         }
 
         /// <summary>
diff --git a/Src/Newtonsoft.Json/Linq/JConstructor.cs b/Src/Newtonsoft.Json/Linq/JConstructor.cs
index 4626470ef..937a6edb0 100644
--- a/Src/Newtonsoft.Json/Linq/JConstructor.cs
+++ b/Src/Newtonsoft.Json/Linq/JConstructor.cs
@@ -96,8 +96,8 @@ public JConstructor()
         /// Initializes a new instance of the <see cref="JConstructor"/> class from another <see cref="JConstructor"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JConstructor"/> object to copy from.</param>
-        public JConstructor(JConstructor other)
-            : base(other)
+        public JConstructor(JConstructor other, JsonCloneSettings? settings = null)
+            : base(other, settings)
         {
             _name = other.Name;
         }
diff --git a/Src/Newtonsoft.Json/Linq/JProperty.cs b/Src/Newtonsoft.Json/Linq/JProperty.cs
index d54276890..11ae7c399 100644
--- a/Src/Newtonsoft.Json/Linq/JProperty.cs
+++ b/Src/Newtonsoft.Json/Linq/JProperty.cs
@@ -186,8 +186,8 @@ public JToken Value
         /// Initializes a new instance of the <see cref="JProperty"/> class from another <see cref="JProperty"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JProperty"/> object to copy from.</param>
-        public JProperty(JProperty other)
-            : base(other)
+        public JProperty(JProperty other, JsonCloneSettings? settings = null)
+            : base(other, settings)
         {
             _name = other.Name;
         }
@@ -284,7 +284,7 @@ internal override bool DeepEquals(JToken node)
 
         internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
-            return new JProperty(this);
+            return new JProperty(this, settings);
         }
 
         /// <summary>
diff --git a/Src/Newtonsoft.Json/Linq/JRaw.cs b/Src/Newtonsoft.Json/Linq/JRaw.cs
index 6be6cdfa0..c0694e793 100644
--- a/Src/Newtonsoft.Json/Linq/JRaw.cs
+++ b/Src/Newtonsoft.Json/Linq/JRaw.cs
@@ -37,8 +37,8 @@ public partial class JRaw : JValue
         /// Initializes a new instance of the <see cref="JRaw"/> class from another <see cref="JRaw"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JRaw"/> object to copy from.</param>
-        public JRaw(JRaw other)
-            : base(other)
+        public JRaw(JRaw other, JsonCloneSettings? settings = null)
+            : base(other, settings)
         {
         }
 
@@ -69,7 +69,7 @@ public static JRaw Create(JsonReader reader)
 
         internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
-            return new JRaw(this);
+            return new JRaw(this, settings);
         }
     }
 }
\ No newline at end of file

From 83fbf1d35b8240b74f05cb5995c8f7fe72be3059 Mon Sep 17 00:00:00 2001
From: Sneha Shivakumar <snshivakumar@microsoft.com>
Date: Wed, 26 Oct 2022 15:54:37 -0700
Subject: [PATCH 4/6] Fixing comments.

---
 Src/Newtonsoft.Json/Linq/JArray.cs       | 2 +-
 Src/Newtonsoft.Json/Linq/JConstructor.cs | 1 +
 Src/Newtonsoft.Json/Linq/JProperty.cs    | 1 +
 Src/Newtonsoft.Json/Linq/JRaw.cs         | 1 +
 Src/Newtonsoft.Json/Linq/JToken.cs       | 2 +-
 Src/Newtonsoft.Json/Linq/JValue.cs       | 2 +-
 6 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/Src/Newtonsoft.Json/Linq/JArray.cs b/Src/Newtonsoft.Json/Linq/JArray.cs
index 29a4c692a..d427b1be2 100644
--- a/Src/Newtonsoft.Json/Linq/JArray.cs
+++ b/Src/Newtonsoft.Json/Linq/JArray.cs
@@ -65,7 +65,7 @@ public JArray()
         /// Initializes a new instance of the <see cref="JArray"/> class from another <see cref="JArray"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JArray"/> object to copy from.</param>
-        /// <param name="settings">
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object to configure cloning settings.</param>
         public JArray(JArray other, JsonCloneSettings? settings = null)
             : base(other, settings)
         {
diff --git a/Src/Newtonsoft.Json/Linq/JConstructor.cs b/Src/Newtonsoft.Json/Linq/JConstructor.cs
index 937a6edb0..d05f6e9e1 100644
--- a/Src/Newtonsoft.Json/Linq/JConstructor.cs
+++ b/Src/Newtonsoft.Json/Linq/JConstructor.cs
@@ -96,6 +96,7 @@ public JConstructor()
         /// Initializes a new instance of the <see cref="JConstructor"/> class from another <see cref="JConstructor"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JConstructor"/> object to copy from.</param>
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object to configure cloning settings.</param>
         public JConstructor(JConstructor other, JsonCloneSettings? settings = null)
             : base(other, settings)
         {
diff --git a/Src/Newtonsoft.Json/Linq/JProperty.cs b/Src/Newtonsoft.Json/Linq/JProperty.cs
index 11ae7c399..20175fbd5 100644
--- a/Src/Newtonsoft.Json/Linq/JProperty.cs
+++ b/Src/Newtonsoft.Json/Linq/JProperty.cs
@@ -186,6 +186,7 @@ public JToken Value
         /// Initializes a new instance of the <see cref="JProperty"/> class from another <see cref="JProperty"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JProperty"/> object to copy from.</param>
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object to configure cloning settings.</param>
         public JProperty(JProperty other, JsonCloneSettings? settings = null)
             : base(other, settings)
         {
diff --git a/Src/Newtonsoft.Json/Linq/JRaw.cs b/Src/Newtonsoft.Json/Linq/JRaw.cs
index c0694e793..904b89942 100644
--- a/Src/Newtonsoft.Json/Linq/JRaw.cs
+++ b/Src/Newtonsoft.Json/Linq/JRaw.cs
@@ -37,6 +37,7 @@ public partial class JRaw : JValue
         /// Initializes a new instance of the <see cref="JRaw"/> class from another <see cref="JRaw"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JRaw"/> object to copy from.</param>
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object to configure cloning settings.</param>
         public JRaw(JRaw other, JsonCloneSettings? settings = null)
             : base(other, settings)
         {
diff --git a/Src/Newtonsoft.Json/Linq/JToken.cs b/Src/Newtonsoft.Json/Linq/JToken.cs
index f6bb7739e..611f708ef 100644
--- a/Src/Newtonsoft.Json/Linq/JToken.cs
+++ b/Src/Newtonsoft.Json/Linq/JToken.cs
@@ -2449,7 +2449,7 @@ public JToken DeepClone()
         /// <summary>
         /// Creates a new instance of the <see cref="JToken"/>. All child tokens are recursively cloned.
         /// </summary>
-        /// <param name="settings">A flag to indicate whether to clone annotations when performing deep clone on a JToken.</param>
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object to configure cloning settings.</param>
         /// <returns>A new instance of the <see cref="JToken"/>.</returns>
         public JToken DeepClone(JsonCloneSettings settings)
         {
diff --git a/Src/Newtonsoft.Json/Linq/JValue.cs b/Src/Newtonsoft.Json/Linq/JValue.cs
index 5cb60b862..23f2e0212 100644
--- a/Src/Newtonsoft.Json/Linq/JValue.cs
+++ b/Src/Newtonsoft.Json/Linq/JValue.cs
@@ -61,7 +61,7 @@ internal JValue(object? value, JTokenType type)
         /// Initializes a new instance of the <see cref="JValue"/> class from another <see cref="JValue"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JValue"/> object to copy from.</param>
-        /// <param name="settings">A <see cref="JsonCloneSettings"/> object that specifies the settings used when cloning JSON.</param>
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object to configure cloning settings.</param>
         public JValue(JValue other, JsonCloneSettings? settings = null)
             : this(other.Value, other.Type)
         {

From 57bdc99312b8a7e145c583444ee4d628e4eb6e70 Mon Sep 17 00:00:00 2001
From: Sneha Shivakumar <snshivakumar@microsoft.com>
Date: Wed, 26 Oct 2022 15:58:34 -0700
Subject: [PATCH 5/6] Matching comment to other comments.

---
 Src/Newtonsoft.Json/Linq/JObject.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Src/Newtonsoft.Json/Linq/JObject.cs b/Src/Newtonsoft.Json/Linq/JObject.cs
index f6727e1bd..e9d9018f0 100644
--- a/Src/Newtonsoft.Json/Linq/JObject.cs
+++ b/Src/Newtonsoft.Json/Linq/JObject.cs
@@ -92,7 +92,7 @@ public JObject()
         /// Initializes a new instance of the <see cref="JObject"/> class from another <see cref="JObject"/> object.
         /// </summary>
         /// <param name="other">A <see cref="JObject"/> object to copy from.</param>
-        /// <param name="settings">A <see cref="JsonCloneSettings"/> object indicating whether to clone annotations when cloning a JToken.</param>
+        /// <param name="settings">A <see cref="JsonCloneSettings"/> object to configure cloning settings.</param>
         public JObject(JObject other, JsonCloneSettings? settings = null)
             : base(other, settings)
         {

From 71b1785d0aa229c84c81caf22cd079a26216565b Mon Sep 17 00:00:00 2001
From: Sneha Shivakumar <snshivakumar@microsoft.com>
Date: Mon, 31 Oct 2022 11:28:00 -0700
Subject: [PATCH 6/6] Added additional unit tests, fixed small bug

---
 .../Linq/AnnotationsTests.cs                  | 42 ++++++++++++++++++-
 Src/Newtonsoft.Json/Linq/JConstructor.cs      |  2 +-
 2 files changed, 42 insertions(+), 2 deletions(-)

diff --git a/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs b/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs
index e86f04810..6fa31be27 100644
--- a/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs
+++ b/Src/Newtonsoft.Json.Tests/Linq/AnnotationsTests.cs
@@ -312,15 +312,55 @@ private void AssertCloneCopy<T>(JToken t, T annotation) where T : class
         public void MultipleAnnotationsAreNotCopiedWithSetting()
         {
             Version version = new Version(1, 2, 3, 4);
+            JsonCloneSettings settings = new JsonCloneSettings() { CopyAnnotations = false };
 
             JObject o = new JObject();
             o.AddAnnotation("string!");
             o.AddAnnotation(version);
 
-            JsonCloneSettings settings = new JsonCloneSettings() { CopyAnnotations = false };
             JObject o2 = (JObject)o.DeepClone(settings);
             Assert.IsNull(o2.Annotation<string>());
             Assert.AreEqual(0, o2.Annotations<Version>().Count());
+
+            JArray a = new JArray();
+            a.AddAnnotation("string!");
+            a.AddAnnotation(version);
+
+            JArray a2 = (JArray)a.DeepClone(settings);
+            Assert.IsNull(a2.Annotation<string>());
+            Assert.AreEqual(0, a2.Annotations<Version>().Count());
+
+            JProperty p = new JProperty("test");
+            p.AddAnnotation("string!");
+            p.AddAnnotation(version);
+
+            JProperty p2 = (JProperty)p.DeepClone(settings);
+            Assert.IsNull(p2.Annotation<string>());
+            Assert.AreEqual(0, p2.Annotations<Version>().Count());
+
+            JRaw r = new JRaw("test");
+            r.AddAnnotation("string!");
+            r.AddAnnotation(version);
+
+            JRaw r2 = (JRaw)r.DeepClone(settings);
+            Assert.IsNull(r2.Annotation<string>());
+            Assert.AreEqual(0, r2.Annotations<Version>().Count());
+
+            JConstructor c = new JConstructor("test");
+            c.AddAnnotation("string!");
+            c.AddAnnotation(version);
+
+            JConstructor c2 = (JConstructor)c.DeepClone(settings);
+            Assert.IsNull(c2.Annotation<string>());
+            Assert.AreEqual(0, c2.Annotations<Version>().Count());
+
+            JValue v = new JValue("test");
+            v.AddAnnotation("string!");
+            v.AddAnnotation(version);
+
+            JValue v2 = (JValue)v.DeepClone(settings);
+            Assert.IsNull(v2.Annotation<string>());
+            Assert.AreEqual(0, v2.Annotations<Version>().Count());
         }
 
 #if !NET20
diff --git a/Src/Newtonsoft.Json/Linq/JConstructor.cs b/Src/Newtonsoft.Json/Linq/JConstructor.cs
index d05f6e9e1..a989e8117 100644
--- a/Src/Newtonsoft.Json/Linq/JConstructor.cs
+++ b/Src/Newtonsoft.Json/Linq/JConstructor.cs
@@ -150,7 +150,7 @@ internal override bool DeepEquals(JToken node)
 
         internal override JToken CloneToken(JsonCloneSettings? settings = null)
         {
-            return new JConstructor(this);
+            return new JConstructor(this, settings);
         }
 
         /// <summary>