Bug 417630 - [preferences] Show all compiler diagnostics in the UI
Summary: [preferences] Show all compiler diagnostics in the UI
Status: ASSIGNED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 4.3   Edit
Hardware: All All
: P3 enhancement with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: JDT-UI-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on: 83548
Blocks:
  Show dependency tree
 
Reported: 2013-09-19 14:45 EDT by Mickael Istria CLA
Modified: 2015-07-30 06:18 EDT (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mickael Istria CLA 2013-09-19 14:45:57 EDT
End-users now think Eclipse is not as good as other IDEs for on-the-fly error reporting. However, JDT provides many warnings which are disabled by default. They are like an hidden feature which would increase the value of the IDE if they were shown.
So I suggest that all JDT rules get shown as warning by default, and let people who don't like them de-activate them.
Comment 1 Dani Megert CLA 2013-09-20 03:54:41 EDT
> End-users now think Eclipse is not as good as other IDEs for on-the-fly error > reporting.

Let's say *some* end users, please ;-).


Blindly enable all warnings is a no go. Some really are optional and must stay disabled by default. Plus, we don't want to flood all existing Eclipse users with tons of new warnings. Just follow the JFace generics thread to see what happens if one does.

Having said that, we can do a pass over our options and enable those we think are indeed useful if set to warning or even error.

Another improvement would be to offer 'Set All To' buttons like we already have on the 'API Errors/Warnings' page. This is captured by bug 248306.
Comment 2 Mickael Istria CLA 2013-09-20 04:11:53 EDT
(In reply to Dani Megert from comment #1)
> Blindly enable all warnings is a no go. Some really are optional and must
> stay disabled by default. Plus, we don't want to flood all existing Eclipse
> users with tons of new warnings.
> Having said that, we can do a pass over our options and enable those we
> think are indeed useful if set to warning or even error.

And what about introducing the INFO level for thos that you think are useless? Cf bug 149334 .

As I already wrote in the JFace Generics disucssion, IMO any validation provides value.
And preferences can be imported/exported from an Eclipse instance to the other, so if one takes an Eclipse 4.4. with all validation reported as warnings and doesn't like it, it's easy to restore his previous configuration.
Other IDEs built on Eclipse can easily override the default settings via plugin_customization.ini to make warnings more relevant to the kind of activity they target.
Comment 3 Dani Megert CLA 2013-09-20 04:22:20 EDT
(In reply to Mickael Istria from comment #2)
> (In reply to Dani Megert from comment #1)
> > Blindly enable all warnings is a no go. Some really are optional and must
> > stay disabled by default. Plus, we don't want to flood all existing Eclipse
> > users with tons of new warnings.
> > Having said that, we can do a pass over our options and enable those we
> > think are indeed useful if set to warning or even error.
> 
> And what about introducing the INFO level for thos that you think are
> useless? Cf bug 149334 .

Might be worth a discussion in bug 83548.


> 
> As I already wrote in the JFace Generics disucssion, IMO any validation
> provides value.
> And preferences can be imported/exported from an Eclipse instance to the
> other, so if one takes an Eclipse 4.4. with all validation reported as
> warnings and doesn't like it, it's easy to restore his previous
> configuration.
> Other IDEs built on Eclipse can easily override the default settings via
> plugin_customization.ini to make warnings more relevant to the kind of
> activity they target.

We don't need to reiterate on this.