diff options
author | Björn Gustavsson <[email protected]> | 2013-09-05 13:57:09 +0200 |
---|---|---|
committer | Björn Gustavsson <[email protected]> | 2013-09-05 14:28:08 +0200 |
commit | 60984adef62f5e5bf8c65f403d4f8522afb6d40a (patch) | |
tree | b44fe1497cce75afc9b675b32f1478bedc233906 /lib/webtool | |
parent | 11c31f5ba8489b0e179caea46e514b990fbadb95 (diff) | |
download | otp-60984adef62f5e5bf8c65f403d4f8522afb6d40a.tar.gz otp-60984adef62f5e5bf8c65f403d4f8522afb6d40a.tar.bz2 otp-60984adef62f5e5bf8c65f403d4f8522afb6d40a.zip |
compiler: Conform returned errors to the documented format
ErrorInfo is documented to be:
{ErrorLine,Module,ErrorDescriptor}
but for some errors with line numbers it would look like:
{Module,ErrorDescriptor}
Ensure that all ErrorInfo tuples have three elements. Use 'none'
instead of a line number:
{none,Module,ErrorDescriptor}
There already are errors that return 'none' when no line number is
available, but that convention was not documented. Mention it in the
documentation.
Also make sure that the compiler will not print 'none' as a line
number in error messages (if the 'report_errors' option is given) as
that looks stupid. That is, when attempting to compile a non-existing
module, the error message should be:
non-existing.erl: no such file or directory
and not:
non-existing.erl:none: no such file or directory
Diffstat (limited to 'lib/webtool')
0 files changed, 0 insertions, 0 deletions