On Thu, Oct 30, 2014 at 5:30 PM, NAKAMURA Takumi <[email protected]> wrote:
> Author: chapuni > Date: Thu Oct 30 19:30:37 2014 > New Revision: 220940 > > URL: http://llvm.org/viewvc/llvm-project?rev=220940&view=rev > Log: > ASTDumper.cpp: Appease g++, for now. > Thanks! > Modified: > cfe/trunk/lib/AST/ASTDumper.cpp > > Modified: cfe/trunk/lib/AST/ASTDumper.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/AST/ASTDumper.cpp?rev=220940&r1=220939&r2=220940&view=diff > > ============================================================================== > --- cfe/trunk/lib/AST/ASTDumper.cpp (original) > +++ cfe/trunk/lib/AST/ASTDumper.cpp Thu Oct 30 19:30:37 2014 > @@ -152,8 +152,8 @@ namespace { > OS << '\n'; > ColorScope Color(*this, IndentColor); > OS << Prefix << (isLastChild ? '`' : '|') << '-'; > - Prefix.push_back(isLastChild ? ' ' : '|'); > - Prefix.push_back(' '); > + this->Prefix.push_back(isLastChild ? ' ' : '|'); > + this->Prefix.push_back(' '); > } > > FirstChild = true; > @@ -166,11 +166,11 @@ namespace { > // Dump those ones out now. > while (Depth < Pending.size()) { > Pending.back()(true); > Do you need to also change these references to Pending... > - Pending.pop_back(); > + this->Pending.pop_back(); > } > > // Restore the old prefix. > - Prefix.resize(Prefix.size() - 2); > + this->Prefix.resize(Prefix.size() - 2); > ... and the second reference to Prefix here? What error does g++ give here?
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
