Skip to content

Commit 83a2fc6

Browse files
committed
Remove unnecessary casts
The rest of the change is save actions modernizing code.
1 parent 46dc8e0 commit 83a2fc6

File tree

5 files changed

+17
-18
lines changed

5 files changed

+17
-18
lines changed

Diff for: libhover/org.eclipse.linuxtools.cdt.libhover/src/org/eclipse/linuxtools/internal/cdt/libhover/LibHover.java

+5-5
Original file line numberDiff line numberDiff line change
@@ -390,12 +390,12 @@ public IFunctionSummary getFunctionInfo(ICHelpInvocationContext context, ICHelpB
390390
if (result[0] != null) {
391391
final IBinding binding = result[0].getBinding();
392392
// Check to see we have a member function.
393-
if (binding instanceof ICPPFunction) {
394-
methodType = ((ICPPFunction)binding).getType();
393+
if (binding instanceof ICPPFunction cppfunction) {
394+
methodType = cppfunction.getType();
395395
// We have a member function, find the class name.
396-
IBinding owner = ((ICPPFunction)binding).getOwner();
397-
if (owner instanceof ICPPClassType) {
398-
className = getClassName((ICPPClassType)owner);
396+
IBinding owner = cppfunction.getOwner();
397+
if (owner instanceof ICPPClassType classType) {
398+
className = getClassName(classType);
399399
}
400400
}
401401
}

Diff for: perf/org.eclipse.linuxtools.perf/src/org/eclipse/linuxtools/internal/perf/ui/PerfDoubleClickAction.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public void run() {
4646
try {
4747
if (obj instanceof PMLineRef line) {
4848
// Open in editor
49-
PMFile file = (PMFile) ((PMSymbol) line.getParent()).getParent();
49+
PMFile file = (PMFile) line.getParent().getParent();
5050
ProfileUIUtils.openEditorAndSelect(file.getPath(),Integer.parseInt(line.getName()), PerfPlugin.getDefault().getProfiledProject());
5151
} else if (obj instanceof PMFile file) {
5252
ProfileUIUtils.openEditorAndSelect(file.getName(), 1);

Diff for: rpm/org.eclipse.linuxtools.rpm.ui.editor.tests/src/org/eclipse/linuxtools/rpm/ui/editor/tests/HeaderRecognitionTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ public void testGetSimpleSectionName() {
3434
newFile(testText);
3535
element = parser.parseLine(testText, specfile, 0);
3636
assertEquals(SpecfileSection.class, element.getClass());
37-
assertEquals(testText.substring(1), ((SpecfileSection) element).getName());
37+
assertEquals(testText.substring(1), element.getName());
3838
}
3939

4040
@Test

Diff for: rpm/org.eclipse.linuxtools.rpm.ui.editor.tests/src/org/eclipse/linuxtools/rpm/ui/editor/tests/parser/SpecfileDefineTest.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
import org.eclipse.linuxtools.rpm.core.utils.RPMQuery;
2222
import org.eclipse.linuxtools.rpm.ui.editor.markers.SpecfileErrorHandler;
2323
import org.eclipse.linuxtools.rpm.ui.editor.parser.SpecfileDefine;
24-
import org.eclipse.linuxtools.rpm.ui.editor.parser.SpecfileElement;
2524
import org.eclipse.linuxtools.rpm.ui.editor.tests.FileTestCase;
2625
import org.eclipse.linuxtools.rpm.ui.editor.tests.SpecfileTestFailure;
2726
import org.junit.Before;
@@ -64,7 +63,7 @@ public void testResolve() {
6463
SpecfileDefine define1 = new SpecfileDefine("name", "testspec", specfile,
6564
specfile.getPackages().getPackage(specfile.getName()));
6665
specfile.addDefine(define1);
67-
assertEquals("testspec", ((SpecfileElement) define1).resolve("%{name}"));
66+
assertEquals("testspec", define1.resolve("%{name}"));
6867
}
6968

7069
@Test

Diff for: systemtap/org.eclipse.linuxtools.systemtap.graphing.ui/src/org/eclipse/linuxtools/systemtap/graphing/ui/datadisplay/DataGrid.java

+9-9
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
/*******************************************************************************
22
* Copyright (c) 2006, 2018 IBM Corporation and others.
3-
*
3+
*
44
* This program and the accompanying materials are made
55
* available under the terms of the Eclipse Public License 2.0
66
* which is available at https://www.eclipse.org/legal/epl-2.0/
@@ -139,10 +139,10 @@ private Menu initMenus() {
139139

140140
IDataSetFilter[] filters = filteredDataSet.getFilters();
141141
if (filters != null && filters.length > 0) {
142-
for (int i = 0; i < filters.length; i++) {
142+
for (IDataSetFilter filter : filters) {
143143
item = new MenuItem(filterMenu, SWT.CASCADE);
144-
item.setText(AvailableFilterTypes.getFilterName(filters[i].getID()));
145-
item.setData(filters[i]);
144+
item.setText(AvailableFilterTypes.getFilterName(filter.getID()));
145+
item.setData(filter);
146146
item.addSelectionListener(new RemoveFilterSelection());
147147
}
148148
} else {
@@ -156,9 +156,9 @@ private Menu initMenus() {
156156
jumpToEntryMenuItem.setText(Localization.getString("DataGrid.JumpToEntry")); //$NON-NLS-1$
157157
jumpToEntryMenuItem.setSelection(prefs.getBoolean(GraphingPreferenceConstants.P_JUMP_NEW_TABLE_ENTRY));
158158

159-
for (int i = 0; i < IFormattingStyles.FORMAT_TITLES.length; i++) {
159+
for (String element : IFormattingStyles.FORMAT_TITLES) {
160160
item = new MenuItem(formatMenu, SWT.RADIO);
161-
item.setText(IFormattingStyles.FORMAT_TITLES[i]);
161+
item.setText(element);
162162
item.addSelectionListener(new MenuFormatSelection());
163163
}
164164

@@ -209,7 +209,7 @@ public void widgetSelected(SelectionEvent e) {
209209
private class RemoveFilterSelection implements SelectionListener {
210210
@Override
211211
public void widgetSelected(SelectionEvent e) {
212-
IDataSetFilter idsf = (IDataSetFilter)((MenuItem)e.widget).getData();
212+
IDataSetFilter idsf = (IDataSetFilter) e.widget.getData();
213213
e.widget.dispose();
214214
if (filterMenu.getItemCount() == 0) {
215215
removeFiltersMenuItem.setEnabled(false);
@@ -234,8 +234,8 @@ public void menuShown(MenuEvent e) {
234234

235235
int selectedCol = Math.max(1, getSelectedColumn());
236236

237-
for (int i = 0; i < items.length; i++) {
238-
items[i].setSelection(false);
237+
for (MenuItem item : items) {
238+
item.setSelection(false);
239239
}
240240
items[columnFormat[selectedCol-1].getFormat()].setSelection(true);
241241

0 commit comments

Comments
 (0)