gamzinav/cling
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Avoid double deletion of the ASTContext's external source.

Browse Source 
This is another hack, working around the lack of multiplexing external AST source.
This commit is contained in:
Vassil Vassilev 2013-10-03 11:43:52 +02:00 committed by sftnight
parent 2af379d972
commit 14ef88e0ad
1 changed files with 24 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
lib/Interpreter/InterpreterCallbacks.cpp
View File

@ -8,6 +8,7 @@
#include "cling/Interpreter/Interpreter.h"
#include "clang/AST/ASTContext.h"
#include "clang/Frontend/CompilerInstance.h"
#include "clang/Lex/Preprocessor.h"
#include "clang/Sema/Sema.h"
@ -70,10 +71,30 @@ namespace cling {
///
InterpreterCallbacks* m_Callbacks; // we don't own it.
public:
InterpreterExternalSemaSource(InterpreterCallbacks* C) : m_Callbacks(C){}
Sema* m_Sema; // we don't own // FIXME: once we remove ForgetSema delete.
~InterpreterExternalSemaSource() {}
public:
InterpreterExternalSemaSource(InterpreterCallbacks* C)
: m_Callbacks(C), m_Sema(0) {}
~InterpreterExternalSemaSource() {
// FIXME: Another gross hack due to the missing multiplexing AST external
// source see Interpreter::setCallbacks.
if (m_Sema) {
ASTContext& C = m_Sema->getASTContext();
// tell the owning ptr to not delete it, the callbacks will delete it.
if (C.ExternalSource.take() == this)
C.ExternalSource.reset(0);
}
}
virtual void InitializeSema(Sema& S) {
m_Sema = &S;
}
virtual void ForgetSema() {
m_Sema = 0;
}
InterpreterCallbacks* getCallbacks() const { return m_Callbacks; }