Created attachment 550315 [details] [diff] [review]
As it turns out, most users of nsHashtable's Enumerate/Reset functions pass a callback that returns PRBool, not PRIntn. It looks like users typically use Reset() to delete entries instead of Enumerate so there is little need to pass anything other than true/false. There was only one user which actually used PRIntn/kHashEnumerateRemove, and I've converted that user to nsClassHashtable in bug 675873.
Comment on attachment 550315 [details] [diff] [review]
Can we just remove nsHashtable?
(In reply to Benjamin Smedberg [:bsmedberg] from comment #1)
> Comment on attachment 550315 [details] [diff] [review]
> Can we just remove nsHashtable?
There's enough users that it's hard to do manually, but maybe automated rewrites can help.