Few comments.

ikvmnet · Dec 9, 2024 · b31a257 · b31a257
1 parent d903f1c
commit b31a257
Show file tree

Hide file tree

Showing 4 changed files with 326 additions and 317 deletions.
diff --git a/src/IKVM.ByteCode/Encoding/CodeBuilder.cs b/src/IKVM.ByteCode/Encoding/CodeBuilder.cs
@@ -305,7 +305,7 @@ internal void WriteByte(byte value)
         }
 
         /// <summary>
-        /// Encodes a 8 bit sized argument.
+        /// Encodes a 8 bit sized unsigned integer argument.
         /// </summary>
         /// <param name="value"></param>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -315,7 +315,7 @@ internal void WriteU1(byte value)
         }
 
         /// <summary>
-        /// Encodes a 8 bit sized argument.
+        /// Encodes a 8 bit sized signed integer argument.
         /// </summary>
         /// <param name="value"></param>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -335,7 +335,7 @@ internal void WriteC1(ConstantHandle handle)
         }
 
         /// <summary>
-        /// Encodes a 8 bit sized constant argument.
+        /// Encodes a 8 bit sized local argument.
         /// </summary>
         /// <param name="local"></param>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -345,7 +345,7 @@ internal void WriteL1(byte local)
         }
 
         /// <summary>
-        /// Encodes a 16-bit sized integer argument.
+        /// Encodes a 16-bit sized unsigned integer argument.
         /// </summary>
         /// <param name="value"></param>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -355,7 +355,7 @@ internal void WriteUInt16(ushort value)
         }
 
         /// <summary>
-        /// Encodes a 16 bit sized argument.
+        /// Encodes a 16 bit sized signed integer argument.
         /// </summary>
         /// <param name="value"></param>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -365,7 +365,7 @@ internal void WriteS2(short value)
         }
 
         /// <summary>
-        /// Encodes a 16 bit sized constant argument.
+        /// Encodes a 16 bit sized local argument.
         /// </summary>
         /// <param name="value"></param>
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
@@ -381,7 +381,7 @@ internal void WriteL2(ushort local)
         [MethodImpl(MethodImplOptions.AggressiveInlining)]
         internal void WriteC2(ConstantHandle handle)
         {
-            WriteUInt16(checked((ushort)handle.Slot));
+            WriteUInt16(handle.Slot);
         }
 
         /// <summary>