Add comment to clarify code intension. #213

msgpack · Feb 11, 2017 · 49f2c1e · 49f2c1e
1 parent cc91838
commit 49f2c1e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/MsgPack/Packer.Packing.cs b/src/MsgPack/Packer.Packing.cs
@@ -1023,6 +1023,7 @@ private void PrivatePackCore( float value )
 
 			var bits = new Float32Bits( value );
 
+			// Float32Bits usage is effectively pointer dereference operation rather than shifting operators, so we must consider endianness here.
 			if ( BitConverter.IsLittleEndian )
 			{
 				this.WriteByte( bits.Byte3 );
@@ -1070,6 +1071,7 @@ private async Task PrivatePackAsyncCore( float value, CancellationToken cancella
 
 			var bits = new Float32Bits( value );
 
+			// Float32Bits usage is effectively pointer dereference operation rather than shifting operators, so we must consider endianness here.
 			if ( BitConverter.IsLittleEndian )
 			{
 				await this.WriteByteAsync( bits.Byte3, cancellationToken ).ConfigureAwait( false );

diff --git a/src/MsgPack/Packer.Packing.tt b/src/MsgPack/Packer.Packing.tt
@@ -362,6 +362,7 @@ foreach ( var isAsync in new [] { false, true } )
 
 			var bits = new Float32Bits( value );
 
+			// Float32Bits usage is effectively pointer dereference operation rather than shifting operators, so we must consider endianness here.
 			if ( BitConverter.IsLittleEndian )
 			{
 <#
-Original file line number
+Diff line change
@@ Expand Up / @@ -362,6 +362,7 @@ foreach ( var isAsync in new [] { false, true } ) @@
     			var bits = new Float32Bits( value );
+    			// Float32Bits usage is effectively pointer dereference operation rather than shifting operators, so we must consider endianness here.
     			if ( BitConverter.IsLittleEndian )
     			{
     <#
@@ Expand Down @@