A bit belated, I have now committed the patch as Rev. 211587.
Thanks for confirming that it now works!
Tobias
Bernd Schmidt wrote:
On 06/04/2014 10:36 PM, Tobias Burnus wrote:
Bernd Schmidt wrote:
Even with this applied, I'm still seeing similar failures.
I didn't claim that the patch would fix everything – nor that it was
well tested.
Just wanted to report back since the problem doesn't really show up on
normal targets.
Can you try the attached version? The change is that I now properly use
"se->ignore_optional" to test whether absent optional arguments should
be skipped - rather than using this mornings ad-hoc solution of doing so
unconditionally. Additionally, the patch has now survived stage2
building – which is more testing than I could do this morning.
This seems to work. Thanks!
Bernd
