Use PDE API-Tools java-annotations instead of javadoc-annotations

Fixes #1043

binaries/.classpath_cocoa

@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <classpath>
     <classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>
+    <classpathentry kind="con" path="org.eclipse.pde.core.requiredPlugins"/>
     <classpathentry kind="src" path="Eclipse SWT/cocoa"/>
     <classpathentry kind="src" path="Eclipse SWT/common"/>
     <classpathentry kind="src" path="Eclipse SWT/emulated/bidi"/>

binaries/.classpath_gtk

@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <classpath>
 	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>
+	<classpathentry kind="con" path="org.eclipse.pde.core.requiredPlugins"/>
 	<classpathentry kind="src" path="Eclipse SWT/gtk"/>
 	<classpathentry kind="src" path="Eclipse SWT/cairo"/>
 	<classpathentry kind="src" path="Eclipse SWT/emulated/bidi"/>

binaries/.classpath_win32

@@ -1,6 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <classpath>
 	<classpathentry kind="con" path="org.eclipse.jdt.launching.JRE_CONTAINER/org.eclipse.jdt.internal.debug.ui.launcher.StandardVMType/JavaSE-17"/>
+	<classpathentry kind="con" path="org.eclipse.pde.core.requiredPlugins"/>
 	<classpathentry kind="src" path="Eclipse SWT/win32"/>
 	<classpathentry kind="src" path="Eclipse SWT/common"/>
 	<classpathentry kind="src" path="Eclipse SWT PI/common"/>

binaries/org.eclipse.swt.cocoa.macosx.aarch64/build.properties

@@ -41,6 +41,9 @@ source.. = \
 output.. = bin/
 src.includes = about.html,about_files/
 
+# Require latest api-tools annotations that provide a value argument. Can be removed when built into Tycho
+additional.bundles = org.eclipse.pde.api.tools.annotations
+
 pom.model.property.os=macosx
 pom.model.property.ws=cocoa
 pom.model.property.arch=aarch64

binaries/org.eclipse.swt.cocoa.macosx.x86_64/build.properties

@@ -41,6 +41,9 @@ source.. = \
 output.. = bin/
 src.includes = about.html,about_files/
 
+# Require latest api-tools annotations that provide a value argument. Can be removed when built into Tycho
+additional.bundles = org.eclipse.pde.api.tools.annotations
+
 pom.model.property.os=macosx
 pom.model.property.ws=cocoa
 pom.model.property.arch=x86_64

binaries/org.eclipse.swt.gtk.linux.aarch64/build.properties

@@ -42,6 +42,9 @@ source.. = \
 output.. = bin/
 src.includes = about.html,about_files/
 
+# Require latest api-tools annotations that provide a value argument. Can be removed when built into Tycho
+additional.bundles = org.eclipse.pde.api.tools.annotations
+
 pom.model.property.os=linux
 pom.model.property.ws=gtk
 pom.model.property.arch=aarch64

binaries/org.eclipse.swt.gtk.linux.loongarch64/build.properties

@@ -42,6 +42,9 @@ source.. = \
 output.. = bin/
 src.includes = about.html,about_files/
 
+# Require latest api-tools annotations that provide a value argument. Can be removed when built into Tycho
+additional.bundles = org.eclipse.pde.api.tools.annotations
+
 pom.model.property.os=linux
 pom.model.property.ws=gtk
 pom.model.property.arch=loongarch64

binaries/org.eclipse.swt.gtk.linux.ppc64le/build.properties

@@ -42,6 +42,9 @@ source.. = \
 output.. = bin/
 src.includes = about.html,about_files/
 
+# Require latest api-tools annotations that provide a value argument. Can be removed when built into Tycho
+additional.bundles = org.eclipse.pde.api.tools.annotations
+
 pom.model.property.os=linux
 pom.model.property.ws=gtk
 pom.model.property.arch=ppc64le

binaries/org.eclipse.swt.gtk.linux.x86_64/build.properties

@@ -43,6 +43,9 @@ source.. = \
 output.. = bin/
 src.includes = about.html,about_files/
 
+# Require latest api-tools annotations that provide a value argument. Can be removed when built into Tycho
+additional.bundles = org.eclipse.pde.api.tools.annotations
+
 pom.model.property.os=linux
 pom.model.property.ws=gtk
 pom.model.property.arch=x86_64

binaries/org.eclipse.swt.win32.win32.x86_64/build.properties

@@ -37,6 +37,9 @@ source.. = \
 output.. = bin/
 src.includes = about.html,about_files/
 
+# Require latest api-tools annotations that provide a value argument. Can be removed when built into Tycho
+additional.bundles = org.eclipse.pde.api.tools.annotations
+
 pom.model.property.os=win32
 pom.model.property.ws=win32
 pom.model.property.arch=x86_64

bundles/org.eclipse.swt.tools/Oomph/platformSwt.setup

@@ -58,6 +58,8 @@
         activeRepositoryList="${eclipse.target.platform}">
       <requirement
           name="*"/>
+      <requirement
+          name="org.eclipse.pde.api.tools.annotations"/>
       <sourceLocator
           rootFolder="${github.clone.platform.swt.location}"
           locateNestedProjects="true">

bundles/org.eclipse.swt/Eclipse SWT Accessibility/cocoa/org/eclipse/swt/accessibility/Accessible.java

@@ -17,6 +17,7 @@
 import java.util.*;
 import java.util.List;
 
+import org.eclipse.pde.api.tools.annotations.*;
 import org.eclipse.swt.*;
 import org.eclipse.swt.graphics.*;
 import org.eclipse.swt.internal.cocoa.*;
@@ -138,9 +139,8 @@ protected Accessible() {
 	 *
 	 * @param control the control to get the accessible object for
 	 * @return the platform specific accessible object
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public static Accessible internal_new_Accessible(Control control) {
 		return new Accessible(control);
 	}
@@ -787,9 +787,8 @@ id getColumnsAttribute(int childID) {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public id internal_accessibilityActionDescription(NSString action, int childID) {
 		NSString returnValue = NSString.string();
 		String actionName = action.getString();
@@ -832,9 +831,8 @@ public id internal_accessibilityActionDescription(NSString action, int childID)
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public NSArray internal_accessibilityActionNames(int childID) {
 		if (accessibleActionListenersSize() > 0) {
 			AccessibleActionEvent event = new AccessibleActionEvent(this);
@@ -917,9 +915,8 @@ public NSArray internal_accessibilityActionNames(int childID) {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public boolean internal_accessibilityIsAttributeSettable(NSString attribute, int childID) {
 		if (accessibleTextExtendedListenersSize() > 0) {
 			if (attribute.isEqualToString(OS.NSAccessibilitySelectedTextRangeAttribute)) return true;
@@ -942,9 +939,8 @@ public boolean internal_accessibilityIsAttributeSettable(NSString attribute, int
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public NSArray internal_accessibilityAttributeNames(int childID) {
 		// The supported attribute set depends on the role played by the control.
 		// We may need to add or remove from the base set as needed.
@@ -1224,9 +1220,8 @@ public NSArray internal_accessibilityAttributeNames(int childID) {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public id internal_accessibilityAttributeValue(NSString attribute, int childID) {
 		if (attribute.isEqualToString(OS.NSAccessibilityRoleAttribute)) return getRoleAttribute(childID);
 		if (attribute.isEqualToString(OS.NSAccessibilitySubroleAttribute)) return getSubroleAttribute(childID);
@@ -1283,9 +1278,8 @@ public id internal_accessibilityAttributeValue(NSString attribute, int childID)
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public id internal_accessibilityAttributeValue_forParameter(NSString attribute, id parameter, int childID) {
 		if (attribute.isEqualToString(OS.NSAccessibilityStringForRangeParameterizedAttribute)) return getStringForRangeParameterizedAttribute(parameter, childID);
 		if (attribute.isEqualToString(OS.NSAccessibilityRangeForLineParameterizedAttribute)) return getRangeForLineParameterizedAttribute(parameter, childID);
@@ -1310,9 +1304,8 @@ public id internal_accessibilityAttributeValue_forParameter(NSString attribute,
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public id internal_accessibilityFocusedUIElement(int childID) {
 		AccessibleControlEvent event = new AccessibleControlEvent(this);
 		event.childID = ACC.CHILDID_MULTIPLE; // set to invalid value, to test if the application sets it in getFocus()
@@ -1350,9 +1343,8 @@ public id internal_accessibilityFocusedUIElement(int childID) {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public id internal_accessibilityHitTest(NSPoint point, int childID) {
 		AccessibleControlEvent event = new AccessibleControlEvent(this);
 		event.x = (int) point.x;
@@ -1391,9 +1383,8 @@ public id internal_accessibilityHitTest(NSPoint point, int childID) {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public boolean internal_accessibilityIsIgnored(int childID) {
 		AccessibleControlEvent event = new AccessibleControlEvent(this);
 		event.childID = childID;
@@ -1420,9 +1411,8 @@ public boolean internal_accessibilityIsIgnored(int childID) {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public NSArray internal_accessibilityParameterizedAttributeNames(int childID) {
 		AccessibleControlEvent event = new AccessibleControlEvent(this);
 		event.childID = childID;
@@ -1481,9 +1471,8 @@ public NSArray internal_accessibilityParameterizedAttributeNames(int childID) {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public boolean internal_accessibilityPerformAction(NSString action, int childID) {
 		String actionName = action.getString();
 		if (accessibleActionListenersSize() > 0) {
@@ -1526,9 +1515,8 @@ public boolean internal_accessibilityPerformAction(NSString action, int childID)
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public void internal_accessibilitySetValue_forAttribute(id value, NSString attribute, int childId) {
 		if (attribute.isEqualToString(OS.NSAccessibilitySelectedTextRangeAttribute)) setSelectedTextRangeAttribute(value, childId);
 		if (attribute.isEqualToString(OS.NSAccessibilitySelectedTextAttribute)) setSelectedTextAttribute(value, childId);
@@ -1586,9 +1574,8 @@ public Control getControl() {
 	 * available on all platforms, and should never be called from
 	 * application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public void internal_dispose_Accessible() {
 		release(true);
 	}
@@ -3556,9 +3543,8 @@ boolean isValidThread () {
 	 * within the packages provided by SWT. It is not available on all
 	 * platforms and should never be accessed from application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public long internal_addRelationAttributes(long defaultAttributes) {
 		NSArray attributes = new NSArray(defaultAttributes);
 		NSMutableArray returnArray = NSMutableArray.arrayWithCapacity(attributes.count());

bundles/org.eclipse.swt/Eclipse SWT Accessibility/gtk/org/eclipse/swt/accessibility/Accessible.java

@@ -17,6 +17,7 @@
 import java.util.*;
 import java.util.List;
 
+import org.eclipse.pde.api.tools.annotations.*;
 import org.eclipse.swt.*;
 import org.eclipse.swt.internal.gtk.*;
 import org.eclipse.swt.internal.gtk3.*;
@@ -533,9 +534,8 @@ long getControlHandle () {
 	 * available on all platforms, and should never be called from
 	 * application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public void internal_dispose_Accessible() {
 		release ();
 	}
@@ -552,9 +552,8 @@ public void internal_dispose_Accessible() {
 	 *
 	 * @param control the control to get the accessible object for
 	 * @return the platform specific accessible object
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public static Accessible internal_new_Accessible (Control control) {
 		return new Accessible (control);
 	}

bundles/org.eclipse.swt/Eclipse SWT Accessibility/win32/org/eclipse/swt/accessibility/Accessible.java

@@ -16,6 +16,7 @@
 import java.util.*;
 import java.util.List;
 
+import org.eclipse.pde.api.tools.annotations.*;
 import org.eclipse.swt.*;
 import org.eclipse.swt.graphics.*;
 import org.eclipse.swt.internal.*;
@@ -546,9 +547,8 @@ void createIServiceProvider() {
 	 *
 	 * @param control the control to get the accessible object for
 	 * @return the platform specific accessible object
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public static Accessible internal_new_Accessible(Control control) {
 		return new Accessible(control);
 	}
@@ -907,9 +907,8 @@ public Control getControl() {
 	 * available on all platforms, and should never be called from
 	 * application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public void internal_dispose_Accessible() {
 		if (iaccessible != null) {
 			iaccessible.Release();
@@ -931,9 +930,8 @@ public void internal_dispose_Accessible() {
 	 * available on all platforms, and should never be called from
 	 * application code.
 	 * </p>
-	 *
-	 * @noreference This method is not intended to be referenced by clients.
 	 */
+	@NoReference("This method is not intended to be referenced by clients.")
 	public long internal_WM_GETOBJECT (long wParam, long lParam) {
 		if (objIAccessible == null) return 0;
 		if ((int)lParam == OS.OBJID_CLIENT) {

bundles/org.eclipse.swt/Eclipse SWT Browser/common/org/eclipse/swt/browser/Browser.java

@@ -13,6 +13,7 @@
  *******************************************************************************/
 package org.eclipse.swt.browser;
 
+import org.eclipse.pde.api.tools.annotations.*;
 import org.eclipse.swt.*;
 import org.eclipse.swt.widgets.*;
 
@@ -39,9 +40,8 @@
  * @see <a href="http://www.eclipse.org/swt/">Sample code and further information</a>
  *
  * @since 3.0
- * @noextend This class is not intended to be subclassed by clients.
  */
-
+@NoExtend("This class is not intended to be subclassed by clients.")
 public class Browser extends Composite {
 	WebBrowser webBrowser;
 	int userStyle;

bundles/org.eclipse.swt/Eclipse SWT Custom Widgets/common/org/eclipse/swt/custom/CBanner.java

@@ -14,6 +14,7 @@
 package org.eclipse.swt.custom;
 
 
+import org.eclipse.pde.api.tools.annotations.*;
 import org.eclipse.swt.*;
 import org.eclipse.swt.graphics.*;
 import org.eclipse.swt.widgets.*;
@@ -42,9 +43,8 @@
  * @see <a href="http://www.eclipse.org/swt/">Sample code and further information</a>
  *
  * @since 3.0
- * @noextend This class is not intended to be subclassed by clients.
  */
-
+@NoExtend("This class is not intended to be subclassed by clients.")
 public class CBanner extends Composite {
 
 	Control left;