r190906 - Revert "Move using directive from Job.h to Job.cpp" (r190902)

[Hans Wennborg]

Author: hans
Date: Tue Sep 17 19:17:07 2013
New Revision: 190906

URL: http://llvm.org/viewvc/llvm-project?rev=190906&view=rev
Log:
Revert "Move using directive from Job.h to Job.cpp" (r190902)

Seems like it was intentional to export ArgStringList as
driver::ArgStringList, and e.g. examples/clang-interpreter/main.cpp
uses it this way.

However, exporting it with a typedef seems like a more common way to do it.

Modified:
    cfe/trunk/include/clang/Driver/Job.h
    cfe/trunk/lib/Driver/Job.cpp

Modified: cfe/trunk/include/clang/Driver/Job.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/Job.h?rev=190906&r1=190905&r2=190906&view=diff
==============================================================================
--- cfe/trunk/include/clang/Driver/Job.h (original)
+++ cfe/trunk/include/clang/Driver/Job.h Tue Sep 17 19:17:07 2013
@@ -24,6 +24,9 @@ class Action;
 class Command;
 class Tool;
 
+// Users of this class will use clang::driver::ArgStringList.
+typedef llvm::opt::ArgStringList ArgStringList;
+
 class Job {
 public:
   enum JobClass {
@@ -65,11 +68,11 @@ class Command : public Job {
 
   /// The list of program arguments (not including the implicit first
   /// argument, which will be the executable).
-  llvm::opt::ArgStringList Arguments;
+  ArgStringList Arguments;
 
 public:
   Command(const Action &_Source, const Tool &_Creator, const char *_Executable,
-          const llvm::opt::ArgStringList &_Arguments);
+          const ArgStringList &_Arguments);
 
   virtual void Print(llvm::raw_ostream &OS, const char *Terminator,
                      bool Quote, bool CrashReport = false) const;
@@ -83,7 +86,7 @@ public:
   /// getCreator - Return the Tool which caused the creation of this job.
   const Tool &getCreator() const { return Creator; }
 
-  const llvm::opt::ArgStringList &getArguments() const { return Arguments; }
+  const ArgStringList &getArguments() const { return Arguments; }
 
   static bool classof(const Job *J) {
     return J->getKind() == CommandClass;

Modified: cfe/trunk/lib/Driver/Job.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Job.cpp?rev=190906&r1=190905&r2=190906&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/Job.cpp (original)
+++ cfe/trunk/lib/Driver/Job.cpp Tue Sep 17 19:17:07 2013
@@ -15,7 +15,6 @@
 #include "llvm/Support/raw_ostream.h"
 #include <cassert>
 using namespace clang::driver;
-using llvm::opt::ArgStringList;
 using llvm::raw_ostream;
 using llvm::StringRef;