Repository navigation
Argument Clinic: make error messages more helpful to developers #115077
Description
Activity
Suggested by @sobolevn in #114930 (comment).
What do you think: is it better to work on this one-by-one or in bulk?
There's also the possibility of always including the offending line, thus making these error messages look more like tracebacks:
Error in file 'test.c' on line 2: fn: ^ Illegal function name.What do you think: is it better to work on this one-by-one or in bulk?
One-by-one is too fine-grained, IMO; I think batches are fine. For example, all the converters in one PR and all the return converters in another.
I would try to do this in one piece. It may expose some patterns and lead to better structure of the code and tests.
But is some cases cannot be easily improved, I would left them out. And if some cases require significant rewriting, they can be split in a separate PR. You cannot say if this is needed before you try.
Reacted by Erlend E. Aasland+1, @serhiy-storchaka. BTW, feel free to contribute to my draft PR, if you wish.
Some1 of the Argument Clinic error messages only point to the problem in the clinic input, but do not suggest how to fix it. Suggesting to do an audit of clinic.py and improve the error messages that are too implicit or provide too little information/context to the developer.
Random selection of error messages that might be improved:
(There might be better examples; this was the result of a very quick and short audit.)
cc. @sobolevn
Linked PRs
Footnotes
but, definitely not all! ↩