Promote warnings to exceptions in *scanf() functions

[kocsismate]

Split off from #4937

The case when more variables than format specifiers are passed is left as a warning.

[nikic]

The case when more variables than format specifiers are passed is left as a warning.

Any particular reason to exclude that one? Is the idea that everything else would still get populated and the extra args just ignored?

[kocsismate]

Yes, that's it. I just didn't want to be too aggressive than what is enough. However, I am happy to include this case as well if you see that it's worth.

[nikic]

Yes, that's it. I just didn't want to be too aggressive than what is enough. However, I am happy to include this case as well if you see that it's worth.

https://3v4l.org/6MW6Y => Based on the behavior here (if there are too many vars, nothing gets populated), I'd say this should definitely also throw.

[kocsismate]

OK, it's a convincing argument for sure. The goto error; should have also told me that it's the case..

[kocsismate]

Closing via 33f7cab