[cfe-dev] Bug in LocationContextManager

[Zhongxing Xu]

2009/11/3 Olaf Krzikalla <Olaf.Krzikalla at tu-dresden.de>:
> Zhongxing Xu schrieb:
>>
>> Fortunately it was not causing bug because only AddPointer() and
>> AddInteger() are used in the profiling.
>
> Indeed it did. In LocationContextManager::getStackFrame a static call to
> StackFrameContext::Profile profiles the (maybe) new element.
> The elements in the container however were profiled using
> LocationContext::Profile only.

Ah, right. FoldingSetImpl::FindNodeOrInsertPos() uses the Profile() in
trait. The old implementation was not virtual. So only the method in
the base class was used. Thanks Olaf.

>
> Best Olaf
>
>