Confusing error count

[FroMage]

compile.tests.js.quick:
[ceylon-compile-js] Error: Errors found. Compilation stopped.
[ceylon-compile-js] Note: Created module check/0.1
[ceylon-compile-js] Note: Created module serialization/0.1
[ceylon-compile-js] Note: Created module default
[ceylon-compile-js] Note: Created module modules.imported/1
[ceylon-compile-js] Note: Created module modules.optional/1
[ceylon-compile-js] Note: Created module annotations/0.1
[ceylon-compile-js] Note: Created module metamodel/0.1
[ceylon-compile-js] Note: Created module modules.required/1
[ceylon-compile-js] Note: Created module resmod/0.1
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 624:0-624:4 of tests.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 747:0-747:4 of tests.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazyList'] at 70:17-70:24 of LazyList.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazySet'] at 43:24-43:30 of LazySet.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 315:0-315:4 of tests.ceylon
[ceylon-compile-js] warning encountered [type is deprecated: 'LazySet'] at 43:24-43:30 of LazySet.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 153:0-153:4 of tests.ceylon
[ceylon-compile-js] ceylon compile-js: 31 errors.
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazySet'] at 55:24-55:30 of LazySet.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 252:0-252:4 of tests.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazyList'] at 28:12-28:19 of LazyList.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 12:0-12:4 of compilerBug2116.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 908:0-908:4 of tests.ceylon
[ceylon-compile-js] warning encountered [type is deprecated: 'LazySet'] at 55:24-55:30 of LazySet.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 225:0-225:4 of tests.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 896:0-896:4 of tests.ceylon
[ceylon-compile-js] warning encountered [type is deprecated: 'LazySet'] at 69:24-69:30 of LazySet.ceylon
[ceylon-compile-js] warning encountered [type is deprecated: 'LazySet'] at 32:24-32:30 of LazySet.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 91:0-91:4 of tests.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 18:0-18:4 of compilerBug1699.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazySet'] at 32:24-32:30 of LazySet.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 12:0-12:4 of bug378.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazyList'] at 56:19-56:26 of LazyList.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 3:0-3:4 of bug409.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 2:0-2:4 of arrays.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 187:0-187:4 of tests.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazyList'] at 82:19-82:26 of LazyList.ceylon
[ceylon-compile-js] error encountered [not a legal statement (not an invocation, assignment, or increment/decrement)] at 244:4-244:38 of arrays.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 417:0-417:4 of tests.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazyList'] at 73:17-73:24 of LazyList.ceylon
[ceylon-compile-js] warning encountered [compiler annotations are an unsupported language feature] at 600:0-600:4 of tests.ceylon
[ceylon-compile-js] warning encountered [declaration is deprecated: 'LazySet'] at 69:24-69:30 of LazySet.ceylon

• The error count is 1, not 31: it should not count warnings
• The error count is lost in the middle, that makes no sense.
• The error message is lost in the middle of warnings. I suspect it'd be a lot more useful to pull errors at the top, no?

[quintesse]

For some other stuff I was working on I fixed some of this almost as if by accident. The error count will now only count errors like it should. We also ignore AnalysisMessages if we have ParseErrors, just like the JVM backend and the IDE do, so the above list might be less "crowded". I don't know about pulling up the errors, we don't do that in the JVM backend either. Personally I'd expect errors and warnings that are close in the file to be also close in the output (and you can always simply filter out the warnings). I don't know what to do about the "ceylon compile-js: 31 errors." being in the middle though, is there some threading going on?

[chochos]

Could the 31 errors in the middle could be due to logging issues? Like, using some logging backend instead of System.out.println?

[quintesse]

No idea, I put a flush() in there somewhere to see if it helped and I have not been able to reproduce it at least so it might be fixed already.

[quintesse]

I can't reproduce this anymore, so let's just consider this closed. Reopen if you still see it @FroMage.