ome / omero-insight

Gradle project containing insight java client for OMERO
https://www.openmicroscopy.org/omero/
GNU General Public License v2.0
7 stars 14 forks source link

break statement may be needed #412

Closed scuniff closed 11 months ago

scuniff commented 12 months ago

Regarding:

https://github.com/ome/omero-insight/blob/40566b97d28f9caf5720a55faf15c324a8c6e5de/src/main/java/org/openmicroscopy/shoola/agents/dataBrowser/view/DataBrowserUI.java#L572

        case COLUMNS_VIEW:
            comp = model.getTableView();
        case SEARCH_VIEW:
                        comp = model.getSearchView();

Is break statement needed?

        case COLUMNS_VIEW:
            comp = model.getTableView();
            break;
        case SEARCH_VIEW:
                        comp = model.getSearchView();
jburel commented 11 months ago

Fixed https://github.com/ome/omero-insight/pull/413