consolidated IF handling and added checks for bad arguments

1 files changed, 70 insertions, 44 deletions

diff --git a/Source/cmIfCommand.cxx b/Source/cmIfCommand.cxx
index 2a36853..5d207d4 100644
--- a/Source/cmIfCommand.cxx
+++ b/Source/cmIfCommand.cxx
@@ -71,75 +71,108 @@ ScopeEnded(cmMakefile &mf)
 
 bool cmIfCommand::InitialPass(std::vector<std::string> const& args)
 {
-  if(args.size() < 1 )
+  bool isValid;
+  bool isTrue = cmIfCommand::IsTrue(args,isValid,m_Makefile);
+  
+  if (!isValid)
     {
-    this->SetError("called with incorrect number of arguments");
+    this->SetError("An IF command had incorrect arguments");
     return false;
     }
-  // create a function blocker
-  cmIfFunctionBlocker *f = NULL;
+  
+  // if is isn't true create a blocker
+  if (!isTrue)
+    {
+    cmIfFunctionBlocker *f = new cmIfFunctionBlocker();
+    for(std::vector<std::string>::const_iterator j = args.begin();
+        j != args.end(); ++j)
+      {   
+      f->m_Args.push_back(*j);
+      }
+    m_Makefile->AddFunctionBlocker(f);
+    }
+  
+  return true;
+}
 
+bool cmIfCommand::IsTrue(const std::vector<std::string> &args, bool &isValid,
+                         const cmMakefile *makefile)
+{
   // check for the different signatures
+  bool isTrue = true;
+  isValid = false;
   const char *def;
   const char *def2;
 
+  if(args.size() < 1 )
+    {
+    return false;
+    }
+
   if (args.size() == 1)
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
     if(cmSystemTools::IsOff(def))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
   if (args.size() == 2 && (args[0] == "NOT"))
     {
-    def = m_Makefile->GetDefinition(args[1].c_str());
+    def = makefile->GetDefinition(args[1].c_str());
     if(!cmSystemTools::IsOff(def))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
+
     }
 
   if (args.size() == 2 && (args[0] == "COMMAND"))
     {
-    if(!m_Makefile->CommandExists(args[1].c_str()))
+    if(!makefile->CommandExists(args[1].c_str()))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
   if (args.size() == 2 && (args[0] == "EXISTS"))
     {
     if(!cmSystemTools::FileExists(args[1].c_str()))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
   if (args.size() == 3 && (args[1] == "AND"))
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
-    def2 = m_Makefile->GetDefinition(args[2].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
+    def2 = makefile->GetDefinition(args[2].c_str());
     if(cmSystemTools::IsOff(def) || cmSystemTools::IsOff(def2))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
   
   if (args.size() == 3 && (args[1] == "OR"))
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
-    def2 = m_Makefile->GetDefinition(args[2].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
+    def2 = makefile->GetDefinition(args[2].c_str());
     if(cmSystemTools::IsOff(def) && cmSystemTools::IsOff(def2))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
   if (args.size() == 3 && (args[1] == "MATCHES"))
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
     if (!def)
       {
       def = args[0].c_str();
@@ -149,14 +182,15 @@ bool cmIfCommand::InitialPass(std::vector<std::string> const& args)
     // check for black line or comment
     if (!regEntry.find(def))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
   
   if (args.size() == 3 && (args[1] == "LESS"))
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
-    def2 = m_Makefile->GetDefinition(args[2].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
+    def2 = makefile->GetDefinition(args[2].c_str());
     if (!def)
       {
       def = args[0].c_str();
@@ -167,14 +201,15 @@ bool cmIfCommand::InitialPass(std::vector<std::string> const& args)
       }    
     if(atof(def) >= atof(def2))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
   if (args.size() == 3 && (args[1] == "GREATER"))
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
-    def2 = m_Makefile->GetDefinition(args[2].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
+    def2 = makefile->GetDefinition(args[2].c_str());
     if (!def)
       {
       def = args[0].c_str();
@@ -185,41 +220,32 @@ bool cmIfCommand::InitialPass(std::vector<std::string> const& args)
       }    
     if(atof(def) <= atof(def2))
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
   if (args.size() == 3 && (args[1] == "STRLESS"))
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
-    def2 = m_Makefile->GetDefinition(args[2].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
+    def2 = makefile->GetDefinition(args[2].c_str());
     if(strcmp(def,def2) >= 0)
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
   if (args.size() == 3 && (args[1] == "STRGREATER"))
     {
-    def = m_Makefile->GetDefinition(args[0].c_str());
-    def2 = m_Makefile->GetDefinition(args[2].c_str());
+    def = makefile->GetDefinition(args[0].c_str());
+    def2 = makefile->GetDefinition(args[2].c_str());
     if(strcmp(def,def2) <= 0)
       {
-      f = new cmIfFunctionBlocker();
+      isTrue = false;
       }
+    isValid = true;
     }
 
-  // if we created a function blocker then set its args
-  if (f)
-    {
-    for(std::vector<std::string>::const_iterator j = args.begin();
-        j != args.end(); ++j)
-      {   
-      f->m_Args.push_back(*j);
-      }
-    m_Makefile->AddFunctionBlocker(f);
-    }
-  
-  return true;
+  return isTrue;
 }
-