diff --git a/.github/workflows/maven.yml b/.github/workflows/maven.yml
index 034d6f2f673..87a935406a5 100644
--- a/.github/workflows/maven.yml
+++ b/.github/workflows/maven.yml
@@ -72,7 +72,7 @@ jobs:
--threads 1C
-DforkCount=1
'-Dnative=${{ matrix.config.native }}'
- -Papi-check
+ -Papi-check -Pjavadoc
'-Dtycho.baseline.replace=none'
--fail-at-end
-DskipNativeTests=false
diff --git a/Jenkinsfile b/Jenkinsfile
index e11010ad978..df61a78b02a 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -351,7 +351,7 @@ pipeline {
sh '''
mvn clean verify \
--batch-mode --threads 1C -V -U -e -DforkCount=0 \
- -Papi-check \
+ -Pbree-libs -Papi-check -Pjavadoc \
-Dcompare-version-with-baselines.skip=false \
-Dorg.eclipse.swt.tests.junit.disable.test_isLocal=true \
-Dmaven.test.failure.ignore=true -Dmaven.test.error.ignore=true
diff --git a/binaries/pom.xml b/binaries/pom.xml
index 743ed32629c..54c2920b8f0 100644
--- a/binaries/pom.xml
+++ b/binaries/pom.xml
@@ -83,6 +83,24 @@
+
* IMPORTANT: This field is not part of the SWT * public API. It is marked public only so that it can be shared * within the packages provided by SWT. It is not available on all @@ -1393,7 +1393,7 @@ public id internal_accessibilityHitTest(NSPoint point, int childID) { * Return YES if the UIElement doesn't show up to the outside world - * i.e. its parent should return the UIElement's children as its own - * cutting the UIElement out. E.g. NSControls are ignored when they are single-celled. - * + *
* IMPORTANT: This field is not part of the SWT * public API. It is marked public only so that it can be shared * within the packages provided by SWT. It is not available on all @@ -1422,7 +1422,7 @@ public boolean internal_accessibilityIsIgnored(int childID) { /** * Return the array of supported attributes that take parameters. - * + *
* IMPORTANT: This field is not part of the SWT * public API. It is marked public only so that it can be shared * within the packages provided by SWT. It is not available on all @@ -1484,7 +1484,7 @@ public NSArray internal_accessibilityParameterizedAttributeNames(int childID) { /** * Performs the specified action. - * + *
* IMPORTANT: This field is not part of the SWT * public API. It is marked public only so that it can be shared * within the packages provided by SWT. It is not available on all @@ -1530,7 +1530,7 @@ public boolean internal_accessibilityPerformAction(NSString action, int childID) /** * Set the value of the specified attribute to the given value. * Unsupported attributes are ignored. - * + *
* IMPORTANT: This field is not part of the SWT * public API. It is marked public only so that it can be shared * within the packages provided by SWT. It is not available on all diff --git a/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/GTK.java b/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/GTK.java index 29f15836b1d..a7793a09f6a 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/GTK.java +++ b/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/GTK.java @@ -185,7 +185,7 @@ public class GTK extends OS { public static final byte[] gtk_application_prefer_dark_theme = OS.ascii("gtk-application-prefer-dark-theme"); /** Named icons. - * See https://docs.google.com/spreadsheet/pub?key=0AsPAM3pPwxagdGF4THNMMUpjUW5xMXZfdUNzMXhEa2c&output=html + * See https://docs.google.com/spreadsheet/pub?key=0AsPAM3pPwxagdGF4THNMMUpjUW5xMXZfdUNzMXhEa2c&output=html * See http://standards.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html#names * Icon preview tool: gtk3-icon-browser * Snippets often demonstrate usage of these. E.x 309, 258. diff --git a/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/OS.java b/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/OS.java index 97cc31ee5d8..37601e1f413 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/OS.java +++ b/bundles/org.eclipse.swt/Eclipse SWT PI/gtk/org/eclipse/swt/internal/gtk/OS.java @@ -741,7 +741,6 @@ public static boolean isX11 () { public static final native long G_OBJECT_CLASS_CONSTRUCTOR(long object_class); /** * @param object_class cast=(GObjectClass *) - * @paramOFF constructor cast=(GObject* (*) (GType, guint, GObjectConstructParam *)) */ public static final native void G_OBJECT_CLASS_SET_CONSTRUCTOR(long object_class, long constructor); /** @param xevent cast=(XEvent *) */ diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/Converter.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/Converter.java index f6903657204..dc819bff0c5 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/Converter.java +++ b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/Converter.java @@ -31,7 +31,7 @@ * * This class is tested via: org.eclipse.swt.tests.gtk.Test_GtkTextEncoding * - * About JNI & string conversion: + * About JNI & string conversion: * ############################# * - Regular JNI String conversion usually uses a modified UTF-8, see: https://en.wikipedia.org/wiki/UTF-8#Modified_UTF-8 * - And in JNI, normally (env*)->GetStringUTFChars(..) is used to convert a javaString into a C string. @@ -50,12 +50,12 @@ * Java uses UTF-16 Wide characters internally to represent a string. * C uses UTF-8 Multibyte characters (null terminated) to represent a string. * - * About encoding on Linux/Gtk & it's relevance to SWT: + * About encoding on Linux/Gtk & it's relevance to SWT: * #################################################### * * UTF-* = variable length encoding. * - * UTF-8 = minimum is 8 bits, max is 6 bytes, but rarely goes beyond 4 bytes. Gtk & most of web uses this. + * UTF-8 = minimum is 8 bits, max is 6 bytes, but rarely goes beyond 4 bytes. Gtk & most of web uses this. * UTF-16 = minimum is 16 bits. Java's string are stored this way. * UTF-16 can be * Big Endian : 65 = 00000000 01000001 # Human friendly, reads left to right. diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/GDBus.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/GDBus.java index 648c9ac8dcd..7cc17b2d6e1 100644 --- a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/GDBus.java +++ b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/internal/GDBus.java @@ -58,7 +58,7 @@ public static class GDBusMethod { * @param inputArgs 2D array pair of Strings in the format of: (DBUS_TYPE_*, argument_name). * Where argument_name is only so that it's seen by command line by user. * @param outputArgs Same as inputArgs, but for returning values. - * @param userFunction A Function
+ *
* Here is an example of using aTable
with style VIRTUAL
:
- *
+ *
* final Table table = new Table (parent, SWT.VIRTUAL | SWT.BORDER);
* table.setItemCount (1000000);
* table.addListener (SWT.SetData, new Listener () {
@@ -73,20 +73,20 @@
* System.out.println (item.getText ());
* }
* });
- *
- * + * + *
* Note that although this class is a subclass of Composite
,
* it does not normally make sense to add Control
children to
* it, or set a layout on it, unless implementing something like a cell
* editor.
- *
+ *
*+ *
* Note: Only one of the styles SINGLE, and MULTI may be specified. *
* IMPORTANT: This class is not intended to be subclassed. @@ -3844,7 +3844,7 @@ void setHeaderImageHeight (int value) { * it visible may not actually cause it to be displayed. *
* - * @param show the new visibility state + * @param value the new visibility state * * @exception SWTException+ *
* Note: Only one of the styles LEFT, RIGHT and CENTER may be specified. *
* IMPORTANT: This class is not intended to be subclassed. @@ -679,7 +679,7 @@ public void setMoveable (boolean moveable) { * cannot be dragged by the user but may be resized * by the programmer. * - * @param resizable the resize attribute + * @param value the resize attribute * * @exception SWTException
Text
widget contents
- * and dialog behaviour accordingly.
- *
- * @param dialogState int
- * The state the dialog should be opened in.
+ * and dialog behavior accordingly.
*/
public String[] open() {
createTextWidgets();
diff --git a/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/graphics/ColorListener.java b/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/graphics/ColorListener.java
index 6105285e4d2..811bee1c51d 100644
--- a/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/graphics/ColorListener.java
+++ b/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/graphics/ColorListener.java
@@ -17,8 +17,8 @@
/**
* Used to perform an action after an item in a Menu has been selected.
*
- * @see org.eclipse.swt.examples.graphics.ColorMenu.java
- * @see org.eclipse.swt.examples.graphics.GraphicsBackground.java
+ * @see org.eclipse.swt.examples.graphics.ColorMenu
+ * @see org.eclipse.swt.examples.graphics.GraphicsBackground
*/
public interface ColorListener {
diff --git a/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/hoverhelp/HoverHelp.java b/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/hoverhelp/HoverHelp.java
index 15a313907c1..9451b18d44a 100644
--- a/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/hoverhelp/HoverHelp.java
+++ b/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/hoverhelp/HoverHelp.java
@@ -265,7 +265,7 @@ public ToolTipHandler(Shell parent) {
/**
* Enables customized hover help for a specified control
*
- * @control the control on which to enable hoverhelp
+ * @param control the control on which to enable hoverhelp
*/
public void activateHoverHelp(final Control control) {
/*
@@ -347,7 +347,6 @@ public void mouseHover (MouseEvent event) {
* Sets the location for a hovering shell
* @param shell the object that is to hover
* @param position the position of a widget to hover over
- * @return the top-left location for a hovering box
*/
private void setHoverLocation(Shell shell, Point position) {
Rectangle displayBounds = shell.getDisplay().getBounds();
diff --git a/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/paint/ContinuousPaintSession.java b/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/paint/ContinuousPaintSession.java
index b3225e70dc4..4312e2596dd 100644
--- a/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/paint/ContinuousPaintSession.java
+++ b/examples/org.eclipse.swt.examples/src/org/eclipse/swt/examples/paint/ContinuousPaintSession.java
@@ -162,8 +162,9 @@ private final void mouseSegmentFinished(MouseEvent event) {
/**
* Draws a continuous segment from points[0] to points[1].
* Assumes points[0] has been drawn already.
- *
- * @post points[0] will refer to the same point as points[1]
+ * * Steps to reproduce: + *
** Steps to reproduce: + *
** Steps to reproduce: + *
** Steps to reproduce: + *
*diff --git a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug579283_TestToolbarOverflow_itemEnablement.java b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug579283_TestToolbarOverflow_itemEnablement.java index 3826a9bafcf..a4a983e85b5 100644 --- a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug579283_TestToolbarOverflow_itemEnablement.java +++ b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Bug579283_TestToolbarOverflow_itemEnablement.java @@ -27,10 +27,11 @@ import org.eclipse.swt.widgets.ToolItem; /** - * Description: Toolbar items in a toolbar overflow submenu (enabled with style flag {@link SWT.WRAP}) + * Description: Toolbar items in a toolbar overflow submenu (enabled with style flag {@link SWT#WRAP}) * do not show correct image enablement. ** Steps to reproduce: + *
**
- * * Expected results: Images of disabled buttons should appear grayed out. * Images should change on overflow submenu item click. * Text and tooltip of an overflow submenu item should also change on click. diff --git a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Issue346_JvmCrashRemovingMenuItemAccelerators.java b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Issue346_JvmCrashRemovingMenuItemAccelerators.java index 32b16e01eb7..6f9cd74a460 100644 --- a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Issue346_JvmCrashRemovingMenuItemAccelerators.java +++ b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/gtk/snippets/Issue346_JvmCrashRemovingMenuItemAccelerators.java @@ -29,9 +29,7 @@ *- Run the snippet.
*- Resize the {@link Shell}, so that only some of the icons are visible.
@@ -40,7 +41,6 @@ *- Click one of the overflow submenu items.
*- Show the overflow menu again, observe that the image of the clicked item is not changed.
*
- * + * following segfault: *
* Native frames: (J=compiled Java code, A=aot compiled Java code, j=interpreted, Vv=VM code, C=native code) * C [libgtk-4.so.1+0x213053] gtk_shortcut_controller_remove_shortcut+0x53 @@ -40,6 +38,7 @@ * j org.eclipse.swt.internal.gtk4.GTK4.gtk_shortcut_controller_remove_shortcut(JJ)V+0 * j org.eclipse.swt.widgets.MenuItem.setAccelerator(I)V+39 *+ * * * * Tested on GTK 4.6.8 (Fedora 36) diff --git a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Template.java b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Template.java index 35c2ad2356b..15b1bc75352 100644 --- a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Template.java +++ b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Template.java @@ -21,7 +21,7 @@ import org.junit.Test; import org.junit.runners.MethodSorters; -/** Convenience class for easy copy & paste */ +/** Convenience class for easy copy & paste */ @FixMethodOrder(MethodSorters.JVM) public class MJ_Template extends MJ_root { diff --git a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Tree.java b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Tree.java index bb284110c12..e3ea5725329 100644 --- a/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Tree.java +++ b/tests/org.eclipse.swt.tests.gtk/ManualTests/org/eclipse/swt/tests/manualJUnit/MJ_Tree.java @@ -36,7 +36,7 @@ import org.junit.Test; import org.junit.runners.MethodSorters; -/** Convenience class for easy copy & paste */ +/** Convenience class for easy copy & paste */ @FixMethodOrder(MethodSorters.JVM) public class MJ_Tree extends MJ_root { diff --git a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/SwtTestUtil.java b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/SwtTestUtil.java index fc3bdeaa009..122e4a1eade 100644 --- a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/SwtTestUtil.java +++ b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/SwtTestUtil.java @@ -452,7 +452,6 @@ public static boolean waitEvent(Runnable trigger, Control control, int swtEvent, * arrive *before* you call this function, and it will * fail to receive event. * @param shell the Shell to wait for - * @return
true
if Shell became active within timeout
*/
public static void waitShellActivate(Runnable trigger, Shell shell) {
final int timeoutInMsec = 3000;
diff --git a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java
index 35adebff18c..87b73ed8d1f 100644
--- a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java
+++ b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_browser_Browser.java
@@ -1031,7 +1031,7 @@ public void test_StatusTextListener_addAndRemove() {
* 3) Upon compleation of page load, move cursor across whole shell.
* (Note, in current jUnit, browser sometimes only takes up half the shell).
* 4) StatusTextListener should get triggered. Test passes.
- * 5) Else timeout & fail.
+ * 5) Else timeout and fail.
*
* Set variable "debug_show_browser" to true to see this being performed at human-observable speed.
*
@@ -2242,8 +2242,8 @@ function callCustomFunction() {
/**
* Test for stacked (cascaded) calls between Java and JS i.e. java calls JS
* which calls Java which calls JS and so on.
- *
- * @see https://github.com/eclipse-platform/eclipse.platform.swt/issues/1919
+ * + * See {@code https://github.com/eclipse-platform/eclipse.platform.swt/issues/1919} * */ @Test diff --git a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_custom_CTabFolder.java b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_custom_CTabFolder.java index f4900529746..536d4f9715c 100644 --- a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_custom_CTabFolder.java +++ b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_custom_CTabFolder.java @@ -280,8 +280,8 @@ public void test_checkSize() { * * We define two {@link CTabFolder tab folders}, of which one has a nested tab folder. * We validate that selecting the nested tab does not break selection highlight for the top-level tabs. - * - * @see org.eclipse.swt.tests.manual.Bug528251_CTabFolder_nested_highlighting + *
+ * See also {@code org.eclipse.swt.tests.manual.Bug528251_CTabFolder_nested_highlighting} */ @Test public void test_nestedTabHighlighting () { diff --git a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_List.java b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_List.java index bc6f2480d32..0f11a00387f 100644 --- a/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_List.java +++ b/tests/org.eclipse.swt.tests/JUnit Tests/org/eclipse/swt/tests/junit/Test_org_eclipse_swt_widgets_List.java @@ -627,8 +627,6 @@ public void test_getSelection() { /** * Returns the number of selected items contained in the receiver. * - * @return the number of selected items - * * @exception SWTException