[Lldb-commits] [PATCH 1/3] struct ParserVars warning fix.

Stephen Wilson wilsons at start.ca
Mon Apr 11 12:50:16 PDT 2011


Committed in r129290 and r129289 (I folded patch 3/3 into 2/3).

Marco, I forgot to credit you in the changelog in one of the patches.
My apologies, it completely slipped my mind.




On Mon, Apr 11, 2011 at 07:53:24AM -0700, Greg Clayton wrote:
> Me too, whomever can commit these first can feel free to do so!
> 
> On Apr 11, 2011, at 4:52 AM, Stephen Wilson wrote:
> 
> > 
> > All of these look good to me.  Built and tested.
> > 
> > 
> > On Mon, Apr 11, 2011 at 10:38:20AM +0200, Marco Minutoli wrote:
> >> ParserVars is declared using the class keyword. This solve the
> >> warning.
> >> ---
> >> include/lldb/Expression/ClangExpressionDeclMap.h  |    2 +-
> >> include/lldb/Expression/ClangExpressionVariable.h |    2 +-
> >> 2 files changed, 2 insertions(+), 2 deletions(-)
> >> 
> >> diff --git a/include/lldb/Expression/ClangExpressionDeclMap.h b/include/lldb/Expression/ClangExpressionDeclMap.h
> >> index b0a08f7..ebae71d 100644
> >> --- a/include/lldb/Expression/ClangExpressionDeclMap.h
> >> +++ b/include/lldb/Expression/ClangExpressionDeclMap.h
> >> @@ -509,7 +509,7 @@ private:
> >>     EnableParserVars()
> >>     {
> >>         if (!m_parser_vars.get())
> >> -            m_parser_vars.reset(new struct ParserVars);
> >> +            m_parser_vars.reset(new ParserVars);
> >>     }
> >> 
> >>     //----------------------------------------------------------------------
> >> diff --git a/include/lldb/Expression/ClangExpressionVariable.h b/include/lldb/Expression/ClangExpressionVariable.h
> >> index dc6b7f8..a1a9deb 100644
> >> --- a/include/lldb/Expression/ClangExpressionVariable.h
> >> +++ b/include/lldb/Expression/ClangExpressionVariable.h
> >> @@ -117,7 +117,7 @@ public:
> >>     EnableParserVars()
> >>     {
> >>         if (!m_parser_vars.get())
> >> -            m_parser_vars.reset(new struct ParserVars);
> >> +            m_parser_vars.reset(new ParserVars);
> >>     }
> >> 
> >>     //----------------------------------------------------------------------
> >> -- 
> >> 1.7.1
> >> 
> >> _______________________________________________
> >> lldb-commits mailing list
> >> lldb-commits at cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
> > 
> > -- 
> > steve
> > 
> > _______________________________________________
> > lldb-commits mailing list
> > lldb-commits at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

-- 
steve




More information about the lldb-commits mailing list