Re: [PATCH] Fix for PR c++/15203 and PR c++/15210

[Tom Tromey <tromey@redhat.com>]

>>>>> "Sergio" == Sergio Durigan Junior <sergiodj@redhat.com> writes:

Sergio> And then, printing the value of `var':
Sergio>     (gdb) print 'foo::bar()::var'
Sergio> GDB would fall in an internal_error:

Sergio> This may not be the ideal solution, according to Keith it would
Sergio> be good to implement productions on c-exp.y in order to
Sergio> recognize CLASS::FUNCTION::VARIABLE, but it is a solution which
Sergio> works with what we have today.

I'm not sure about this idea.  Extensions make me a bit nervous, I
suppose.

Sergio> +      else if (sym->flags & (BSF_GLOBAL | BSF_LOCAL | BSF_WEAK | BSF_GNU_UNIQUE))

I think this line is too long.

Sergio> +# Regression test for PR c++/15203 and PR c++/15210
Sergio> +gdb_test "print 'gnu_obj_1::method()::sintvar'" "\\$\[0-9\]+ = 4" \
Sergio> +	   "simple object, static const int, accessing via 'class::method::variable"

I think the second line should only be indented 4 spaces.

Ok with those changes.

Tom