Comments/code style adjustments, some documentation, no code changes

Author: h3xds1nz

src/Microsoft.DotNet.Wpf/src/PresentationCore/System/Windows/Media/Animation/KeySplineConverter.cs

@@ -23,7 +23,11 @@ public class KeySplineConverter : TypeConverter
         /// <summary>
         /// CanConvertFrom - Returns whether or not this class can convert from a given type
         /// </summary>
-        /// <ExternalAPI/>
+        /// <returns>
+        /// <see langword="true"/> if the given <paramref name="sourceType"/> can be converted from, <see langword="false"/> otherwise.
+        /// </returns>
+        /// <param name="typeDescriptor">The <see cref="ITypeDescriptorContext"/> for this call.</param>
+        /// <param name="destinationType">The <see cref="Type"/> being queried for support.</param>
         public override bool CanConvertFrom(ITypeDescriptorContext typeDescriptor, Type destinationType)
         {
             return destinationType == typeof(string);
@@ -34,16 +38,22 @@ public override bool CanConvertFrom(ITypeDescriptorContext typeDescriptor, Type
         /// </summary>
         /// <param name="context">ITypeDescriptorContext</param>
         /// <param name="destinationType">Type to convert to</param>
-        /// <returns>true if conversion is possible</returns>
-        /// <ExternalAPI/>
+        /// <returns>
+        /// <see langword="true"/> if this class can convert to <paramref name="destinationType"/>, <see langword="false"/> otherwise.
+        /// </returns>
         public override bool CanConvertTo(ITypeDescriptorContext context, Type destinationType)
         {
             return destinationType == typeof(InstanceDescriptor) || destinationType == typeof(string);
         }
 
         /// <summary>
-        /// ConvertFrom
+        /// Converts <paramref name="value"/> of <see langword="string"/> type to its <see cref="KeySpline"/> represensation.
         /// </summary>
+        /// <param name="context">The <see cref="ITypeDescriptorContext"/> for this call.</param>
+        /// <param name="cultureInfo">The <see cref="CultureInfo"/> which is respected during conversion.</param>
+        /// <param name="value"> The object to convert to a <see cref="KeySpline"/>.</param>
+        /// <returns>A new instance of <see cref="KeySpline"/> class representing the data contained in <paramref name="value"/>.</returns>
+        /// <exception cref="NotSupportedException">Thrown in case the <paramref name="value"/> was not a <see cref="string"/>.</exception>
         public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo cultureInfo, object value)
         {
             if (value is not string stringValue)
@@ -58,14 +68,15 @@ public override object ConvertFrom(ITypeDescriptorContext context, CultureInfo c
         }
 
         /// <summary>
-        /// TypeConverter method implementation.
+        /// Attempt to convert a <see cref="KeySpline"/> class to the <paramref name="destinationType"/>.
         /// </summary>
         /// <param name="context">ITypeDescriptorContext</param>
         /// <param name="cultureInfo">current culture (see CLR specs), null is a valid value</param>
         /// <param name="value">value to convert from</param>
         /// <param name="destinationType">Type to convert to</param>
-        /// <returns>converted value</returns>
-        /// <ExternalAPI/>
+        /// <returns>
+        /// The formatted <paramref name="value"/> as <see cref="string"/> using the specified <paramref name="cultureInfo"/> or an <see cref="InstanceDescriptor"/>.
+        /// </returns>
         public override object ConvertTo(ITypeDescriptorContext context, CultureInfo cultureInfo, object value, Type destinationType)
         {
             if (value is KeySpline keySpline && destinationType is not null)
@@ -85,17 +96,17 @@ public override object ConvertTo(ITypeDescriptorContext context, CultureInfo cul
         }
 
         /// <summary>
-        /// Converts <paramref name="keySpline"/> to its string representation using the specified <paramref name="cultureInfo"/>.
+        /// Converts <paramref name="keySpline"/> to its <see cref="string"/> representation using the specified <paramref name="cultureInfo"/>.
         /// </summary>
         /// <param name="keySpline">The <see cref="KeySpline"/> to convert to string.</param>
         /// <param name="cultureInfo">Culture to use when formatting doubles and choosing separator.</param>
-        /// <returns>The formatted <paramref name="keySpline"/> as string using the specified <paramref name="cultureInfo"/>.</returns>
+        /// <returns>The formatted <paramref name="keySpline"/> as <see cref="string"/> using the specified <paramref name="cultureInfo"/>.</returns>
         private static string ToString(KeySpline keySpline, CultureInfo cultureInfo)
         {
             string listSeparator = cultureInfo != null ? cultureInfo.TextInfo.ListSeparator : CultureInfo.InvariantCulture.TextInfo.ListSeparator;
 
             // Initial capacity [64] is an estimate based on a sum of:
-            // 48 = 4x double (fourteen digits is generous for the range of values likely)
+            // 60 = 4x double (fifteen digits is generous for the range of values)
             //  3 = 3x separator characters
             //  1 = 1x scratch space for alignment
             DefaultInterpolatedStringHandler handler = new(3, 4, cultureInfo, stackalloc char[64]);

src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/Controls/VirtualizationCacheLengthConverter.cs

@@ -11,9 +11,8 @@
 namespace System.Windows.Controls
 {
     /// <summary>
-    /// VirtualizationCacheLengthConverter - Converter class for converting
-    /// instances of other types to and from VirtualizationCacheLength instances.
-    /// </summary>
+    /// Converter class for converting instances of other types to and from <see cref="VirtualizationCacheLength"/> instances.
+    /// </summary> 
     public class VirtualizationCacheLengthConverter : TypeConverter
     {
         #region Public Methods
@@ -22,10 +21,10 @@ public class VirtualizationCacheLengthConverter : TypeConverter
         /// CanConvertFrom - Returns whether or not this class can convert from a given type.
         /// </summary>
         /// <returns>
-        /// bool - True if thie converter can convert from the provided type, false if not.
+        /// <see langword="true"/> if the given <paramref name="sourceType"/> can be converted from, <see langword="false"/> otherwise.
         /// </returns>
-        /// <param name="typeDescriptorContext"> The ITypeDescriptorContext for this call. </param>
-        /// <param name="sourceType"> The Type being queried for support. </param>
+        /// <param name="typeDescriptorContext">The <see cref="ITypeDescriptorContext"/> for this call.</param>
+        /// <param name="sourceType">The <see cref="Type"/> being queried for support.</param>
         public override bool CanConvertFrom(ITypeDescriptorContext typeDescriptorContext, Type sourceType)
         {
             // We can only handle strings, integral and floating types

src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/CornerRadiusConverter.cs

@@ -11,7 +11,7 @@
 namespace System.Windows
 {
     /// <summary>
-    /// CornerRadiusConverter - Converter class for converting instances of other types to and from CornerRadius instances.
+    /// Converter class for converting instances of other types to and from <see cref="CornerRadius"/> instances.
     /// </summary> 
     public class CornerRadiusConverter : TypeConverter
     {
@@ -21,10 +21,10 @@ public class CornerRadiusConverter : TypeConverter
         /// CanConvertFrom - Returns whether or not this class can convert from a given type.
         /// </summary>
         /// <returns>
-        /// bool - True if thie converter can convert from the provided type, false if not.
+        /// <see langword="true"/> if the given <paramref name="sourceType"/> can be converted from, <see langword="false"/> otherwise.
         /// </returns>
-        /// <param name="typeDescriptorContext"> The ITypeDescriptorContext for this call. </param>
-        /// <param name="sourceType"> The Type being queried for support. </param>
+        /// <param name="typeDescriptorContext">The <see cref="ITypeDescriptorContext"/> for this call.</param>
+        /// <param name="sourceType">The <see cref="Type"/> being queried for support.</param>
         public override bool CanConvertFrom(ITypeDescriptorContext typeDescriptorContext, Type sourceType)
         {
             // We can only handle strings, integral and floating types

src/Microsoft.DotNet.Wpf/src/PresentationFramework/System/Windows/ThicknessConverter.cs

@@ -12,20 +12,20 @@
 namespace System.Windows
 {
     /// <summary>
-    /// ThicknessConverter - Converter class for converting instances of other types to and from Thickness instances.
+    /// Converter class for converting instances of other types to and from <see cref="Thickness"/> instances.
     /// </summary> 
     public class ThicknessConverter : TypeConverter
     {
         #region Public Methods
 
         /// <summary>
-        /// CanConvertFrom - Returns whether or not this class can convert from a given type.
+        /// Returns whether this class can convert specific <paramref name="sourceType"/> into <see cref="Thickness"/>.
         /// </summary>
         /// <returns>
-        /// bool - True if thie converter can convert from the provided type, false if not.
+        /// <see langword="true"/> if the given <paramref name="sourceType"/> can be converted from, <see langword="false"/> otherwise.
         /// </returns>
-        /// <param name="typeDescriptorContext"> The ITypeDescriptorContext for this call. </param>
-        /// <param name="sourceType"> The Type being queried for support. </param>
+        /// <param name="typeDescriptorContext">The <see cref="ITypeDescriptorContext"/> for this call.</param>
+        /// <param name="sourceType">The <see cref="Type"/> being queried for support.</param>
         public override bool CanConvertFrom(ITypeDescriptorContext typeDescriptorContext, Type sourceType)
         {
             // We can only handle strings, integral and floating types
@@ -95,22 +95,22 @@ public override object ConvertFrom(ITypeDescriptorContext typeDescriptorContext,
         }
 
         /// <summary>
-        /// ConvertTo - Attempt to convert a Thickness to the given type
+        /// Attempt to convert a <see cref="Thickness"/> struct to the <paramref name="destinationType"/>.
         /// </summary>
         /// <returns>
-        /// The object which was constructed.
+        /// The formatted <paramref name="value"/> as <see cref="string"/> using the specified <paramref name="cultureInfo"/> or an <see cref="InstanceDescriptor"/>.
         /// </returns>
+        /// <param name="typeDescriptorContext"> The ITypeDescriptorContext for this call. </param>
+        /// <param name="cultureInfo"> The CultureInfo which is respected when converting. </param>
+        /// <param name="value"> The Thickness to convert. </param>
+        /// <param name="destinationType">The type to which to convert the Thickness instance. </param>
         /// <exception cref="ArgumentNullException">
         /// An ArgumentNullException is thrown if the example object is null.
         /// </exception>
         /// <exception cref="ArgumentException">
         /// An ArgumentException is thrown if the object is not null and is not a Thickness,
         /// or if the destinationType isn't one of the valid destination types.
         /// </exception>
-        /// <param name="typeDescriptorContext"> The ITypeDescriptorContext for this call. </param>
-        /// <param name="cultureInfo"> The CultureInfo which is respected when converting. </param>
-        /// <param name="value"> The Thickness to convert. </param>
-        /// <param name="destinationType">The type to which to convert the Thickness instance. </param>
         public override object ConvertTo(ITypeDescriptorContext typeDescriptorContext, CultureInfo cultureInfo, object value, Type destinationType)
         {
             ArgumentNullException.ThrowIfNull(value);
@@ -144,9 +144,9 @@ public override object ConvertTo(ITypeDescriptorContext typeDescriptorContext, C
         /// <summary>
         /// Converts <paramref name="thickness"/> to its string representation using the specified <paramref name="cultureInfo"/>.
         /// </summary>
-        /// <param name="thickness">The <see cref="Thickness"/> to convert to string.</param>
+        /// <param name="thickness">The <see cref="Thickness"/> to convert to <see cref="string"/>.</param>
         /// <param name="cultureInfo">Culture to use when formatting doubles and choosing separator.</param>
-        /// <returns>The formatted <paramref name="thickness"/> as string using the specified <paramref name="cultureInfo"/>.</returns>
+        /// <returns>The formatted <paramref name="thickness"/> as <see cref="string"/> using the specified <paramref name="cultureInfo"/>.</returns>
         internal static string ToString(Thickness thickness, CultureInfo cultureInfo)
         {
             char listSeparator = TokenizerHelper.GetNumericListSeparator(cultureInfo);