[Cmake-commits] [cmake-commits] david.cole committed cmStringCommand.cxx 1.30 1.31

[cmake-commits at cmake.org]

Update of /cvsroot/CMake/CMake/Source
In directory public:/mounts/ram/cvs-serv16195/Source

Modified Files:
	cmStringCommand.cxx 
Log Message:
Correct some typos in error messages in the string command. Add a test that covers more of the code implemented in cmStringCommand.cxx, especially the error handlers.


Index: cmStringCommand.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/cmStringCommand.cxx,v
retrieving revision 1.30
retrieving revision 1.31
diff -C 2 -d -r1.30 -r1.31
*** cmStringCommand.cxx	30 Sep 2009 15:41:34 -0000	1.30
--- cmStringCommand.cxx	2 Oct 2009 18:51:43 -0000	1.31
***************
*** 382,386 ****
          std::string e = "sub-command REGEX, mode REPLACE: Unknown escape \"";
          e += replace.substr(r, 2);
!         e += "\"in replace-expression.";
          this->SetError(e.c_str());
          return false;
--- 382,386 ----
          std::string e = "sub-command REGEX, mode REPLACE: Unknown escape \"";
          e += replace.substr(r, 2);
!         e += "\" in replace-expression.";
          this->SetError(e.c_str());
          return false;
***************
*** 560,564 ****
    if(args.size() < 5)
      {
!     this->SetError("sub-command REPLACE requires four arguments.");
      return false;
      }
--- 560,564 ----
    if(args.size() < 5)
      {
!     this->SetError("sub-command REPLACE requires at least four arguments.");
      return false;
      }
***************
*** 587,591 ****
    if(args.size() != 5)
      {
!     this->SetError("sub-command REPLACE requires four arguments.");
      return false;
      }
--- 587,591 ----
    if(args.size() != 5)
      {
!     this->SetError("sub-command SUBSTRING requires four arguments.");
      return false;
      }
***************
*** 648,652 ****
   if(args.size() != 3)
      {
!     this->SetError("sub-command LENGTH requires two arguments.");
      return false;
      }
--- 648,652 ----
   if(args.size() != 3)
      {
!     this->SetError("sub-command STRIP requires two arguments.");
      return false;
      }