[cfe-dev] [Patch] Remove AllowLookup argument to DeclContext::addDecl()

[Piotr Rak]

Hi,

Because DeclContext::addDecl() builds lazily lookup structure,
AllowLookup parameter works only when LookupPtr != 0.
To fix it we would have to store it somehow in DeclContext.
However, I noticed that it was no single use with other value that true.
I think, the best option is to just remove it and assume it is always true.
In case it would be some cases for it in future, we could just learn
DeclContext::buildLookup() about them. Attached patch removes this argument.
It also removes needless now ASTContext argument to DeclContext::addDecl(),
and DeclContext::insert().

Piotr
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Minor-DeclContext-changes.patch
Type: text/x-patch
Size: 20196 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20090112/c5ca7541/attachment.bin>