Skip to content

Commit da880f2

Browse files
committed
Do not load sequence number from target files anymore
As the sequence number is not persisted into a target file by the target editor, there is actually no point in loading it as it only can lead to inconsistent state.
1 parent 3dfbf59 commit da880f2

File tree

4 files changed

+0
-26
lines changed

4 files changed

+0
-26
lines changed

ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/TargetDefinition.java

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1126,15 +1126,6 @@ public int incrementSequenceNumber() {
11261126
return ++fSequenceNumber;
11271127
}
11281128

1129-
/**
1130-
* Convenience method to set the sequence number to a specific
1131-
* value. Used when loading a target from a persisted file.
1132-
* @param value value to set the sequence number to
1133-
*/
1134-
void setSequenceNumber(int value) {
1135-
fSequenceNumber = value;
1136-
}
1137-
11381129
private void removeElement(String... childNames) {
11391130
if (fRoot != null) {
11401131
TargetDefinitionDocumentTools.removeElement(fRoot, childNames);

ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/TargetDefinitionPersistenceHelper.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ public class TargetDefinitionPersistenceHelper {
9090
static final String ATTR_FOLLOW_REPOSITORY_REFERENCES = "followRepositoryReferences"; //$NON-NLS-1$
9191
static final String ATTR_VERSION = "version"; //$NON-NLS-1$
9292
static final String ATTR_CONFIGURATION = "configuration"; //$NON-NLS-1$
93-
static final String ATTR_SEQUENCE_NUMBER = "sequenceNumber"; //$NON-NLS-1$
9493
static final String CONTENT = "content"; //$NON-NLS-1$
9594
static final String ATTR_USE_ALL = "useAllPlugins"; //$NON-NLS-1$
9695
static final String PLUGINS = "plugins"; //$NON-NLS-1$

ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/TargetPersistence36Helper.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -165,14 +165,6 @@ public static void initFromDoc(ITargetDefinition definition, Element root) throw
165165
}
166166
}
167167
}
168-
169-
// Set the sequence number at the very end
170-
String sequenceNumber = root.getAttribute(TargetDefinitionPersistenceHelper.ATTR_SEQUENCE_NUMBER);
171-
try {
172-
((TargetDefinition) definition).setSequenceNumber(Integer.parseInt(sequenceNumber));
173-
} catch (NumberFormatException e) {
174-
((TargetDefinition) definition).setSequenceNumber(0);
175-
}
176168
}
177169

178170
/**

ui/org.eclipse.pde.core/src/org/eclipse/pde/internal/core/target/TargetPersistence38Helper.java

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -193,14 +193,6 @@ public static void initFromDoc(ITargetDefinition definition, Element root) {
193193
}
194194
}
195195
}
196-
197-
// Set the sequence number at the very end
198-
String sequenceNumber = root.getAttribute(TargetDefinitionPersistenceHelper.ATTR_SEQUENCE_NUMBER);
199-
try {
200-
((TargetDefinition) definition).setSequenceNumber(Integer.parseInt(sequenceNumber));
201-
} catch (NumberFormatException e) {
202-
((TargetDefinition) definition).setSequenceNumber(0);
203-
}
204196
}
205197

206198
/**

0 commit comments

Comments
 (0)