)]}' { "commit": "0e7d9bae45409b95b027ba9a791e834c79024db6", "tree": "9d8d28b14b0bc07999e293d1377ea415a745c310", "parents": [ "35514cc6134ff327a7674a49de034c615336887a" ], "author": { "name": "Jeff Johnston", "email": "jjohnstn@redhat.com", "time": "Wed Jul 14 17:05:48 2021 -0400" }, "committer": { "name": "Jeff Johnston", "email": "jjohnstn@redhat.com", "time": "Wed Jul 14 20:44:49 2021 -0400" }, "message": "Bug 574824 - [cleanup] Switch expression cleanup generates invalid code\n\n- fix SwitchExpressionsFixCore.SwitchStatementsFinder to handle\n a switch case that falls off the end of the statement (i.e. no break)\n- fix SwitchExpressionsFixCore.SwitchExpressionsFixOperation.rewriteAST()\n method to properly copy over type and final modifier of existing\n statement\n- add new test to CleanUpTest14\n\nChange-Id: I4238530e7782eb6bb3278d0cce82ecbdbcc1d9b5\nReviewed-on: https://git.eclipse.org/r/c/jdt/eclipse.jdt.ui/+/183069\nTested-by: JDT Bot \u003cjdt-bot@eclipse.org\u003e\nTested-by: Jeff Johnston \u003cjjohnstn@redhat.com\u003e\nReviewed-by: Jeff Johnston \u003cjjohnstn@redhat.com\u003e\n", "tree_diff": [ { "type": "modify", "old_id": "b3848236eb47d439a738dbe6590bc78bb6d0eb02", "old_mode": 33188, "old_path": "org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/fix/SwitchExpressionsFixCore.java", "new_id": "b89c7c73250b9a9f05a499967cab3f10d427353e", "new_mode": 33188, "new_path": "org.eclipse.jdt.core.manipulation/core extension/org/eclipse/jdt/internal/corext/fix/SwitchExpressionsFixCore.java" }, { "type": "modify", "old_id": "06213f754783abe59a234b0b523ad6d004e859f0", "old_mode": 33188, "old_path": "org.eclipse.jdt.ui.tests/ui/org/eclipse/jdt/ui/tests/quickfix/CleanUpTest14.java", "new_id": "910ade920854b7042ba0476484756f1ad2daf6cc", "new_mode": 33188, "new_path": "org.eclipse.jdt.ui.tests/ui/org/eclipse/jdt/ui/tests/quickfix/CleanUpTest14.java" } ] }