updateLookupWithKey bug? Was: Data.Map,
Data.IntMap documentation (1611)
ahey at iee.org
Fri Nov 2 09:59:09 EDT 2007
Andriy Palamarchuk wrote:
> Adrian, question whether to deprecate
> updateLookupWithKey is worth discussing, but we need
> to fix the function anyway.
I would say just document the code as it is, warts and all.
Don't change the functions because maybe someone is using them
(though I suspect this is not the case).
But also add a DEPRECATE pragma to discourage further use.
Maybe do this for both IntMap and Map, because even if they were
consistent, I don't think they're very useful. An API that clearly
separates the process of searching for the key from whatever you want
to do when it's found (or not) is so much more flexible and easy to
BTW, the way I did this should also work for IntMap tries too AFAICT,
but I'm not sure it would offer any performance advantage over a
repeated lookup in this case. So maybe keep them for IntMap?
More information about the Libraries