[Mono-bugs] [Bug 22926] Changed - FactoryCache is set to low I think...

[bugzilla-daemon@rocky.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by dihlewis@yahoo.co.uk.

http://bugzilla.ximian.com/show_bug.cgi?id=22926

--- shadow/22926	Tue Apr  2 02:00:36 2002
+++ shadow/22926.tmp.5901	Sat Apr  6 12:32:40 2002
@@ -1,14 +1,14 @@
 Bug#: 22926
 Product: Mono/Class Libraries
 Version: unspecified
-OS: 
+OS: unknown
 OS Details: 
-Status: NEW   
-Resolution: 
-Severity: 
+Status: RESOLVED   
+Resolution: FIXED
+Severity: Unknown
 Priority: Wishlist
 Component: System
 AssignedTo: mono-bugs@ximian.com                            
 ReportedBy: manyoso@yahoo.com               
 QAContact: mono-bugs@ximian.com
 TargetMilestone: ---
@@ -42,6 +42,19 @@
 #0: 0x00030 stfld      in System.Text.RegularExpressions.MRUList::Evict ()
 #1: 0x00019 callvirt   in System.Text.RegularExpressions.FactoryCache::Add ([0x80c4fb0] [0] [0x81f349c] )
 #2: 0x00091 callvirt   in System.Text.RegularExpressions.Regex::.ctor ([0x80c4fb0] [0] )
 #3: 0x00002 newobj     in System.Text.RegularExpressions.Regex::IsMatch ([0x80c4df8] [0x80c4fb0] )
 #4: 0x00007 call       in .Foo::IsBlah ([0x80c4df8] )
 #5: 0x0000b call       in .Foo::Main ([0x80c4e20] )
+
+------- Additional Comments From dihlewis@yahoo.co.uk  2002-04-06 12:32 -------
+The key I was using for the cache defined "new int GetHashCode" 
+rather than "override int GetHashCode", so Hashtable was using the 
+default ValueType hashing function (which is to return zero).
+
+Windows has a more sophisticated ValueType.GetHashCode() which 
+appears to hash on the contents of the struct. This is why the code 
+worked with the windows corlib, but not ours.
+
+The null reference bug in the Evict method seen above was fixed in 
+CVS on 3 Apr 2002.
+