Add a small note about modulo constants

Author: Genbox

README.md

@@ -220,6 +220,7 @@ There are many benefits gained from generating data structures at compile time:
 * Enables otherwise time-consuming data analysis (e.g. zero runtime overhead)
 * No defensive copying of data (takes time and needs double the memory)
 * No virtual dispatching (virtual method calls & inheritance) and no unnecessary branching
+* Modulo operations are known constants and compilers optimize it to bitwise operations
 
 ### Data analysis
 

Src/FastData/FastDataGenerator.cs

@@ -94,10 +94,10 @@ public static string Generate<T>(ReadOnlySpan<T> data, FastDataConfig fdCfg, ICo
         }
 
         ILogger logger = factory.CreateLogger(typeof(FastDataGenerator));
+        LogUserStructureType(logger, fdCfg.StructureType);
         LogUniqueItems(logger, uniq.Count);
 
         DataProperties<T> props = DataProperties<T>.Create(data);
-
         LogDataType(logger, props.DataType);
 
         if (props.FloatProps != null)
@@ -107,7 +107,6 @@ public static string Generate<T>(ReadOnlySpan<T> data, FastDataConfig fdCfg, ICo
         else if (props.StringProps != null)
             LogMinMaxLength(logger, props.StringProps.LengthData.Min, props.StringProps.LengthData.Max);
 
-        LogUserStructureType(logger, fdCfg.StructureType);
         GeneratorConfig<T> genCfg = new GeneratorConfig<T>(fdCfg.StructureType, DefaultStringComparison, props);
 
         switch (fdCfg.StructureType)

Src/FastData/Internal/Misc/HashData.cs

@@ -13,8 +13,6 @@ internal static HashData Create<T>(ReadOnlySpan<T> data, DataType dataType, int
             stringHash = new DefaultStringHash();
 
         HashFunc<T> hashFunc;
-
-        //If we have a string hash, use it.
         if (stringHash == null)
             hashFunc = PrimitiveHash.GetHash<T>(dataType);
         else
@@ -24,7 +22,7 @@ internal static HashData Create<T>(ReadOnlySpan<T> data, DataType dataType, int
 
         ulong[] hashCodes = new ulong[size];
         HashSet<ulong> uniqSet = new HashSet<ulong>();
-        HashSet<ulong> perfectSet = new HashSet<ulong>(); //TOOD: Use direct addressing
+        HashSet<ulong> perfectSet = new HashSet<ulong>(); //TODO: Use direct addressing
 
         bool uniq = true;
         bool perfect = true;