improve wording

Author: Max Charlamb

docs/design/datacontracts/data_descriptor.md

@@ -214,7 +214,7 @@ The global values will be in an array, with each value described by a dictionary
 * `"type": "type name"` the type of the global value
 * optional `"value": VALUE | [ int ] ` the value of the global value, or an offset in an auxiliary array containing the value.
 
-The `VALUE` may be either a number of string. JSON numeric constants are always parsed as numbers. JSON strings are always parsed as strings and may additionally parse as a hex (with prefix `0x` or `0X`) or decimal number.
+The `VALUE` may be either a number or a string. JSON numeric constants are always parsed as numbers. JSON strings are always parsed as strings and may additionally parse as a hex (with prefix `0x` or `0X`) or decimal number.
 Numeric constants must be within the range of the type of the global value.
 
 
@@ -225,7 +225,7 @@ The global values will be in a dictionary, with each key being the name of a glo
 * `[VALUE | [int], "type name"]` the type and value of a global
 * `VALUE | [int]` just the value of a global
 
-`VALUE` may be either a number of string. JSON numeric constants are always parsed as numbers. JSON strings are always parsed as strings and may additionally parse as a hex (with prefix `0x` or `0X`) or decimal number.
+`VALUE` may be either a number or a string. JSON numeric constants are always parsed as numbers. JSON strings are always parsed as strings and may additionally parse as a hex (with prefix `0x` or `0X`) or decimal number.
 Numeric constants must be within the range of the type of the global value.
 
 Note that a two element array is unambiguously "type and value", whereas a one-element array is

src/native/managed/cdacreader/Microsoft.Diagnostics.DataContractReader.Abstractions/Target.cs

@@ -99,14 +99,14 @@ public abstract class Target
     /// <param name="name">The name of the global</param>
     /// <param name="value">The value of the global if found</param>
     /// <returns>True if a global is found, false otherwise</returns>
-    public abstract bool TryReadStringGlobal(string name, [NotNullWhen(true)] out string? value);
+    public abstract bool TryReadGlobalString(string name, [NotNullWhen(true)] out string? value);
 
     /// <summary>
     /// Read a well known global from the target process as a string
     /// </summary>
     /// <param name="name">The name of the global</param>
     /// <returns>A string value</returns>
-    public abstract string ReadStringGlobal(string name);
+    public abstract string ReadGlobalString(string name);
 
     /// <summary>
     /// Read a well known global from the target process as a number in the target endianness

src/native/managed/cdacreader/Microsoft.Diagnostics.DataContractReader/ContractDescriptorTarget.cs

@@ -555,7 +555,7 @@ public TargetPointer ReadGlobalPointer(string name, out string? type)
         return value.Value;
     }
 
-    public override string ReadStringGlobal(string name)
+    public override string ReadGlobalString(string name)
         => ReadStringGlobal(name, out _);
 
     public string ReadStringGlobal(string name, out string? type)
@@ -566,7 +566,7 @@ public string ReadStringGlobal(string name, out string? type)
         return value;
     }
 
-    public override bool TryReadStringGlobal(string name, [NotNullWhen(true)] out string? value)
+    public override bool TryReadGlobalString(string name, [NotNullWhen(true)] out string? value)
         => TryReadStringGlobal(name, out value, out _);
 
     public bool TryReadStringGlobal(string name, [NotNullWhen(true)] out string? value, out string? type)

src/native/managed/cdacreader/tests/ContractDescriptor/TargetTests.cs

@@ -297,10 +297,10 @@ private static void ValidateGlobalStrings(
         {
             string actualString;
 
-            Assert.True(target.TryReadStringGlobal(name, out actualString));
+            Assert.True(target.TryReadGlobalString(name, out actualString));
             AssertEqualsWithCallerInfo(value, actualString);
 
-            actualString = target.ReadStringGlobal(name);
+            actualString = target.ReadGlobalString(name);
             AssertEqualsWithCallerInfo(value, actualString);
 
             if (numericValue != null)

src/native/managed/cdacreader/tests/TestPlaceholderTarget.cs

@@ -132,17 +132,17 @@ public override T ReadGlobal<T>(string name)
         throw new NotImplementedException();
     }
 
-    public override string ReadStringGlobal(string name)
+    public override string ReadGlobalString(string name)
     {
-        if (TryReadStringGlobal(name, out string? value))
+        if (TryReadGlobalString(name, out string? value))
         {
             return value;
         }
 
         throw new NotImplementedException();
     }
 
-    public override bool TryReadStringGlobal(string name, [NotNullWhen(true)] out string? value)
+    public override bool TryReadGlobalString(string name, [NotNullWhen(true)] out string? value)
     {
         value = null;