<div dir="ltr">Hi,<div><br></div><div>renaming for TMormotHashFactory to TGenericsHashFactory is somehow acceptable (but IMO bad taste) but change for default hashing algorithm in such important library is really really really bad idea.</div><div><br></div><div>This was not consulted, change like that or proposition should be reported here :</div><div><br></div><div><a href="https://github.com/maciej-izak/generics.collections/issues">https://github.com/maciej-izak/generics.collections/issues</a><br></div><div><br></div><div>OR IN ANY OTHER WAY, not in comment in other bug report where can be easily omitted (!). 

<div style="font-size:small;text-decoration-style:initial;text-decoration-color:initial">The problem with thashmapextendedequalitycomparer example was really minor and should be fixed in other way not by changing *default hashing algorithm* for whole library (!). </div><div style="font-size:small;text-decoration-style:initial;text-decoration-color:initial"><br></div><div style="font-size:small;text-decoration-style:initial;text-decoration-color:initial">Widely used TDictionary<TKey, TValue> is also affected by this change.</div><div style="font-size:small;text-decoration-style:initial;text-decoration-color:initial"> </div></div><div>Results BEFORE Michael Van Canneyt changes:</div><div><br></div><div>

<div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><span class="gmail-il">THashSet</span><string> add    2000000 items. Time elapsed : 1844<br></div><div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><div><span class="gmail-il">THashSet</span><string> lookup 2000000 items. Time elapsed : 812</div><div>TSortedHashSet<string> add      2000000 items. Time elapsed : 18640</div><div>TSortedHashSet<string> lookup   2000000 items. Time elapsed : 641</div><div><br></div></div><div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><span class="gmail-il">THashSet</span><integer> add   10000000 items. Time elapsed : 1328</div><div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><span class="gmail-il">THashSet</span><integer> lookup        10000000 items. Time elapsed : 640</div><div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">TSortedHashSet<integer> add     10000000 items. Time elapsed : 4235</div><div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial">TSortedHashSet<integer> lookup  10000000 items. Time elapsed : 765</div><div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><br></div><div style="background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><span style="font-size:12.8px">Results AFTER Michael Van Canneyt changes:</span></div><div style="font-size:12.8px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><br></div><div style="background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial"><div><span style="font-size:12.8px">THashSet<string> add    2000000 items. Time elapsed : 2250</span></div><div><span style="font-size:12.8px">THashSet<string> lookup 2000000 items. Time elapsed : 906</span></div><div><span style="font-size:12.8px">TSortedHashSet<string> add      2000000 items. Time elapsed : 18828</span></div><div><span style="font-size:12.8px">TSortedHashSet<string> lookup   2000000 items. Time elapsed : 688</span></div><div><br></div><div><span style="font-size:12.8px">THashSet<integer> add   10000000 items. Time elapsed : 2219</span></div><div><span style="font-size:12.8px">THashSet<integer> lookup        10000000 items. Time elapsed : 1140</span></div><div><span style="font-size:12.8px">TSortedHashSet<integer> add     10000000 items. Time elapsed : 5250</span></div><div><span style="font-size:12.8px">TSortedHashSet<integer> lookup  10000000 items. Time elapsed : 1234</span></div></div>

</div><div><br></div><div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Best regards,<br>Maciej Izak</div></div></div>
</div></div>