[cfe-dev] CountMap or CountMap::Factory

Ted Kremenek kremenek at apple.com
Thu Mar 6 00:09:06 PST 2008


On Mar 2, 2008, at 10:06 PM, Zhongxing Xu wrote:

> In file GRBlockCounter.cpp:
>
> GRBlockCounter::Factory::Factory(llvm::BumpPtrAllocator& Alloc) {
>   F = new CountMap::Factory(Alloc);
> }
>
> GRBlockCounter::Factory::~Factory() {
>   delete static_cast<CountMap*>(F);
> }
>
> Should the dtor cast F to CountMap::Factory?
>

Hi Zhongxing,

You are absolutely correct.  This is a bug.  Fixed!

Thanks so much.
Ted



More information about the cfe-dev mailing list