Mike could you review this for me. This fixes the problem. I don't think it introduces any new problems. The problem was that if the unresolved entry was found first then when we hit the resolved entry we would not look for the record that had already been added to the name table. So, The iid table would get the new record, but the name table ends up with two records for the same name. When it later does a lookup by name it finds the unresolved record. I also had to fix two other things while I was here.. I changed "found in IIDTable" to "shouldAddToIIDTable" (and switched the logic) because it was driving me nutty. I also made the 'zero' iid into a static so it would not get init'd every time through. [it just occered to me to make it const too: "static const nsID zero =...". This is not in the dif. But it compiles and runs.]
John - Your diff looks great. Thanks for handling this. Mike
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
fix checked in.
jband, could you please verify this one ?
You need to log in before you can comment on or make changes to this bug.