r220940 - ASTDumper.cpp: Appease g++, for now.
Richard Smith
richard at metafoo.co.uk
Thu Oct 30 17:50:45 PDT 2014
On Thu, Oct 30, 2014 at 5:30 PM, NAKAMURA Takumi <geek4civic at gmail.com>
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?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141030/2ca27fc2/attachment.html>
More information about the cfe-commits
mailing list