Address review comments

src/Types/BlockHashInput.cs

@@ -8,7 +8,7 @@ namespace Concordium.Sdk.Types;
 public interface IBlockHashInput
 {
     /// <summary>
-    /// Convert this into type expected by the gRPC.
+    /// Convert this into the type expected by the gRPC API.
     /// </summary>
     BlockHashInput Into();
 }
@@ -21,7 +21,7 @@ public sealed record Best : IBlockHashInput
     private static readonly Empty _empty = new();
 
     /// <summary>
-    /// Convert this into type expected by the gRPC.
+    /// Convert this into the type expected by the gRPC API.
     /// </summary>
     public BlockHashInput Into() => new()
     {
@@ -38,7 +38,7 @@ public sealed record LastFinal : IBlockHashInput
     private static readonly Empty _empty = new();
 
     /// <summary>
-    /// Convert this into type expected by the gRPC.
+    /// Convert this into the type expected by the gRPC API.
     /// </summary>
     public BlockHashInput Into() => new()
     {
@@ -52,7 +52,7 @@ public sealed record LastFinal : IBlockHashInput
 public sealed record Given(BlockHash BlockHash) : IBlockHashInput
 {
     /// <summary>
-    /// Convert this into type expected by the gRPC.
+    /// Convert this into the type expected by the gRPC API.
     /// </summary>
     public BlockHashInput Into() => new() { Given = this.BlockHash.ToProto() };
 }

src/Types/BlockHeight.cs

@@ -8,7 +8,7 @@ namespace Concordium.Sdk.Types;
 public interface IBlockHeight
 {
     /// <summary>
-    /// Convert into a request for blocks at height.
+    /// Convert into a BlocksAtHeightRequest.
     /// </summary>
     BlocksAtHeightRequest Into();
 }
@@ -22,7 +22,7 @@ public sealed record AbsoluteHeight(ulong Height) : IBlockHeight
     internal static AbsoluteHeight From(AbsoluteBlockHeight blockHeight) => new(blockHeight.Value);
 
     /// <summary>
-    /// Convert an AbsoluteHeight into a request for blocks at height.
+    /// Convert an AbsoluteHeight into a BlocksAtHeightRequest.
     /// </summary>
     public BlocksAtHeightRequest Into() =>
         new()
@@ -47,7 +47,7 @@ public BlocksAtHeightRequest Into() =>
 public sealed record RelativeHeight(ulong Height, uint GenesisIndex, bool Restrict) : IBlockHeight
 {
     /// <summary>
-    /// Convert an RelativeHeight into a request for blocks at height.
+    /// Convert a RelativeHeight into a BlocksAtHeightRequest.
     /// </summary>
     public BlocksAtHeightRequest Into() =>
         new()

src/Types/CcdAmount.cs

@@ -139,17 +139,7 @@ public static bool TryDeserial(ReadOnlySpan<byte> bytes, out (CcdAmount? Amount,
     /// <summary>
     /// Return the largest of two amounts.
     /// </summary>
-    public static CcdAmount Max(CcdAmount first, CcdAmount second)
-    {
-        if (first < second)
-        {
-            return second;
-        }
-        else
-        {
-            return first;
-        }
-    }
+    public static CcdAmount Max(CcdAmount first, CcdAmount second) => first < second ? second : first;
 
     /// <summary> Determines whether one CcdAmount is less than a second CcdAmount. </summary>
     public static bool operator <(CcdAmount left, CcdAmount right) => left.Value.CompareTo(right.Value) < 0;

src/Types/ReceiveName.cs

@@ -71,7 +71,7 @@ public enum ValidationError
         MissingDotSeparator,
         /// <summary> The receive name is more than 100 characters. </summary>
         TooLong,
-        /// <summary> The receive name contained invalid characters. </summary>
+        /// <summary> The receive name contained characters that are not ASCII alphanumeric or ASCII punctuation. </summary>
         InvalidCharacters,
     }