[cfe-commits] r135765 - /cfe/trunk/include/clang/AST/PrettyPrinter.h

[Chris Lattner]

On Jul 22, 2011, at 3:16 AM, Chandler Carruth wrote:

> Author: chandlerc
> Date: Fri Jul 22 05:16:15 2011
> New Revision: 135765
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=135765&view=rev
> Log:
> Add a missing forward declaration and namespace qualifier to this
> header. This showed up in client code that has a different include
> ordering after Chris's shuffle of raw_ostream. All of the Clang code
> ends up with a header included before this one that provides the needed
> type and using declaration.

Hi Chandler,

Please include AST/LLVM.h to get this forward decl.

-Chris

> 
> Modified:
>    cfe/trunk/include/clang/AST/PrettyPrinter.h
> 
> Modified: cfe/trunk/include/clang/AST/PrettyPrinter.h
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/PrettyPrinter.h?rev=135765&r1=135764&r2=135765&view=diff
> ==============================================================================
> --- cfe/trunk/include/clang/AST/PrettyPrinter.h (original)
> +++ cfe/trunk/include/clang/AST/PrettyPrinter.h Fri Jul 22 05:16:15 2011
> @@ -16,6 +16,10 @@
> 
> #include "clang/Basic/LangOptions.h"
> 
> +namespace llvm {
> +  class raw_ostream;
> +}
> +
> namespace clang {
> 
> class Stmt;
> @@ -25,7 +29,7 @@
> class PrinterHelper {
> public:
>   virtual ~PrinterHelper();
> -  virtual bool handledStmt(Stmt* E, raw_ostream& OS) = 0;
> +  virtual bool handledStmt(Stmt* E, llvm::raw_ostream& OS) = 0;
> };
> 
> /// \brief Describes how types, statements, expressions, and
> 
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits