Skip to content

Commit fa57b5d

Browse files
committed
tmf: extend api for configuration 6
Review round #1
1 parent 8c2f3ef commit fa57b5d

1 file changed

Lines changed: 7 additions & 11 deletions

File tree

  • analyses/org.eclipse.tracecompass.incubator.inandout.core/src/org/eclipse/tracecompass/incubator/internal/inandout/core/analysis

analyses/org.eclipse.tracecompass.incubator.inandout.core/src/org/eclipse/tracecompass/incubator/internal/inandout/core/analysis/InAndOutDataProviderFactory.java

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,6 @@
4343
import org.eclipse.tracecompass.tmf.core.model.DataProviderDescriptor;
4444
import org.eclipse.tracecompass.tmf.core.model.tree.ITmfTreeDataModel;
4545
import org.eclipse.tracecompass.tmf.core.model.tree.ITmfTreeDataProvider;
46-
import org.eclipse.tracecompass.tmf.core.signal.TmfSignalManager;
4746
import org.eclipse.tracecompass.tmf.core.trace.ITmfTrace;
4847
import org.eclipse.tracecompass.tmf.core.trace.TmfTraceManager;
4948
import org.eclipse.tracecompass.tmf.core.trace.experiment.TmfExperiment;
@@ -96,13 +95,6 @@ public class InAndOutDataProviderFactory extends AbstractTmfDataProviderConfigur
9695
.setCapabilities(new DataProviderCapabilities.Builder().setCanCreate(true).build())
9796
.build();
9897

99-
/**
100-
* Default constructor
101-
*/
102-
public InAndOutDataProviderFactory() {
103-
TmfSignalManager.register(this);
104-
}
105-
10698
@Override
10799
public @Nullable ITmfTreeDataProvider<? extends ITmfTreeDataModel> createProvider(ITmfTrace trace) {
108100
return null;
@@ -182,7 +174,7 @@ protected void removeConfiguration(ITmfTrace trace, ITmfConfiguration config) {
182174
* the
183175
* @throws TmfConfigurationException if an error occurs
184176
*/
185-
private void remove(ITmfConfiguration config, @NonNull ITmfTrace trace, String baseAnalysisId) throws TmfConfigurationException {
177+
private void remove(ITmfConfiguration config, ITmfTrace trace, String baseAnalysisId) throws TmfConfigurationException {
186178
IPath traceConfig = getConfigurationRootFolder(trace);
187179
traceConfig = traceConfig.append(File.separator).append(config.getId()).addFileExtension(JSON_EXTENSION);
188180
File configFile = traceConfig.toFile();
@@ -211,11 +203,10 @@ private void remove(ITmfConfiguration config, @NonNull ITmfTrace trace, String b
211203
* the trace to apply it to
212204
* @param writeConfig
213205
* write the config (do only once)
214-
* @return InAndOutAnalysisModule
215206
* @throws TmfConfigurationException
216207
* if an error occurs
217208
*/
218-
private void create(@NonNull ITmfConfiguration config, @NonNull ITmfTrace trace, boolean writeConfig, IAnalysisModule module) throws TmfConfigurationException {
209+
private void create(ITmfConfiguration config, ITmfTrace trace, boolean writeConfig, IAnalysisModule module) throws TmfConfigurationException {
219210
/*
220211
* Apply configuration to each trace (no need to check trace type here)
221212
*/
@@ -249,4 +240,9 @@ protected IPath getConfigurationRootFolder(ITmfTrace trace) {
249240
return supplPath;
250241
}
251242

243+
@Override
244+
public void dispose() {
245+
super.dispose();
246+
}
247+
252248
}

0 commit comments

Comments
 (0)