This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[RFA/commit] #include language.h in expression.h


Hello,

Also while working on using "struct parse_context", I found that
expression.h is using some declarations made in language.h. As of
now, the only declaration that it is using is "struct language_defn *".
So we could possible add an advance stub declaration to fix the problem.

But in "[RFC/RFA] Introduce new struct parse_context"
(http://www.sourceware.org/ml/gdb-patches/2007-12/msg00255.html),
I am introducing the use of current_language through the
current_parse_context macro too.

I think we are better off including "language.h" from "expression.h"
rather than not, because we will then be certain that any file including
expression.h can use our macro without needing another include. This is
what the attached patch does.

2007-12-17  Joel Brobecker  <brobecker@adacore.com>

        * expression.h: Add #include "language.h".
        * Makefile.in (expression_h): Add dependency on language.h.

Any objections?

-- 
Joel

Attachment: expression.h.diff
Description: Text document


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]