Bugzilla – Full Text Bug Listing
| Summary: | [compiler] Improve wording for the warning for masked/hidden catch block | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] JDT | Reporter: | Ayushman Jain <amj87.iitr> | ||||
| Component: | Core | Assignee: | Ayushman Jain <amj87.iitr> | ||||
| Status: | VERIFIED FIXED | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | deepakazad, satyam.kandula, srikanth_sankaran | ||||
| Version: | 3.7 | Flags: | srikanth_sankaran:
review+
|
||||
| Target Milestone: | 3.8 M3 | ||||||
| Hardware: | PC | ||||||
| OS: | Windows XP | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
Sorry this is not a precise rethrow problem. My bad. Changing bug description. Targetting 3.8. This will be fixed on HEAD. Created attachment 204265 [details]
proposed fix
Fixed message and updated tests
Fixed in HEAD with commit 283ea669e65ae7c25e43949f506630d59e6cdb74 Oops, forgot to get this reviewed. Srikanth, let me know if the new diagnostic message "Unreachable catch block for IOException. Only more specific exceptions are thrown and they are handled by previous catch block(s)." is ok. Agree with the change. |
BETA_JAVA7 void foo() { try { throw new FileNotFoundException(); } catch (FileNotFoundException ex) { } catch (IOException ex) { // see warning here } } In the above case, the warning says "Unreachable catch block for IOException. Only more specific exceptions are thrown and handled by previous catch block(s)." This kind of implies that the exceptions are being thrown AND handled by prev. catch blocks. But should actually be "Unreachable catch block for IOException. Only more specific exceptions are thrown and they are handled by previous catch block(s).