Skip to content

Commit 8bc9e27

Browse files
eclipse-equinox-botlaeubi
authored andcommitted
Perform clean code of bundles/org.eclipse.equinox.p2.publisher
1 parent bdb0fda commit 8bc9e27

File tree

3 files changed

+4
-8
lines changed

3 files changed

+4
-8
lines changed

bundles/org.eclipse.equinox.p2.publisher/src/org/eclipse/equinox/p2/publisher/AbstractPublisherAction.java

+2-4
Original file line numberDiff line numberDiff line change
@@ -213,8 +213,7 @@ protected String createCUIdString(String id, String type, String flavor, String
213213
protected Collection<IRequirement> createIURequirements(Collection<? extends IVersionedId> children) {
214214
ArrayList<IRequirement> result = new ArrayList<>(children.size());
215215
for (IVersionedId next : children) {
216-
if (next instanceof IInstallableUnit) {
217-
IInstallableUnit iu = (IInstallableUnit) next;
216+
if (next instanceof IInstallableUnit iu) {
218217
VersionRange range = new VersionRange(iu.getVersion(), true, iu.getVersion(), true);
219218
result.add(MetadataFactory.createRequirement(IInstallableUnit.NAMESPACE_IU_ID, iu.getId(), range,
220219
iu.getFilter() == null ? null : iu.getFilter(), false, false));
@@ -471,11 +470,10 @@ protected static void processCapabilityAdvice(InstallableUnitDescription iu, IPu
471470
}
472471

473472
protected static IRequiredCapability toRequiredCapability(IRequirement requirement) {
474-
if (!(requirement instanceof IRequiredCapability)) {
473+
if (!(requirement instanceof IRequiredCapability requiredCapability)) {
475474
return null;
476475
}
477476

478-
IRequiredCapability requiredCapability = (IRequiredCapability) requirement;
479477
if (!RequiredCapability.isVersionRangeRequirement(requiredCapability.getMatches())) {
480478
return null;
481479
}

bundles/org.eclipse.equinox.p2.publisher/src/org/eclipse/equinox/p2/publisher/actions/RootIUAction.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -141,8 +141,7 @@ private Collection<? extends IVersionedId> getChildren(IPublisherResult result)
141141
for (Object object : list) {
142142
// if the advice is a string, look it up in the result. if not there then
143143
// query the known metadata repos
144-
if (object instanceof String) {
145-
String childId = (String) object;
144+
if (object instanceof String childId) {
146145
IInstallableUnit iu = queryForIU(result, childId, getVersionAdvice(childId));
147146
if (iu != null) {
148147
children.add(iu);

bundles/org.eclipse.equinox.p2.publisher/src/org/eclipse/equinox/p2/publisher/actions/VersionAdvice.java

+1-2
Original file line numberDiff line numberDiff line change
@@ -109,10 +109,9 @@ public void setVersion(String namespace, String id, Version version) {
109109
}
110110

111111
public IPublisherAdvice merge(IPublisherAdvice advice) {
112-
if (!(advice instanceof VersionAdvice)) {
112+
if (!(advice instanceof VersionAdvice source)) {
113113
return this;
114114
}
115-
VersionAdvice source = (VersionAdvice) advice;
116115
for (String namespace : source.versions.keySet()) {
117116
Map<String, Version> myValues = versions.get(namespace);
118117
Map<String, Version> sourceValues = source.versions.get(namespace);

0 commit comments

Comments
 (0)