Re: 78270: Subject: widl: duplicate type in case of argument with range attribute

2011-09-01 Thread buildbot
The Buildbot has detected a failed build on builder runtests while building Wine. Full details are available at: http://buildbot.kegel.com/builders/runtests/builds/215 BUILD FAILED: failed git For more info about this message, see http://wiki.winehq.org/BuildBot

Re: 78272: Subject: widl: duplicate type in case of argument with range attribute

2011-09-01 Thread buildbot
The Buildbot has detected a failed build on builder runtests while building Wine. Full details are available at: http://buildbot.kegel.com/builders/runtests/builds/217 BUILD FAILED: failed git For more info about this message, see http://wiki.winehq.org/BuildBot

Re: widl: duplicate type in case of argument with range attribute

2011-08-30 Thread Alexandre Julliard
Jérôme Gardou writes: > Anyway, I've looked into this a bit, and it seems that the place to > write a proper test is rpcrt4 test suite. > While writing the test would per se be very easy, the problem won't > show up if -Oif isn't set on the tested idl file. Adding this to the > EXTRAIDLFLAGS vari

Re: widl: duplicate type in case of argument with range attribute

2011-08-30 Thread Jérôme Gardou
Le 30/08/2011 17:50, Jérôme Gardou a écrit : Hello. Maybe the comment isn't clear enough. The ranges are correctly output into the format string, but the offset of each output is lost during the processing. For instance, declaring a function like this one : DWORD Test( [in, range(0, 102

Re: widl: duplicate type in case of argument with range attribute

2011-08-30 Thread Jérôme Gardou
Hello. Maybe the comment isn't clear enough. The ranges are correctly output into the format string, but the offset of each output is lost during the processing. For instance, declaring a function like this one : DWORD Test( [in, range(0, 1024)] DWORD dwParam1, [in, range(0, 255)] DW

Re: widl: duplicate type in case of argument with range attribute

2011-08-30 Thread Alexandre Julliard
Jérôme Gardou writes: > @@ -1418,6 +1418,11 @@ static var_t *declare_var(attr_list_t *attrs, > decl_spec_t *decl_spec, const decl >type_t *func_type = decl ? decl->func_type : NULL; >type_t *type = decl_spec->type; > > + /* In case of a range attribute, duplicate the type to keep trac