Re: [PATCH v3 00/10] Remove `expout*' globals from parser-defs.h

[Sergio Durigan Junior <sergiodj at redhat dot com>]

On Saturday, January 25 2014, I wrote:

> Hi,

Ping.

I'll try to resubmit a new version of this patch tomorrow, but until
then, this version is good enough for review.

> This is the third attempt to remove some globals from GDB's parser.  You
> can see the last attempt here:
>
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00054.html>
>
> (Yeah, quite a while ago...)
>
> I've addressed the comments made by Tom here:
>
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00096.html>
>
> And by Doug here:
>
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00071.html>
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00424.html>
>
> Some of the patches from this series were also approved when I submitted
> the second attempt:
>
> - Language independent
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00093.html>
>
> - SystemTap integration
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00094.html>
>
> - Ada
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00406.html>
>
> - Fortran
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00409.html>
>
> - Modula-2
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00542.html>
>
> - Pascal
> <https://sourceware.org/ml/gdb-patches/2012-06/msg00113.html>
>
>
> However, I am resubmitting them nevertheless, this time with ChangeLogs.
> Take a look if you want/can!
>
> Other than that, there's not much to say.  The patches compile OK on
> x86_64 Fedora 17/18.
>
> OK to apply?
>
>  gdb/aarch64-linux-tdep.c |  26 +-
>  gdb/ada-exp.y            | 488 ++++++++++++++++++---------------
>  gdb/ada-lang.c           |   4 +-
>  gdb/ada-lang.h           |   3 +-
>  gdb/ada-lex.l            |  54 ++--
>  gdb/arm-linux-tdep.c     |  26 +-
>  gdb/c-exp.y              | 698 +++++++++++++++++++++++++----------------------
>  gdb/c-lang.h             |   3 +-
>  gdb/f-exp.y              | 276 +++++++++++--------
>  gdb/f-lang.h             |   3 +-
>  gdb/go-exp.y             | 268 +++++++++---------
>  gdb/go-lang.h            |   4 +-
>  gdb/i386-tdep.c          |  88 +++---
>  gdb/jv-exp.y             | 368 +++++++++++++------------
>  gdb/jv-lang.h            |   3 +-
>  gdb/language.c           |   4 +-
>  gdb/language.h           |   3 +-
>  gdb/m2-exp.y             | 275 +++++++++++--------
>  gdb/m2-lang.h            |   3 +-
>  gdb/objc-lang.c          |   8 +-
>  gdb/objc-lang.h          |   3 +-
>  gdb/p-exp.y              | 352 +++++++++++++-----------
>  gdb/p-lang.h             |   3 +-
>  gdb/parse.c              | 277 ++++++++++---------
>  gdb/parser-defs.h        |  83 ++++--
>  gdb/ppc-linux-tdep.c     |   6 +-
>  gdb/stap-probe.c         |  64 ++---
>  gdb/stap-probe.h         |   6 +
>  gdb/utils.c              |  18 ++
>  gdb/utils.h              |   4 +
>  30 files changed, 1873 insertions(+), 1548 deletions(-)

-- 
Sergio