Skip to content

Commit 0a13daa

Browse files
EcljpseB0Tmickaelistria
authored andcommitted
e4 x-friends to reduce "Discouraged access" warnings
1 parent c41b487 commit 0a13daa

File tree

8 files changed

+23
-16
lines changed

8 files changed

+23
-16
lines changed

bundles/org.eclipse.e4.core.commands/META-INF/MANIFEST.MF

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,11 @@ Import-Package: jakarta.annotation;version="[2.1.0,3.0.0)",
1414
org.eclipse.e4.core.contexts,
1515
org.eclipse.e4.core.services.log
1616
Export-Package: org.eclipse.e4.core.commands,
17-
org.eclipse.e4.core.commands.internal;x-friends:="org.eclipse.e4.ui.bindings,org.eclipse.ui.workbench"
17+
org.eclipse.e4.core.commands.internal;x-friends:=
18+
"org.eclipse.e4.ui.bindings,
19+
org.eclipse.ui.workbench,
20+
org.eclipse.e4.ui.workbench,
21+
org.eclipse.e4.ui.workbench.renderers.swt"
1822
Require-Bundle: org.eclipse.equinox.common;bundle-version="[3.6.0,4.0.0)",
1923
org.eclipse.e4.core.di;bundle-version="0.9.0"
2024
Automatic-Module-Name: org.eclipse.e4.core.commands

bundles/org.eclipse.e4.ui.css.swt/META-INF/MANIFEST.MF

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,10 @@ Export-Package: org.eclipse.e4.ui.css.swt;x-friends:="org.eclipse.ui.workbench",
1616
org.eclipse.e4.ui.css.swt.dom.html;x-internal:=true,
1717
org.eclipse.e4.ui.css.swt.dom.preference;x-internal:=true,
1818
org.eclipse.e4.ui.css.swt.engine;x-friends:="org.eclipse.e4.ui.css.swt.theme,org.eclipse.e4.ui.workbench.swt",
19-
org.eclipse.e4.ui.css.swt.helpers;x-friends:="org.eclipse.ui.views.properties.tabbed,org.eclipse.e4.ui.css.swt.theme",
19+
org.eclipse.e4.ui.css.swt.helpers;x-friends:="
20+
org.eclipse.ui.views.properties.tabbed,
21+
org.eclipse.e4.ui.workbench.swt,
22+
org.eclipse.e4.ui.css.swt.theme",
2023
org.eclipse.e4.ui.css.swt.properties;x-friends:="org.eclipse.ui.workbench,org.eclipse.ui.views.properties.tabbed,org.eclipse.ui.forms",
2124
org.eclipse.e4.ui.css.swt.properties.converters;x-internal:=true,
2225
org.eclipse.e4.ui.css.swt.properties.css2;x-internal:=true,

bundles/org.eclipse.e4.ui.model.workbench/META-INF/MANIFEST.MF

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ Export-Package: org.eclipse.e4.ui.model;x-friends:="org.eclipse.e4.ui.workbench,
2424
org.eclipse.e4.ui.workbench,
2525
org.eclipse.e4.ui.workbench.swt,
2626
org.eclipse.e4.ui.workbench.addons.swt,
27+
org.eclipse.e4.tools.persistence,
2728
org.eclipse.e4.emf.xpath.test",
2829
org.eclipse.e4.ui.model.application.ui,
2930
org.eclipse.e4.ui.model.application.ui.advanced,
@@ -32,6 +33,7 @@ Export-Package: org.eclipse.e4.ui.model;x-friends:="org.eclipse.e4.ui.workbench,
3233
org.eclipse.e4.ui.workbench.addons.swt,
3334
org.eclipse.e4.ui.workbench.swt,
3435
org.eclipse.ui.workbench,
36+
org.eclipse.e4.tools.persistence,
3537
org.eclipse.e4.emf.xpath.test",
3638
org.eclipse.e4.ui.model.application.ui.advanced.util;x-internal:=true,
3739
org.eclipse.e4.ui.model.application.ui.basic,
@@ -41,6 +43,7 @@ Export-Package: org.eclipse.e4.ui.model;x-friends:="org.eclipse.e4.ui.workbench,
4143
org.eclipse.e4.ui.workbench.addons.swt,
4244
org.eclipse.e4.ui.workbench.swt,
4345
org.eclipse.ui.workbench,
46+
org.eclipse.e4.tools.persistence,
4447
org.eclipse.e4.emf.xpath.test",
4548
org.eclipse.e4.ui.model.application.ui.basic.util;x-internal:=true,
4649
org.eclipse.e4.ui.model.application.ui.impl;
@@ -65,7 +68,9 @@ Export-Package: org.eclipse.e4.ui.model;x-friends:="org.eclipse.e4.ui.workbench,
6568
org.eclipse.e4.ui.model.fragment,
6669
org.eclipse.e4.ui.model.fragment.impl;x-friends:="org.eclipse.e4.ui.workbench",
6770
org.eclipse.e4.ui.model.fragment.util;x-internal:=true,
68-
org.eclipse.e4.ui.model.internal;x-friends:="org.eclipse.e4.ui.workbench"
71+
org.eclipse.e4.ui.model.internal;x-friends:=
72+
"org.eclipse.e4.ui.workbench,
73+
org.eclipse.ui.workbench"
6974
Require-Bundle: org.eclipse.core.runtime;bundle-version="[3.29.0,4.0.0)",
7075
org.eclipse.e4.core.services;bundle-version="0.9.0",
7176
org.eclipse.e4.core.contexts;bundle-version="1.0.0",

bundles/org.eclipse.e4.ui.workbench.addons.swt/META-INF/MANIFEST.MF

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ Import-Package: jakarta.annotation;version="[2.1.0,3.0.0)",
2727
org.osgi.service.event;version="[1.3.0,2.0.0)"
2828
Export-Package: org.eclipse.e4.ui.workbench.addons.cleanupaddon;x-internal:=true,
2929
org.eclipse.e4.ui.workbench.addons.dndaddon;x-internal:=true,
30-
org.eclipse.e4.ui.workbench.addons.minmax;x-internal:=true,
30+
org.eclipse.e4.ui.workbench.addons.minmax;x-friends:="org.eclipse.ui.workbench",
3131
org.eclipse.e4.ui.workbench.addons.splitteraddon;x-friends:="org.eclipse.ui.workbench",
3232
org.eclipse.e4.ui.workbench.addons.swt;x-internal:=true
3333
Automatic-Module-Name: org.eclipse.e4.ui.workbench.addons.swt

bundles/org.eclipse.e4.ui.workbench/META-INF/MANIFEST.MF

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ Export-Package: org.eclipse.e4.ui.internal.workbench;
2828
org.eclipse.ui.ide.application,
2929
org.eclipse.ui.ide,
3030
org.eclipse.e4.emf.xpath.test",
31-
org.eclipse.e4.ui.internal.workbench.addons;x-internal:=true,
31+
org.eclipse.e4.ui.internal.workbench.addons;x-friends:="org.eclipse.ui.workbench",
3232
org.eclipse.e4.ui.internal.workbench.handlers;x-internal:=true,
3333
org.eclipse.e4.ui.workbench,
3434
org.eclipse.e4.ui.workbench.lifecycle,

bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/WorkbenchPage.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,7 @@
9292
import org.eclipse.e4.ui.workbench.IPresentationEngine;
9393
import org.eclipse.e4.ui.workbench.UIEvents;
9494
import org.eclipse.e4.ui.workbench.UIEvents.EventTags;
95+
import org.eclipse.e4.ui.workbench.addons.minmax.TrimStack;
9596
import org.eclipse.e4.ui.workbench.modeling.EModelService;
9697
import org.eclipse.e4.ui.workbench.modeling.EPartService;
9798
import org.eclipse.e4.ui.workbench.modeling.EPartService.PartState;
@@ -873,13 +874,10 @@ private void handleMinimizedStacks(@UIEventTopic(UIEvents.ApplicationElement.TOP
873874

874875
final MToolControl minimizedStack = (MToolControl) changedObj;
875876

876-
// Note: The non-API type TrimStack is not imported to avoid
877-
// https://bugs.eclipse.org/435521
878-
if (!(minimizedStack.getObject() instanceof org.eclipse.e4.ui.workbench.addons.minmax.TrimStack))
877+
if (!(minimizedStack.getObject() instanceof TrimStack))
879878
return;
880879

881-
org.eclipse.e4.ui.workbench.addons.minmax.TrimStack ts = (org.eclipse.e4.ui.workbench.addons.minmax.TrimStack) minimizedStack
882-
.getObject();
880+
TrimStack ts = (TrimStack) minimizedStack.getObject();
883881
if (!(ts.getMinimizedElement() instanceof MPartStack))
884882
return;
885883

@@ -926,7 +924,7 @@ private void handleMinimizedStacks(@UIEventTopic(UIEvents.ApplicationElement.TOP
926924
* @param w the parent window
927925
* @param input the page input
928926
*/
929-
public WorkbenchPage(WorkbenchWindow w, IAdaptable input) throws WorkbenchException {
927+
public WorkbenchPage(WorkbenchWindow w, IAdaptable input) {
930928
super();
931929
init(w, input);
932930
}

bundles/org.eclipse.ui.workbench/Eclipse UI/org/eclipse/ui/internal/WorkbenchWindow.java

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -815,11 +815,7 @@ public boolean saveParts(Collection<MPart> dirtyParts, boolean confirm) {
815815
fireWindowOpening();
816816
configureShell(getShell(), windowContext);
817817

818-
try {
819-
page = new WorkbenchPage(this, input);
820-
} catch (WorkbenchException e) {
821-
WorkbenchPlugin.log(e);
822-
}
818+
page = new WorkbenchPage(this, input);
823819
menuOverride = new MenuOverrides(page);
824820
toolbarOverride = new ToolbarOverrides(page);
825821

bundles/org.eclipse.ui.workbench/META-INF/MANIFEST.MF

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@ Export-Package: org.eclipse.e4.ui.workbench.addons.perspectiveswitcher;x-interna
3030
org.eclipse.ui.ide.application,
3131
org.eclipse.ui.views,
3232
org.eclipse.ui.navigator.resources,
33+
org.eclipse.ui.tests.browser,
3334
org.eclipse.debug.ui",
3435
org.eclipse.ui.internal.about;x-friends:="org.eclipse.ui,org.eclipse.equinox.p2.ui",
3536
org.eclipse.ui.internal.actions;x-friends:="org.eclipse.ui.ide",

0 commit comments

Comments
 (0)