Reverted implementation of ImmutableMap::find() to return a TreeTy* instead of

an iterator, since the implementation returned an iterator that pointed to a
different node! Renamed this implementation to SlimFind() so that users do not
expect it to return an iterator (it is a more efficient implementation than
returning an iterator if the user just wants to find the value of a key).

Added a FIXME to implement ImmutableMap::find() that returns an iterator.

llvm-svn: 46150
This commit is contained in:
Ted Kremenek 2008-01-18 00:38:04 +00:00
parent 8ef89eabc2
commit ab405774b6
1 changed files with 5 additions and 3 deletions

View File

@ -188,15 +188,17 @@ public:
iterator begin() const { return iterator(Root); }
iterator end() const { return iterator(); }
iterator find(key_type_ref K) const {
TreeTy* SlimFind(key_type_ref K) const {
if (Root) {
TreeTy* T = Root->find(K);
if (T) return iterator(T);
if (T) return T;
}
return iterator();
return NULL;
}
// FIXME: Add 'find' that returns an iterator instead of a TreeTy*.
//===--------------------------------------------------===//
// Utility methods.
//===--------------------------------------------------===//