| Summary: | Delete SubMonitor.step | ||
|---|---|---|---|
| Product: | [Eclipse Project] Equinox | Reporter: | Stefan Xenos <sxenos> |
| Component: | Components | Assignee: | Stefan Xenos <sxenos> |
| Status: | RESOLVED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | daniel_megert, eclipse.sprigogin, jarthana, Lars.Vogel |
| Version: | unspecified | ||
| Target Milestone: | Oxygen M4 | ||
| Hardware: | PC | ||
| OS: | Linux | ||
| See Also: |
https://git.eclipse.org/r/84267 https://git.eclipse.org/c/equinox/rt.equinox.bundles.git/commit/?id=35dc5d75234efce06240f8d10e8c1f1b4b26b35b https://git.eclipse.org/r/84429 https://git.eclipse.org/r/84431 https://git.eclipse.org/r/84432 https://git.eclipse.org/c/platform/eclipse.platform.resources.git/commit/?id=78330b022d1b150ae0f8e598c6876acf83f609fe https://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=1353b654d0a7eed0f50b859d59d390ade519ee39 https://git.eclipse.org/c/equinox/rt.equinox.bundles.git/commit/?id=f3511f0a38be465923c1d735072f234597af1bff |
||
| Whiteboard: | |||
| Bug Depends on: | 505936 | ||
| Bug Blocks: | |||
|
Description
Stefan Xenos
Step was introduced in bug 500483. Fine for me, if Sergey also gives his +1. As step() is already used in platform UI, PDE and (IIRC) resources, we first need to convert these before deleting the new API. (In reply to Lars Vogel out-of-office until 21.10.2016 from comment #2) +1 for deletion. New Gerrit change created: https://git.eclipse.org/r/84267 New Gerrit change created: https://git.eclipse.org/r/84276 Gerrit change https://git.eclipse.org/r/84267 was merged to [master]. Commit: http://git.eclipse.org/c/equinox/rt.equinox.bundles.git/commit/?id=35dc5d75234efce06240f8d10e8c1f1b4b26b35b Please note that using @noreference (or any other API Tools tags) for things that are introduced in the same stream, don't have any effect. API Tools compares against the API baseline only, should be set to R4.6.1 in our current master stream. I suggest you either open bugs against all affected projects along with a Gerrit change, or attach the Gerrit changes to this bug. Stefan, could you please confirm that a simple replacement of ".step(" with ".split(" is all that is needed? Please refer to bug 506971 for the relevant change in jdt.core.
> could you please confirm that a simple replacement of ".step("
> with ".split(" is all that is needed?
Yes. That's all that's needed.
> or attach the Gerrit changes to this bug
Will do.
New Gerrit change created: https://git.eclipse.org/r/84429 New Gerrit change created: https://git.eclipse.org/r/84431 New Gerrit change created: https://git.eclipse.org/r/84432 Gerrit change https://git.eclipse.org/r/84431 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.resources.git/commit/?id=78330b022d1b150ae0f8e598c6876acf83f609fe Gerrit change https://git.eclipse.org/r/84432 was merged to [master]. Commit: http://git.eclipse.org/c/platform/eclipse.platform.ui.git/commit/?id=1353b654d0a7eed0f50b859d59d390ade519ee39 Gerrit change https://git.eclipse.org/r/84429 was merged to [master]. Commit: http://git.eclipse.org/c/equinox/rt.equinox.bundles.git/commit/?id=f3511f0a38be465923c1d735072f234597af1bff Gerrit change https://git.eclipse.org/r/84429 was merged to [master]. Commit: http://git.eclipse.org/c/equinox/rt.equinox.bundles.git/commit/?id=f3511f0a38be465923c1d735072f234597af1bff |