Update wordings for diagnostics tools dependencies comments.

[tfs-changeset: 1616096]

Commit migrated from https://github.com/dotnet/corefx/commit/bcb337cfc50bfa3ef7822f5895cc459bd0c1627a

diff --git a/src/libraries/System.Collections.Immutable/src/System/Collections/Immutable/AllocFreeConcurrentStack.cs b/src/libraries/System.Collections.Immutable/src/System/Collections/Immutable/AllocFreeConcurrentStack.cs
index e2a8b5c..c38efa9 100644 (file)
--- a/src/libraries/System.Collections.Immutable/src/System/Collections/Immutable/AllocFreeConcurrentStack.cs
+++ b/src/libraries/System.Collections.Immutable/src/System/Collections/Immutable/AllocFreeConcurrentStack.cs
@@ -62,10 +62,10 @@ namespace System.Collections.Immutable
 
     internal static class AllocFreeConcurrentStack
     {
-        // The following field is required for interop with the VS Debugger
-        // Prior to making any changes to this field, please reach out to the VS Debugger 
-        // team to make sure that your changes are not going to prevent the debugger
-        // from working.
+        // WARNING: We allow diagnostic tools to directly inspect this member (t_stacks). 
+        // See https://github.com/dotnet/corert/blob/master/Documentation/design-docs/diagnostics/diagnostics-tools-contract.md for more details. 
+        // Please do not change the type, the name, or the semantic usage of this member without understanding the implication for tools. 
+        // Get in touch with the diagnostics team if you have questions.
 
         // Workaround for https://github.com/dotnet/coreclr/issues/2191.
         // When that's fixed, a [ThreadStatic] Stack should be added back to AllocFreeConcurrentStack<T>.

diff --git a/src/libraries/System.Collections/src/System/Collections/Generic/Comparer.cs b/src/libraries/System.Collections/src/System/Collections/Generic/Comparer.cs
index 7e8e960..d964a53 100644 (file)
--- a/src/libraries/System.Collections/src/System/Collections/Generic/Comparer.cs
+++ b/src/libraries/System.Collections/src/System/Collections/Generic/Comparer.cs
@@ -62,10 +62,10 @@ namespace System.Collections.Generic
             return new DefaultComparer<T>();
         }
 
-        // The following field is required for interop with the VS Debugger
-        // Prior to making any changes to this field, please reach out to the VS Debugger 
-        // team to make sure that your changes are not going to prevent the debugger
-        // from working.
+        // WARNING: We allow diagnostic tools to directly inspect this member (_default). 
+        // See https://github.com/dotnet/corert/blob/master/Documentation/design-docs/diagnostics/diagnostics-tools-contract.md for more details. 
+        // Please do not change the type, the name, or the semantic usage of this member without understanding the implication for tools. 
+        // Get in touch with the diagnostics team if you have questions.
         private static Comparer<T> _default;
     }
 

diff --git a/src/libraries/System.Collections/src/System/Collections/Generic/EqualityComparer.cs b/src/libraries/System.Collections/src/System/Collections/Generic/EqualityComparer.cs
index e8974c3..3a5e404 100644 (file)
--- a/src/libraries/System.Collections/src/System/Collections/Generic/EqualityComparer.cs
+++ b/src/libraries/System.Collections/src/System/Collections/Generic/EqualityComparer.cs
@@ -83,10 +83,10 @@ namespace System.Collections.Generic
             }
         }
         
-        // The following field is required for interop with the VS Debugger
-        // Prior to making any changes to this field, please reach out to the VS Debugger 
-        // team to make sure that your changes are not going to prevent the debugger
-        // from working.
+        // WARNING: We allow diagnostic tools to directly inspect this member (_default). 
+        // See https://github.com/dotnet/corert/blob/master/Documentation/design-docs/diagnostics/diagnostics-tools-contract.md for more details. 
+        // Please do not change the type, the name, or the semantic usage of this member without understanding the implication for tools. 
+        // Get in touch with the diagnostics team if you have questions.
         private static volatile EqualityComparer<T> _default;
 
         public abstract bool Equals(T x, T y);