Difference between revisions of "JDT Core/Plan/4.8/JEP325"
(→Content Assist support) |
(→UI Changes) |
||
| (38 intermediate revisions by 2 users not shown) | |||
| Line 1: | Line 1: | ||
| − | = Planning page for JDT Implementation of Switch Expressions (JEP 325) for Java | + | = Planning page for JDT Implementation of Switch Expressions (JEP 325) for Java 12 = |
| + | <br /> | ||
| + | [https://bugs.eclipse.org/bugs/showdependencytree.cgi?id=536055&hide_resolved=1 | Bug 536055] | ||
| − | Main tracking bug: {{bug| | + | Main tracking bug: {{bug|542557}} |
== Compiler changes == | == Compiler changes == | ||
| − | Changes regarding the compiler: | + | Changes regarding the compiler: {{bug|542557}} |
| + | |||
=== Grammar changes === | === Grammar changes === | ||
Switch Expression grammar changes - add bug number | Switch Expression grammar changes - add bug number | ||
This would be a major grammar change effort/time estimate for grammar - | This would be a major grammar change effort/time estimate for grammar - | ||
Currently here - need to rewrite the grammar - est : done | Currently here - need to rewrite the grammar - est : done | ||
| + | Additional minor but required grammar change was done to include parenthesis. | ||
| − | == Compiler AST Creation == | + | === Compiler AST Creation === |
| − | Tied with parsing - | + | Tied with parsing - done |
| − | + | ||
| − | + | ||
| − | === Type | + | === Type Inference === |
| − | Type inference | + | Type inference since switch expression may be an expression - [https://bugs.eclipse.org/bugs/show_bug.cgi?id=542871 bug 542871] - Done |
| + | |||
| + | === Flow analysis changes === | ||
| + | will have flow analysis changes - [https://bugs.eclipse.org/bugs/show_bug.cgi?id=542707 bug 542707] - done | ||
=== Error reports === | === Error reports === | ||
| − | * new error reports. ( | + | * new error reports. (0 days - in parallel with flow analysis) - done |
=== Code generation === | === Code generation === | ||
| − | Addressing the breaks without label but with expressions and implicit break labels for switch label expressions - | + | Addressing the breaks without label but with expressions and implicit break labels for switch label expressions -0d [https://bugs.eclipse.org/bugs/show_bug.cgi?id=542838 bug 542838] - done |
| + | Further issues found [https://bugs.eclipse.org/bugs/show_bug.cgi?id=544458 bug 544458] - done - 0d | ||
== Java Search support == | == Java Search support == | ||
| − | slight change expected - | + | slight change expected - 0d {{bug|542559}} - done |
| + | Additional issue found in {{bug|544304}} - done | ||
== DOM support == | == DOM support == | ||
| − | ast changes expected - | + | ast changes expected DOM AST change -0d - done |
| + | {{bug|542558}}<br> | ||
| + | ast rewrite - 0d - done | ||
| + | {{bug|543720}}<br> | ||
| + | |||
| + | == Formatter support == | ||
| + | ast changes expected - done | ||
| + | {{bug|543818}}<br> | ||
== Content Assist support == | == Content Assist support == | ||
| − | * additional place the content assist be applicable - | + | * additional place the content assist be applicable - 2 days ({{bug|542560}} - in progress |
| + | |||
| + | == Code Select support == | ||
| + | * additional place the content assist be applicable - ({{bug|542561}} - done - 0d | ||
== UI Changes == | == UI Changes == | ||
| − | * Switch ASTs to | + | * Switch ASTs to JLS12 (generic - 1d) |
=== Code hover === | === Code hover === | ||
| Line 45: | Line 62: | ||
=== Quick fixes === | === Quick fixes === | ||
* convert to a switch statement ( 3 DAYS? jdt.ui to comment) | * convert to a switch statement ( 3 DAYS? jdt.ui to comment) | ||
| − | * coalesce two or more case statements to comma separated | + | * coalesce two or more case statements to comma separated case label statements - 4 ays> jdt.ui ? opportunistic |
| + | * see [https://bugs.eclipse.org/bugs/show_bug.cgi?id=543665 bug 543665] | ||
=== Code templates === | === Code templates === | ||
Latest revision as of 00:15, 18 March 2019
Contents
Planning page for JDT Implementation of Switch Expressions (JEP 325) for Java 12
Main tracking bug: bug 542557
Compiler changes
Changes regarding the compiler: bug 542557
Grammar changes
Switch Expression grammar changes - add bug number This would be a major grammar change effort/time estimate for grammar - Currently here - need to rewrite the grammar - est : done Additional minor but required grammar change was done to include parenthesis.
Compiler AST Creation
Tied with parsing - done
Type Inference
Type inference since switch expression may be an expression - bug 542871 - Done
Flow analysis changes
will have flow analysis changes - bug 542707 - done
Error reports
- new error reports. (0 days - in parallel with flow analysis) - done
Code generation
Addressing the breaks without label but with expressions and implicit break labels for switch label expressions -0d bug 542838 - done Further issues found bug 544458 - done - 0d
Java Search support
slight change expected - 0d bug 542559 - done Additional issue found in bug 544304 - done
DOM support
ast changes expected DOM AST change -0d - done
bug 542558
ast rewrite - 0d - done
bug 543720
Formatter support
ast changes expected - done
bug 543818
Content Assist support
- additional place the content assist be applicable - 2 days (bug 542560 - in progress
Code Select support
- additional place the content assist be applicable - (bug 542561 - done - 0d
UI Changes
- Switch ASTs to JLS12 (generic - 1d)
Code hover
- Show inferred type of variable - no change expected
Preferences
- Allow JDT UI to target Java 11 in Compiler Preferences - generic (1d in parallel)
Quick fixes
- convert to a switch statement ( 3 DAYS? jdt.ui to comment)
- coalesce two or more case statements to comma separated case label statements - 4 ays> jdt.ui ? opportunistic
- see bug 543665
Code templates
- add switch in templates - 1d
JDT Debug
- Expected to have an impact - especially breakpoints - 1w approx