Regression when using --enable-build-with-cxx (was: Re: [PATCH] Non bit-packed packed arrays as variable-length fields)

Tue May 19 18:31:00 GMT 2015

On Friday, May 15 2015, Joel Brobecker wrote:

> From: Jerome Guitton <guitton@adacore.com>
>
> On behalf of Jerome Guitton:
>
> In the case of non bit-packed arrays, GNAT does not generate its
> traditional XP encoding; it is not needed. However, it still generates
> the so-called "implementation type" with a P suffix. This
> implementation type shall be skipped when looking for other
> descriptive types such as XA encodings for variable-length
> fields.
>
> Note also that there may be an intermediate typedef between the
> implementation type and its XA description. It shall be skipped
> as well.

Hey Joel!

This commit broke --enable-build-with-cxx, as can be seen on BuildBot:

  <http://gdb-build.sergiodj.net/builders/Fedora-x86_64-cxx-build-m64/builds/18/steps/compile%20gdb/logs/stdio>

I think the main reason is that 'typename' is a reserved C++ keyword.
Should be straightforward to fix :-).

Cheers,

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/