Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTorbjörn Svensson2018-10-06 17:09:56 -0400
committerWilliam Riley2018-10-09 05:47:15 -0400
commitaef0eaee8c5cf28b441ec0cfe535acd30418e98d (patch)
tree38b88a101c188ae18a275ae190057d1d4c48db5e /build/org.eclipse.cdt.managedbuilder.ui
parentae78fde323f048080c240d0494eb723537eba7da (diff)
downloadorg.eclipse.cdt-aef0eaee8c5cf28b441ec0cfe535acd30418e98d.tar.gz
org.eclipse.cdt-aef0eaee8c5cf28b441ec0cfe535acd30418e98d.tar.xz
org.eclipse.cdt-aef0eaee8c5cf28b441ec0cfe535acd30418e98d.zip
Bug 539882: Applicability calculator should be called with option category
In some situations, it's convenient to create option categories on the abstract tool and use an option applicability calculator to hide the option category if no option is added to the category in an extending instance of the tool. To make the decision, the calculator needs to know what category is to be checked. Change-Id: Id4a269ee583f1f3dee88dee797ba89a0e92d1a42 Signed-off-by: Torbjörn Svensson <torbjorn.svensson@st.com>
Diffstat (limited to 'build/org.eclipse.cdt.managedbuilder.ui')
-rw-r--r--build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/ToolListContentProvider.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/ToolListContentProvider.java b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/ToolListContentProvider.java
index f8a3962deb..947caeaf6d 100644
--- a/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/ToolListContentProvider.java
+++ b/build/org.eclipse.cdt.managedbuilder.ui/src/org/eclipse/cdt/managedbuilder/ui/properties/ToolListContentProvider.java
@@ -120,7 +120,7 @@ public class ToolListContentProvider implements ITreeContentProvider{
for (int i=0; i<cats.length; i++) {
ToolListElement e = new ToolListElement(parentElement, optHolder, cats[i]);
applicabilityCalculator = e.getOptionCategory().getApplicabilityCalculator();
- if (applicabilityCalculator == null || applicabilityCalculator.isOptionCategoryVisible(config, optHolder, parent)) {
+ if (applicabilityCalculator == null || applicabilityCalculator.isOptionCategoryVisible(config, e.getHoldOptions(), e.getOptionCategory())) {
parentElement.addChildElement(e);
createChildElements(e,config);
}

Back to the top