Hi Andre,
> Because this patch is obvious I plan to commit it tomorrow if no one objects?!
The patch itself is obviously OK.
About the test case: In general, it is better not to change existing
test cases unless absolutely necessary (e.g. adjust an error message).
This makes it easier to track regressions.
I would prefer if you made a new test case from your existing one,
with the changes you did and a small explanation of what was
tested in the comments.
If you are worried about runtime for an additonal test, you can use the
! { dg-do run }
hack (notice the two spaces between the dg-do and the run) to have the
test case execute only once.
OK with that change.
Regards
Thomas
