kolan
/
CMake
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

cmDefinitions: Remove Parent pointer. 

All structural knowledge of the stack of scopes is now external.
This commit is contained in:
Stephen Kelly 2015-04-26 16:19:11 +02:00
parent 8b1745a1c5
commit 191573f792
3 changed files with 1 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Source/cmDefinitions.cxx
View File

@ -16,12 +16,6 @@
//----------------------------------------------------------------------------
cmDefinitions::Def cmDefinitions::NoDef;
//----------------------------------------------------------------------------
cmDefinitions::cmDefinitions(cmDefinitions* parent)
: Up(parent)
{
}
//----------------------------------------------------------------------------
cmDefinitions::Def const& cmDefinitions::GetInternal(
const std::string& key,

9
Source/cmDefinitions.h
View File

@ -29,12 +29,6 @@
class cmDefinitions
{
public:
/** Construct with the given parent scope. */
cmDefinitions(cmDefinitions* parent = 0);
/** Returns the parent scope, if any. */
cmDefinitions* GetParent() const { return this->Up; }
/** Get the value associated with a key; null if none.
Store the result locally if it came from a parent. */
const char* Get(const std::string& key,
@ -71,9 +65,6 @@ private:
};
static Def NoDef;
// Parent scope, if any.
cmDefinitions* Up;
// Local definitions, set or unset.
#if defined(CMAKE_BUILD_WITH_CMAKE)
typedef cmsys::hash_map<std::string, Def> MapType;

7
Source/cmMakefile.cxx
View File

@ -53,12 +53,7 @@ public:
void PushDefinitions()
{
cmDefinitions* parent = 0;
if (!this->VarStack.empty())
{
parent = &this->VarStack.back();
}
this->VarStack.push_back(cmDefinitions(parent));
this->VarStack.push_back(cmDefinitions());
}
void InitializeDefinitions(cmMakefile* parent)