From: Hans Wennborg Date: Tue, 17 Sep 2013 23:37:44 +0000 (+0000) Subject: Move using directive from Job.h to Job.cpp X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=f1666bbaba75eff59ae47e42d6d601e3d36781ff;p=clang Move using directive from Job.h to Job.cpp I don't think it belongs in the header, but seems handy in the .cpp file. git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@190902 91177308-0d34-0410-b5e6-96231b3b80d8 --- diff --git a/include/clang/Driver/Job.h b/include/clang/Driver/Job.h index 161fda19b2..c4e6a31185 100644 --- a/include/clang/Driver/Job.h +++ b/include/clang/Driver/Job.h @@ -24,8 +24,6 @@ class Action; class Command; class Tool; -using llvm::opt::ArgStringList; - class Job { public: enum JobClass { diff --git a/lib/Driver/Job.cpp b/lib/Driver/Job.cpp index 0a783eb93b..43518ad8f9 100644 --- a/lib/Driver/Job.cpp +++ b/lib/Driver/Job.cpp @@ -15,14 +15,14 @@ #include "llvm/Support/raw_ostream.h" #include using namespace clang::driver; +using llvm::opt::ArgStringList; using llvm::raw_ostream; using llvm::StringRef; Job::~Job() {} Command::Command(const Action &_Source, const Tool &_Creator, - const char *_Executable, - const llvm::opt::ArgStringList &_Arguments) + const char *_Executable, const ArgStringList &_Arguments) : Job(CommandClass), Source(_Source), Creator(_Creator), Executable(_Executable), Arguments(_Arguments) {} @@ -113,7 +113,7 @@ void Command::Print(raw_ostream &OS, const char *Terminator, bool Quote, OS << Terminator; } -int Command::Execute(const llvm::StringRef **Redirects, std::string *ErrMsg, +int Command::Execute(const StringRef **Redirects, std::string *ErrMsg, bool *ExecutionFailed) const { SmallVector Argv; Argv.push_back(Executable);