Re: [PATCH 4/5] Simplify FrameVars

[Tom Tromey <tromey@adacore.com>]

>>>>> "Tom" == Tom Tromey via Gdb-patches <gdb-patches@sourceware.org> writes:

Tom> FrameVars implements its own variant of Symbol.is_variable and/or
Tom> Symbol.is_control.  This patch replaces this code.

Tom> -                if self.fetch_b(sym):
Tom> +                if sym.is_variable or sym.is_constant:

I found out that using is_constant here is wrong, as this picks up local
enum constants, which aren't really interesting as variables.

In v2 of the series I fix this, and I tack on a test case at the end of
the series.

Tom