Adding comment to field names that cannot be changed for compatibility with desktop .NET as they appear in serialization payload.

3 files changed, 40 insertions, 40 deletions

diff --git a/src/System.Private.CoreLib/src/System/Collections/Generic/KeyValuePair.cs b/src/System.Private.CoreLib/src/System/Collections/Generic/KeyValuePair.cs
index 02c831ed6..f7198c1e4 100644
--- a/src/System.Private.CoreLib/src/System/Collections/Generic/KeyValuePair.cs
+++ b/src/System.Private.CoreLib/src/System/Collections/Generic/KeyValuePair.cs
@@ -40,8 +40,8 @@ namespace System.Collections.Generic
     // and IReadOnlyDictionary<TKey, TValue>.
     public struct KeyValuePair<TKey, TValue>
     {
-        private TKey key;
-        private TValue value;
+        private TKey key;       // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private TValue value;   // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public KeyValuePair(TKey key, TValue value)
         {
diff --git a/src/System.Private.CoreLib/src/System/Collections/ObjectModel/ReadOnlyCollection.cs b/src/System.Private.CoreLib/src/System/Collections/ObjectModel/ReadOnlyCollection.cs
index a3a58d5f0..6ed20ceea 100644
--- a/src/System.Private.CoreLib/src/System/Collections/ObjectModel/ReadOnlyCollection.cs
+++ b/src/System.Private.CoreLib/src/System/Collections/ObjectModel/ReadOnlyCollection.cs
@@ -14,8 +14,8 @@ namespace System.Collections.ObjectModel
     [DebuggerDisplay("Count = {Count}")]
     public class ReadOnlyCollection<T> : IList<T>, IList, IReadOnlyList<T>
     {
-        private IList<T> list;
-        private Object _syncRoot;
+        private IList<T> list;      // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private Object _syncRoot;   // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public ReadOnlyCollection(IList<T> list)
         {
diff --git a/src/System.Private.CoreLib/src/System/Tuple.cs b/src/System.Private.CoreLib/src/System/Tuple.cs
index a348e5e73..f7575ef74 100644
--- a/src/System.Private.CoreLib/src/System/Tuple.cs
+++ b/src/System.Private.CoreLib/src/System/Tuple.cs
@@ -100,7 +100,7 @@ namespace System
 
     public class Tuple<T1> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
 
@@ -186,8 +186,8 @@ namespace System
 
     public class Tuple<T1, T2> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
-        private readonly T2 m_Item2;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T2 m_Item2;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
         public T2 Item2 { get { return m_Item2; } }
@@ -283,9 +283,9 @@ namespace System
 
     public class Tuple<T1, T2, T3> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
-        private readonly T2 m_Item2;
-        private readonly T3 m_Item3;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T2 m_Item2;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T3 m_Item3;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
         public T2 Item2 { get { return m_Item2; } }
@@ -389,10 +389,10 @@ namespace System
 
     public class Tuple<T1, T2, T3, T4> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
-        private readonly T2 m_Item2;
-        private readonly T3 m_Item3;
-        private readonly T4 m_Item4;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T2 m_Item2;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T3 m_Item3;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T4 m_Item4;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
         public T2 Item2 { get { return m_Item2; } }
@@ -504,11 +504,11 @@ namespace System
 
     public class Tuple<T1, T2, T3, T4, T5> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
-        private readonly T2 m_Item2;
-        private readonly T3 m_Item3;
-        private readonly T4 m_Item4;
-        private readonly T5 m_Item5;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T2 m_Item2;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T3 m_Item3;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T4 m_Item4;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T5 m_Item5;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
         public T2 Item2 { get { return m_Item2; } }
@@ -628,12 +628,12 @@ namespace System
 
     public class Tuple<T1, T2, T3, T4, T5, T6> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
-        private readonly T2 m_Item2;
-        private readonly T3 m_Item3;
-        private readonly T4 m_Item4;
-        private readonly T5 m_Item5;
-        private readonly T6 m_Item6;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T2 m_Item2;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T3 m_Item3;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T4 m_Item4;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T5 m_Item5;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T6 m_Item6;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
         public T2 Item2 { get { return m_Item2; } }
@@ -761,13 +761,13 @@ namespace System
 
     public class Tuple<T1, T2, T3, T4, T5, T6, T7> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
-        private readonly T2 m_Item2;
-        private readonly T3 m_Item3;
-        private readonly T4 m_Item4;
-        private readonly T5 m_Item5;
-        private readonly T6 m_Item6;
-        private readonly T7 m_Item7;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T2 m_Item2;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T3 m_Item3;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T4 m_Item4;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T5 m_Item5;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T6 m_Item6;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T7 m_Item7;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
         public T2 Item2 { get { return m_Item2; } }
@@ -903,14 +903,14 @@ namespace System
 
     public class Tuple<T1, T2, T3, T4, T5, T6, T7, TRest> : IStructuralEquatable, IStructuralComparable, IComparable, ITuple
     {
-        private readonly T1 m_Item1;
-        private readonly T2 m_Item2;
-        private readonly T3 m_Item3;
-        private readonly T4 m_Item4;
-        private readonly T5 m_Item5;
-        private readonly T6 m_Item6;
-        private readonly T7 m_Item7;
-        private readonly TRest m_Rest;
+        private readonly T1 m_Item1;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T2 m_Item2;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T3 m_Item3;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T4 m_Item4;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T5 m_Item5;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T6 m_Item6;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly T7 m_Item7;    // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
+        private readonly TRest m_Rest;  // DO NOT change the field name, it's required for compatibility with desktop .NET as it appears in serialization payload.
 
         public T1 Item1 { get { return m_Item1; } }
         public T2 Item2 { get { return m_Item2; } }