eclipse-jdt / eclipse-jdt/eclipse.jdt.core

rephrase compiler messages: move most significant info to the beginning

Open
#2,202 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Java
Stars
237
Forks
195
Avg merge
1d 12h
Merged PRs (30d)
47

Description

IMHO, compiler error and warning messages etc. should have the static part at the beginning to be easier to recognize (especially when the description column is small) and sort. For example:

| Description | Type | Path | Resource | Location |
| -- | -- | -- | -- | -- |
| The method canBeStatic(int) from the type Warnings can be declared as static | Java Problem| /Test (mvn)/src/main/java/test |Warnings.java | line 12|
|The method canBeStatic(int) from the type Warnings is never used locally |Java Problem |/Test (mvn)/src/main/java/test |Warnings.java |line 12|
|The method close() of type Warnings should be tagged with @​Override since it actually overrides a superinterface method |Java Problem |/Test (mvn)/src/main/java/test |Warnings.java |line 19|
|The method unused() from the type Warnings is never used locally |Java Problem |/Test (mvn)/src/main/java/test |Warnings.java |line 16|

should be rephrased to something like

| Description | Type | Path | Resource | Location |
| -- | -- | -- | -- | -- |
|method can be declared as static: canBeStatic(int) from the type Warnings |Java Problem |/Test (mvn)/src/main/java/test |Warnings.java |line 12|
|unused methed: canBeStatic(int) from the type Warnings |Java Problem |/Test (mvn)/src/main/java/test |Warnings.java |line 12|
|missing @​Override annotation: method close() of type Warnings overrides a superinterface method |Java Problem |/Test (mvn)/src/main/java/test |Warnings.java |line 19|
|unused method: unused() from the type Warnings |Java Problem |/Test (mvn)/src/main/java/test |Warnings.java |line 16|

This probably applies to a lot more messages, too.

Thanks!

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.