Merge pull request #2843 from hughbe/bitconverter-comments

Remove irrelevant endianness comment, fix #834

1 files changed, 1 insertions, 15 deletions

diff --git a/src/mscorlib/src/System/BitConverter.cs b/src/mscorlib/src/System/BitConverter.cs
index f66d710160..a45b3632a4 100644
--- a/src/mscorlib/src/System/BitConverter.cs
+++ b/src/mscorlib/src/System/BitConverter.cs
@@ -411,7 +411,7 @@ namespace System {
         public static String ToString (byte [] value, int startIndex) {
             if (value == null)
                 throw new ArgumentNullException("value");
-            Contract.Ensures(Contract.Result<String>() != null);            
+            Contract.Ensures(Contract.Result<String>() != null);
             Contract.EndContractBlock();
             return ToString(value, startIndex, value.Length - startIndex);
         }
@@ -439,26 +439,12 @@ namespace System {
 
         [SecuritySafeCritical]
         public static unsafe long DoubleToInt64Bits(double value) {
-            // If we're on a big endian machine, what should this method do?  You could argue for
-            // either big endian or little endian, depending on whether you are writing to a file that
-            // should be used by other programs on that processor, or for compatibility across multiple
-            // formats.  Because this is ambiguous, we're excluding this from the Portable Library & Win8 Profile.
-            // If we ever run on big endian machines, produce two versions where endianness is specified.
-            Contract.Assert(IsLittleEndian, "This method is implemented assuming little endian with an ambiguous spec.");
             return *((long *)&value);
         }
 
         [SecuritySafeCritical]
         public static unsafe double Int64BitsToDouble(long value) {
-            // If we're on a big endian machine, what should this method do?  You could argue for
-            // either big endian or little endian, depending on whether you are writing to a file that
-            // should be used by other programs on that processor, or for compatibility across multiple
-            // formats.  Because this is ambiguous, we're excluding this from the Portable Library & Win8 Profile.
-            // If we ever run on big endian machines, produce two versions where endianness is specified.
-            Contract.Assert(IsLittleEndian, "This method is implemented assuming little endian with an ambiguous spec.");
             return *((double*)&value);
         }                    
     }
-
-
 }