Re: Remove cpp_quote hackery from wined3d.idl (was: tools/widl: Split expr_int_const off from expr)

2009-12-24 Thread Henri Verbeet
2009/12/24 Gerald Pfeifer : > On Mon, 21 Dec 2009, Henri Verbeet wrote: >>> How shall we best tackle this? >> At least for the wined3d ones, we might as well just add them to the >> enum, the macros aren't strictly necessary. > > Done thusly.  We just need to move WINEMAKEFOURCC to a local header,

Re: tools/widl: Split expr_int_const off from expr

2009-12-21 Thread Henri Verbeet
2009/12/21 Gerald Pfeifer : > How shall we best tackle this? > At least for the wined3d ones, we might as well just add them to the enum, the macros aren't strictly necessary.

Re: tools/widl: Split expr_int_const off from expr

2009-12-21 Thread Gerald Pfeifer
Hi Rob, first of all sorry for the delay in getting back to this. I kept looking for alternative approaches, but now ended up figuring that really laying down the issue at hand I was trying to solve in front of everyone a priori will be the better approach. Let me first respond to concrete feedba

Re: tools/widl: Split expr_int_const off from expr

2009-07-02 Thread Rob Shearman
2009/6/28 Gerald Pfeifer : > diff --git a/tools/widl/parser.y b/tools/widl/parser.y > index c2f1abc..01aa060 100644 > --- a/tools/widl/parser.y > +++ b/tools/widl/parser.y > @@ -622,56 +633,54 @@ m_expr:                                           { $$ > = make_expr(EXPR_VOID); } >        | expr >