Re: [RFA 05/42] Move pending_macros to buildsym_compunit

Simon Marchi
2018-07-07T15:41:38Z

On 2018-05-23 12:58 AM, Tom Tromey wrote:
> This moves the pending_macros global into buildsym_compunit.

LGTM. As in the other patch, I would like it if the new fields
were private though.

The only access of m_pending_macros outside the class is to check if
the macro table is instantiated. So it could be replaced with this
method:

bool has_macro_table () const
{ return m_pending_macros != nullptr; }

Simon