--- old/SWT/JemmySWT/pom.xml 2018-05-20 10:11:32.036234212 +0200 +++ new/SWT/JemmySWT/pom.xml 2018-05-20 10:11:31.981232534 +0200 @@ -26,12 +26,12 @@ --> 4.0.0 - org.jemmy.v3 - Jemmy - 1.0 + org.adoptopenjdk + jemmy + ${revision} ../../pom.xml - JemmySWT + jemmy-swt jar 4.3 @@ -81,19 +81,19 @@ ${swt.version} - org.jemmy.v3 - JemmyCore - 1.0 + org.adoptopenjdk + jemmy-core + ${revision} - org.jemmy.v3 - JemmyBrowser - 1.0 + org.adoptopenjdk + jemmy-browser + ${revision} - org.jemmy.v3 - JemmyAWTInput - 1.0 + org.adoptopenjdk + jemmy-awt-input + ${revision} --- old/SWT/JemmySWT/src/org/jemmy/swt/CTabFolderWrap.java 2018-05-20 10:11:32.241240468 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/CTabFolderWrap.java 2018-05-20 10:11:32.191238942 +0200 @@ -42,8 +42,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(CTabFolder.class) public class CTabFolderWrap extends ControlWrap implements Selectable { --- old/SWT/JemmySWT/src/org/jemmy/swt/ComboWrap.java 2018-05-20 10:11:32.448246785 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/ComboWrap.java 2018-05-20 10:11:32.399245290 +0200 @@ -44,8 +44,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(Combo.class) public class ComboWrap extends ControlWrap implements --- old/SWT/JemmySWT/src/org/jemmy/swt/CompositeWrap.java 2018-05-20 10:11:32.663253347 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/CompositeWrap.java 2018-05-20 10:11:32.609251699 +0200 @@ -43,8 +43,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(Composite.class) public class CompositeWrap extends ControlWrap implements Parent { --- old/SWT/JemmySWT/src/org/jemmy/swt/ControlWrap.java 2018-05-20 10:11:32.876259847 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/ControlWrap.java 2018-05-20 10:11:32.826258321 +0200 @@ -45,8 +45,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(org.eclipse.swt.widgets.Control.class) public class ControlWrap extends Wrap implements Focusable { --- old/SWT/JemmySWT/src/org/jemmy/swt/ItemWrap.java 2018-05-20 10:11:33.103266774 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/ItemWrap.java 2018-05-20 10:11:33.048265096 +0200 @@ -36,8 +36,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ public class ItemWrap extends Wrap { --- old/SWT/JemmySWT/src/org/jemmy/swt/ScrollableWrap.java 2018-05-20 10:11:33.306272969 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/ScrollableWrap.java 2018-05-20 10:11:33.257271474 +0200 @@ -34,7 +34,6 @@ * Base wrapper for scrollable controls such as Table and Tree * * @author erikgreijus - * @param */ @ControlType(Scrollable.class) public class ScrollableWrap extends ControlWrap { --- old/SWT/JemmySWT/src/org/jemmy/swt/TabFolderWrap.java 2018-05-20 10:11:33.510279195 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/TabFolderWrap.java 2018-05-20 10:11:33.462277730 +0200 @@ -41,8 +41,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(TabFolder.class) public class TabFolderWrap extends ControlWrap implements Selectable { --- old/SWT/JemmySWT/src/org/jemmy/swt/TextWrap.java 2018-05-20 10:11:33.718285543 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/TextWrap.java 2018-05-20 10:11:33.668284017 +0200 @@ -37,8 +37,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(Text.class) public class TextWrap extends ControlWrap implements Focusable { --- old/SWT/JemmySWT/src/org/jemmy/swt/ToolBarWrap.java 2018-05-20 10:11:33.928291951 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/ToolBarWrap.java 2018-05-20 10:11:33.874290303 +0200 @@ -41,8 +41,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(ToolBar.class) public class ToolBarWrap extends ControlWrap implements Selectable { --- old/SWT/JemmySWT/src/org/jemmy/swt/TreeWrap.java 2018-05-20 10:11:34.139298391 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/TreeWrap.java 2018-05-20 10:11:34.090296895 +0200 @@ -47,8 +47,8 @@ /** * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ @ControlType(Tree.class) public class TreeWrap extends ScrollableWrap implements KeyboardSelectable, Focusable { --- old/SWT/JemmySWT/src/org/jemmy/swt/Version.java 2018-05-20 10:11:34.342304586 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/Version.java 2018-05-20 10:11:34.294303121 +0200 @@ -32,16 +32,10 @@ public static final Version VERSION = new Version(); - /** - * - */ public Version() { super(Version.class.getPackage().getName()); } - /** - * - * @param args - */ + public static void main(String[] args) { System.out.println("JemmySWT version: " + VERSION.getVersion()); } --- old/SWT/JemmySWT/src/org/jemmy/swt/jemmy.properties 2018-05-20 10:11:34.549310903 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/jemmy.properties 2018-05-20 10:11:34.499309377 +0200 @@ -1,4 +1,4 @@ -version.major=0 -version.minor=9 -version.mini=6 +version.major=1 +version.minor=0 +version.mini=0 build=${buildnumber} --- old/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByItemLookup.java 2018-05-20 10:11:34.764317464 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByItemLookup.java 2018-05-20 10:11:34.709315786 +0200 @@ -35,9 +35,8 @@ import org.jemmy.resources.StringComparePolicy; /** - * - * @param - * @author klara, erikgreijus + * @author shura + * @author erikgreijus */ public class ByItemLookup extends QueueLookup { --- old/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByItemStringsLookup.java 2018-05-20 10:11:34.974323873 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByItemStringsLookup.java 2018-05-20 10:11:34.925322377 +0200 @@ -32,9 +32,7 @@ import org.jemmy.resources.StringComparePolicy; /** - * * @author erikgreijus - * @param */ public class ByItemStringsLookup extends QueueLookup { --- old/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByItemToolTipLookup.java 2018-05-20 10:11:35.188330404 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByItemToolTipLookup.java 2018-05-20 10:11:35.137328847 +0200 @@ -34,8 +34,6 @@ import org.jemmy.resources.StringComparePolicy; /** - * - * @param * @author klara */ public class ByItemToolTipLookup extends QueueLookup { --- old/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByStringQueueLookup.java 2018-05-20 10:11:35.402336934 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByStringQueueLookup.java 2018-05-20 10:11:35.347335256 +0200 @@ -27,8 +27,6 @@ import org.jemmy.resources.StringComparePolicy; /** - * - * @param * @author klara */ public abstract class ByStringQueueLookup extends QueueLookup { @@ -36,36 +34,17 @@ private final StringComparePolicy policy; private final String text; - /** - * - * @param text - */ protected ByStringQueueLookup(String text) { this(text, StringComparePolicy.SUBSTRING); } - /** - * - * @param text - * @param policy - */ protected ByStringQueueLookup(String text, StringComparePolicy policy) { this.policy = policy; this.text = text; } - /** - * - * @param control - * @return - */ public abstract String getText(T control); - /** - * - * @param control - * @return - */ public boolean doCheck(T control) { return policy.compare(text, getText(control)); } --- old/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByTextControlLookup.java 2018-05-20 10:11:35.609343251 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByTextControlLookup.java 2018-05-20 10:11:35.561341787 +0200 @@ -30,9 +30,8 @@ import org.jemmy.resources.StringComparePolicy; /** - * - * @author klara, erikgreijus - * @param + * @author klara + * @author erikgreijus */ public class ByTextControlLookup extends ByStringQueueLookup { --- old/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByToolTipItem.java 2018-05-20 10:11:35.836350179 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/lookup/ByToolTipItem.java 2018-05-20 10:11:35.781348500 +0200 @@ -30,9 +30,8 @@ import org.jemmy.resources.StringComparePolicy; /** - * - * @author klara, erikgreijus - * @param + * @author klara + * @author erikgreijus */ public class ByToolTipItem extends ByStringQueueLookup { --- old/SWT/JemmySWT/src/org/jemmy/swt/lookup/CoordinateLookup.java 2018-05-20 10:11:36.050356710 +0200 +++ new/SWT/JemmySWT/src/org/jemmy/swt/lookup/CoordinateLookup.java 2018-05-20 10:11:35.994355001 +0200 @@ -31,9 +31,8 @@ import org.jemmy.swt.ControlWrap; /** - * - * @author shura, erikgreijus - * @param + * @author shura + * @author erikgreijus */ public class CoordinateLookup extends RelativeCoordinateLookup { --- old/core/JemmyAWTInput/pom.xml 2018-05-20 10:11:36.269363393 +0200 +++ new/core/JemmyAWTInput/pom.xml 2018-05-20 10:11:36.215361745 +0200 @@ -26,12 +26,12 @@ --> 4.0.0 - org.jemmy.v3 - Jemmy - 1.0 + org.adoptopenjdk + jemmy + ${revision} ../../pom.xml - JemmyAWTInput + jemmy-awt-input jar org/jemmy/input/jemmy.properties @@ -39,9 +39,9 @@ - org.jemmy.v3 - JemmyCore - 1.0 + org.adoptopenjdk + jemmy-core + ${revision} --- old/core/JemmyAWTInput/src/org/jemmy/image/AWTImage.java 2018-05-20 10:11:36.479369802 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/AWTImage.java 2018-05-20 10:11:36.429368276 +0200 @@ -36,14 +36,10 @@ /** - * * @author shura */ public class AWTImage implements Image, WriteableRaster { - /** - * - */ public static final String OUTPUT = AWTImage.class.getName() + ".OUTPUT"; public static final String PNG_FILE = ".png"; @@ -89,7 +85,7 @@ * @see Wrap#waitImage(org.jemmy.image.Image, java.lang.String, * java.lang.String) * @see ImageComparator - * @param comparator + * @param comparator todo document */ public static void setComparator(ImageComparator comparator) { Environment.getEnvironment().setProperty(ImageComparator.class, comparator); @@ -103,11 +99,6 @@ private BufferedImage image; - /** - * Creates an instance - * - * @param img - */ public AWTImage(BufferedImage img) { this.image = img; } @@ -120,7 +111,7 @@ * Compares using current comparator. * * @see AWTImage#getComparator() - * @param img + * @param img todo document * @return diff image. */ public Image compareTo(Image img) { --- old/core/JemmyAWTInput/src/org/jemmy/image/AverageDistanceImageComparator.java 2018-05-20 10:11:36.692376302 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/AverageDistanceImageComparator.java 2018-05-20 10:11:36.639374685 +0200 @@ -42,7 +42,7 @@ /** * Creates comparator with the default sensitivity value = 0.02 * (around 5 in 0-255 color component value). - * @see #NaturalImageComparator(double) + * @see #AverageDistanceImageComparator(double) */ public AverageDistanceImageComparator() { this(0.02); --- old/core/JemmyAWTInput/src/org/jemmy/image/ClasspathImageLoader.java 2018-05-20 10:11:36.895382497 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/ClasspathImageLoader.java 2018-05-20 10:11:36.844380941 +0200 @@ -31,7 +31,8 @@ /** * ImageLoader implementation which is able to load images through * a given classloader. - * @author mrkam, shura + * @author mrkam + * @author shura */ public class ClasspathImageLoader implements ImageLoader { @@ -67,10 +68,6 @@ this.classLoader = classLoader; } - /** - * TODO: Add JavaDoc - * @param rootPackage - */ public void setRootPackage(Package rootPackage) { if (rootPackage != null) { this.packagePrefix = rootPackage.getName().replace('.', '/') + "/"; --- old/core/JemmyAWTInput/src/org/jemmy/image/NaturalImageComparator.java 2018-05-20 10:11:37.123389455 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/NaturalImageComparator.java 2018-05-20 10:11:37.072387899 +0200 @@ -55,9 +55,7 @@ * Meaningful values are between 0 and approx 1.733. 0 means colors should * be equal to pass the comparison, 1.733 (which is more than square root * of 3) means that comparison will be passed even if the colors are - * completely different. You could also use {@linkplain - * #findSensitivity(java.awt.image.BufferedImage, java.awt.image.BufferedImage) - * findSensitivity()} method to obtain necessary sensitivity value. + * completely different. */ public NaturalImageComparator(double sensitivity) { super(new MaxDistanceComparator(sensitivity)); --- old/core/JemmyAWTInput/src/org/jemmy/image/PNGDecoder.java 2018-05-20 10:11:37.339396047 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/PNGDecoder.java 2018-05-20 10:11:37.280394246 +0200 @@ -95,7 +95,7 @@ /** * Decodes image from an input stream passed into constructor. * @return a BufferedImage object - * @throws IOException + * @throws IOException todo document */ public BufferedImage decode() throws IOException { return decode(true); @@ -105,7 +105,7 @@ * Decodes image from an input stream passed into constructor. * @return a BufferedImage object * @param closeStream requests method to close the stream after the image is read - * @throws IOException + * @throws IOException todo document */ public BufferedImage decode(boolean closeStream) throws IOException { --- old/core/JemmyAWTInput/src/org/jemmy/image/PNGEncoder.java 2018-05-20 10:11:37.567403005 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/PNGEncoder.java 2018-05-20 10:11:37.510401265 +0200 @@ -44,12 +44,14 @@ import java.util.zip.DeflaterOutputStream; import org.jemmy.control.ScreenArea; -/** This class allows to encode BufferedImage into B/W, greyscale or true color PNG +/** + * This class allows to encode BufferedImage into B/W, greyscale or true color PNG * image format with maximum compression.
* It also provides complete functionality for capturing full screen, part of * screen or single component, encoding and saving captured image info PNG file. * @author Adam Sotona - * @version 1.0 */ + * @version 1.0 + */ public class PNGEncoder extends Object { /** black and white image mode. */ @@ -63,11 +65,6 @@ CRC32 crc; byte mode; - /** - * - * @param file - * @throws java.io.FileNotFoundException - */ public PNGEncoder(File file) throws FileNotFoundException { this(new FileOutputStream(file)); } --- old/core/JemmyAWTInput/src/org/jemmy/image/ResizeImageComparator.java 2018-05-20 10:11:37.794409932 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/ResizeImageComparator.java 2018-05-20 10:11:37.735408132 +0200 @@ -37,31 +37,40 @@ * Comparator that makes image sizes equal to compare them with other comparator * most of them work only for the images with equal dimensions. * - * It is controlled by three parameters:
  • Resize Mode - defines way of - * resizing images that are being compared. One of the following constants:
    • {@linkplain ResizeMode#NO_RESIZE NO_RESIZE},
    • {@linkplain ResizeMode#PROPORTIONAL_RESIZE PROPORTIONAL_RESIZE},
    • {@linkplain ResizeMode#ARBITRARY_RESIZE ARBITRARY_RESIZE}.
    - * Default value is {@linkplain ResizeMode#PROPORTIONAL_RESIZE}.
  • Resize - * Hint - defines the way of images scaling that is specified when {@linkplain java.awt.Image#getScaledInstance(int, int, int) - * Image.getScaledInstance()} method is invoked. One of the Image.SCALE_XXX is - * expected. Default value is - * {@linkplain java.awt.Image#SCALE_DEFAULT SCALE_DEFAULT}.
  • Proportion - * Distortion Threshold - defines maximum proportion distortion that is used in {@linkplain ResizeMode#PROPORTIONAL_RESIZE - * PROPORTIONAL_RESIZE} mode to deal with cases when rounding and other problems - * could cause sizes to be not ideally proportional.

    + * It is controlled by three parameters: + *

    + * Resize Mode - defines way of resizing images that are being compared. One of the following constants: + *

      +*
    • {@linkplain ResizeMode#NO_RESIZE NO_RESIZE},
    • +*
    • {@linkplain ResizeMode#PROPORTIONAL_RESIZE PROPORTIONAL_RESIZE},
    • +*
    • {@linkplain ResizeMode#ARBITRARY_RESIZE ARBITRARY_RESIZE}.
    • + *
    + * + * Default value is {@linkplain ResizeMode#PROPORTIONAL_RESIZE}. + *
      + *
    • Resize Hint - defines the way of images scaling that is specified when + * {@linkplain java.awt.Image#getScaledInstance(int, int, int) Image.getScaledInstance()} method is invoked. + * One of the Image.SCALE_XXX is expected. Default value is {@linkplain java.awt.Image#SCALE_DEFAULT SCALE_DEFAULT}.
    • + *
    • Proportion Distortion Threshold - defines maximum proportion distortion that is used in + * {@linkplain ResizeMode#PROPORTIONAL_RESIZE PROPORTIONAL_RESIZE} mode to deal with cases when rounding and other + * problems could cause sizes to be not ideally proportional. + *

      * Threshold value is applied in the following way: {@code Math.abs(image.getSize() * scale * - size) / image.getSize() > PROPORTION_DISTORTION_THRESHOLD}, where {@code image.getSize()} * is both image dimensions and {@code size} is target width and height (which * is defined as mininum of sizes of two images) and scale is the scale factor * that scales the particular image to fit the width and height. - *

      + *

      * Default value is 0.02 so as much as 2% of width/height could differ (around 5 * in 0-255 color component values).

    • + *
    * * @author mrkam */ public class ResizeImageComparator extends ResizeComparator { /** - * Indentifies output where resize details are printed. Output is disabled + * Identifies output where resize details are printed. Output is disabled * by default. * * @see Environment#getOutput(java.lang.String) @@ -79,12 +88,8 @@ /** * Resize Modes * - * @see - * #ResizeImageComparator(org.jemmy.image.ResizeImageComparator.ResizeMode, - * org.jemmy.image.ImageComparator) - * @see - * #ResizeImageComparator(org.jemmy.image.ResizeImageComparator.ResizeMode, - * int, double, org.jemmy.image.ImageComparator) + * @see ResizeImageComparator#ResizeImageComparator(ResizeMode, ImageComparator) + * @see ResizeImageComparator#ResizeImageComparator(ResizeMode, double, int, ImageComparator) */ public static enum ResizeMode { @@ -98,9 +103,7 @@ * If images have different proportions no resize is done and original * images are compared. * - * @see - * #ResizeImageComparator(org.jemmy.image.ResizeImageComparator.ResizeMode, - * int, double, org.jemmy.image.ImageComparator) + * @see ResizeImageComparator#ResizeImageComparator(ResizeMode, ImageComparator) */ PROPORTIONAL_RESIZE, /** @@ -111,9 +114,12 @@ } /** - * Creates ResizeImageComparator with default resize settings:
  • resize - * mode: {@linkplain ResizeMode#PROPORTIONAL_RESIZE ResizeMode.PROPORTIONAL_RESIZE}.
  • - *
  • proportion distortion threshold: 0.02.
  • resize hint: {@linkplain java.awt.Image#SCALE_DEFAULT Image.SCALE_DEFAULT}.
  • + * Creates ResizeImageComparator with default resize settings: + *
      + *
    • resize mode: {@linkplain ResizeMode#PROPORTIONAL_RESIZE ResizeMode.PROPORTIONAL_RESIZE}.
    • + *
    • proportion distortion threshold: 0.02.
    • + *
    • resize hint: {@linkplain java.awt.Image#SCALE_DEFAULT Image.SCALE_DEFAULT}.
    • + *
    * * @param subComparator comparator to compare images after resize. * @see java.awt.Image#getScaledInstance(int, int, int) @@ -129,8 +135,11 @@ /** * Creates ResizeImageComparator with the specified resize mode and default - * settings for other parameters:
  • proportion distortion threshold: - * 0.02.
  • resize hint: {@linkplain java.awt.Image#SCALE_DEFAULT Image.SCALE_DEFAULT}.
  • + * settings for other parameters: + *
      + *
    • proportion distortion threshold: 0.02.
    • + *
    • resize hint: {@linkplain java.awt.Image#SCALE_DEFAULT Image.SCALE_DEFAULT}.
    • + *
    * * @param resizeMode resize mode for this comparator. * @param subComparator comparator to compare images after resize. @@ -144,9 +153,12 @@ } /** - * Creates ResizeImageComparator with the following settings:
  • resize - * mode: {@linkplain ResizeMode#PROPORTIONAL_RESIZE ResizeMode.PROPORTIONAL_RESIZE}.
  • - *
  • specified proportion distortion threshold.
  • resize hint: {@linkplain java.awt.Image#SCALE_DEFAULT Image.SCALE_DEFAULT}.
  • + * Creates ResizeImageComparator with the following settings: + *
      + *
    • resize mode: {@linkplain ResizeMode#PROPORTIONAL_RESIZE ResizeMode.PROPORTIONAL_RESIZE}.
    • + *
    • specified proportion distortion threshold.
    • + *
    • resize hint: {@linkplain java.awt.Image#SCALE_DEFAULT Image.SCALE_DEFAULT}.
    • + *
    * * @param propDistThreshold proportion distortion threshold. * @param subComparator comparator to compare images after resize. --- old/core/JemmyAWTInput/src/org/jemmy/image/RoughImageComparator.java 2018-05-20 10:11:38.024416951 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/RoughImageComparator.java 2018-05-20 10:11:37.966415181 +0200 @@ -29,7 +29,8 @@ /** * Compares two images roughly (i.e. not all of the pixel colors should match). * - * @author Alexandre Iline (alexandre.iline@sun.com), KAM + * @author shura + * @author mrkam */ public class RoughImageComparator extends BufferedImageComparator { --- old/core/JemmyAWTInput/src/org/jemmy/image/RoughImageFinder.java 2018-05-20 10:11:38.263424245 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/RoughImageFinder.java 2018-05-20 10:11:38.207422536 +0200 @@ -30,7 +30,7 @@ /** * Performs "rough" image search. * - * @author Alexandre Iline (alexandre.iline@sun.com) + * @author shura */ public class RoughImageFinder implements ImageFinder { double roughness = .0; @@ -60,7 +60,7 @@ * @param image an image to search. * @param index an ordinal image location index. * @return Point where number of unmatching pixels less or equal to - * image1.getWidth() * image1.getHeight() * roughness + * image1.getWidth() * image1.getHeight() * roughness */ public Point findImage(BufferedImage image, int index) { int smallWidth = image.getWidth(); --- old/core/JemmyAWTInput/src/org/jemmy/image/StrictImageComparator.java 2018-05-20 10:11:38.489431142 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/StrictImageComparator.java 2018-05-20 10:11:38.431429372 +0200 @@ -29,7 +29,8 @@ /** * Compares two images strictly (i.e. all the pixel colors should match). * - * @author Alexandre Iline (alexandre.iline@sun.com), KAM + * @author shura + * @author mrkam */ public class StrictImageComparator extends BufferedImageComparator { --- old/core/JemmyAWTInput/src/org/jemmy/image/Utils.java 2018-05-20 10:11:38.711437917 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/image/Utils.java 2018-05-20 10:11:38.648435994 +0200 @@ -32,60 +32,30 @@ /** * - * @author Alexander Kouznetsov + * @author mrkam */ public class Utils { - /** - * - * @param r - * @return java.awt.Rectangle - */ public static java.awt.Rectangle convert(Rectangle r) { return new java.awt.Rectangle(r.x, r.y, r.width, r.height); } - /** - * - * @param r - * @return org.jemmy.Rectangle - */ public static Rectangle convert(java.awt.Rectangle r) { return new Rectangle(r.x, r.y, r.width, r.height); } - /** - * - * @param p - * @return java.awt.Point - */ public static java.awt.Point convert(Point p) { return new java.awt.Point(p.x, p.y); } - /** - * - * @param p - * @return org.jemmy.Point - */ public static Point convert(java.awt.Point p) { return new Point(p.x, p.y); } - /** - * - * @param d - * @return java.awt.Dimension - */ public static java.awt.Dimension convert(Dimension d) { return new java.awt.Dimension(d.width, d.height); } - /** - * - * @param d - * @return org.jemmy.Dimension - */ public static Dimension convert(java.awt.Dimension d) { return new Dimension(d.width, d.height); } --- old/core/JemmyAWTInput/src/org/jemmy/input/AWTMap.java 2018-05-20 10:11:38.940444906 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/AWTMap.java 2018-05-20 10:11:38.882443135 +0200 @@ -43,7 +43,7 @@ /** * Converts - * @author Alexander Kouznetsov + * @author mrkam */ public class AWTMap { @@ -129,7 +129,7 @@ /** * TODO Provide javadoc - * @param button + * @param button todo document * @return One of InputEvent.VK_* constants * @see InputEvent */ @@ -141,11 +141,6 @@ } } - /** - * TODO Provide javadoc - * @param modifiers - * @return - */ public int convert(Modifier... modifiers) { int result = 0; for (Modifier modifier : modifiers) { @@ -158,11 +153,6 @@ return result; } - /** - * TODO Provide javadoc - * @param button - * @return - */ public int convert(MouseButton button) { try { return button2int.get(button); @@ -171,11 +161,6 @@ } } - /** - * TODO Provide javadoc - * @param key - * @return - */ public KeyboardButton convertKeyboardButton(int key) { KeyboardButton res = int2key.get(key); if (res == null) { @@ -184,11 +169,6 @@ return res; } - /** - * TODO Provide javadoc - * @param modifiers - * @return - */ public Modifier[] convertModifiers(int modifiers) { List result = new ArrayList(); for (int key : int2modifier.keySet()) { @@ -203,11 +183,6 @@ return result.toArray(new Modifier[result.size()]); } - /** - * TODO Provide javadoc - * @param button - * @return - */ public MouseButton convertMouseButton(int button) { MouseButton res = int2button.get(button); if (res == null) { --- old/core/JemmyAWTInput/src/org/jemmy/input/ClassReference.java 2018-05-20 10:11:39.164451741 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/ClassReference.java 2018-05-20 10:11:39.104449910 +0200 @@ -55,7 +55,7 @@ * to access static methods and fields only. * * @param className name of class - * @exception ClassNotFoundException + * @throws ClassNotFoundException todo document */ public ClassReference(String className) throws ClassNotFoundException { @@ -69,8 +69,8 @@ * with a zero-length java.lang.String array * as a parameter. * - * @exception NoSuchMethodException - * @exception InvocationTargetException + * @throws NoSuchMethodException when the method cannot be found. + * @throws InvocationTargetException when the invoked method throws an exception. */ public void startApplication() throws InvocationTargetException, NoSuchMethodException { @@ -83,8 +83,8 @@ * * @param params The java.lang.String array to pass * to main(java.lang.String[]). - * @exception NoSuchMethodException - * @exception InvocationTargetException + * @throws NoSuchMethodException when the method cannot be found. + * @throws InvocationTargetException when the invoked method throws an exception. */ public void startApplication(String[] params) throws InvocationTargetException, NoSuchMethodException { @@ -111,8 +111,8 @@ * @param method_name Name of method. * @param params Method parameters. * @param params_classes Method parameters types. - * @return the return value from an invocation of the Method.
    - * If method_name method is void, null is returned.
    + * @return the return value from an invocation of the Method.
    + * If method_name method is void, null is returned.
    * If method_name method returns a primitive type, then * return wrapper class instance. * @throws InvocationTargetException when the invoked method throws an exception. --- old/core/JemmyAWTInput/src/org/jemmy/input/DragImpl.java 2018-05-20 10:11:39.397458852 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/DragImpl.java 2018-05-20 10:11:39.336456990 +0200 @@ -40,14 +40,10 @@ /** - * * @author shura */ public class DragImpl implements Drag { - /** - * - */ public static final int DND_POINTS = 10; static { @@ -58,60 +54,26 @@ private Wrap source; - /** - * - * @param source - */ public DragImpl(Wrap source) { this.source = source; } - /** - * - * @param targetPoint - */ public void dnd(Point targetPoint) { dnd(source, targetPoint); } - /** - * - * @param target - * @param targetPoint - */ public void dnd(Wrap target, Point targetPoint) { dnd(source.getClickPoint(), target, targetPoint); } - /** - * - * @param point - * @param target - * @param targetPoint - */ public void dnd(Point point, Wrap target, Point targetPoint) { dnd(point, target, targetPoint, MouseButtons.BUTTON1); } - /** - * - * @param point - * @param target - * @param targetPoint - * @param button - */ public void dnd(Point point, Wrap target, Point targetPoint, MouseButton button) { dnd(point, target, targetPoint, button, new Modifier[]{}); } - /** - * - * @param point - * @param target - * @param targetPoint - * @param button - * @param modifiers - */ public void dnd(Point pointParam, final Wrap target, final Point targetPoint, final MouseButton button, final Modifier... modifiers) { final Point point = pointParam == null ? source.getClickPoint() : pointParam; source.getEnvironment().getExecutor().execute(target.getEnvironment(), false, new Action() { --- old/core/JemmyAWTInput/src/org/jemmy/input/KeyboardImpl.java 2018-05-20 10:11:39.630465963 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/KeyboardImpl.java 2018-05-20 10:11:39.571464162 +0200 @@ -46,10 +46,7 @@ Wrap target; RobotDriver robotDriver; private boolean detached; - /** - * Constructs a KeyRobotDriver object. - * @param target - */ + public KeyboardImpl(Wrap target) { //TODO: super(target.getEnvironment().getTimeout(RobotDriver.ROBOT_DELAY_TIMEOUT_NAME)); robotDriver = new RobotDriver(target.getEnvironment()); @@ -72,20 +69,10 @@ } } - /** - * - * @return Environment - */ public Environment getEnvironment() { return env; } - /** - * - * @param kbdButton - * @param modifiers - * @param pushTime - */ public void pushKey(final KeyboardButton kbdButton, final Modifier modifiers[], final Timeout pushTime) { runAction(new Action() { public void run(Object... parameters) { @@ -101,11 +88,6 @@ }); } - /** - * - * @param keyChar - * @param pushTime - */ @Override public void typeChar(char keyChar, Timeout pushTime) { pushKey(pushTime, map.getCharKey(keyChar), map.getCharModifiers(keyChar)); @@ -151,47 +133,28 @@ }); } - /** - * - * @param kbdButton - */ @Override public void pressKey(KeyboardButton kbdButton) { pressKey(kbdButton, new Modifier[]{}); } - /** - * - * @param kbdButton - */ + @Override public void releaseKey(KeyboardButton kbdButton) { releaseKey(kbdButton, new Modifier[]{}); } - /** - * - * @param kbdButton - * @param modifiers - */ + @Override public void pushKey(KeyboardButton kbdButton, Modifier... modifiers) { pushKey(kbdButton, modifiers, getEnvironment().getTimeout(PUSH.getName())); } - /** - * - * @param kbdButton - */ @Override public void pushKey(KeyboardButton kbdButton) { pushKey(kbdButton, new Modifier[]{}); } - /** - * - * @param keyChar - */ @Override public void typeChar(char keyChar) { typeChar(keyChar, getEnvironment().getTimeout(PUSH.getName())); --- old/core/JemmyAWTInput/src/org/jemmy/input/MouseImpl.java 2018-05-20 10:11:39.856472860 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/MouseImpl.java 2018-05-20 10:11:39.798471090 +0200 @@ -52,10 +52,6 @@ } } - /** - * - * @param target - */ public MouseImpl(Wrap target) { this.target = target; this.robotDriver = new RobotDriver(new Timeout("", 10)); @@ -74,28 +70,16 @@ } } - /** - * - */ @Override public void press() { press(MouseButtons.BUTTON1); } - /** - * - * @param button - */ @Override public void press(MouseButton button) { press(button, new Modifier[]{}); } - /** - * - * @param button - * @param modifiers - */ @Override public void press(final MouseButton button, final Modifier... modifiers) { runAction(new Action() { @@ -111,27 +95,15 @@ }); } - /** - * - */ public void release() { release(MouseButtons.BUTTON1); } - /** - * - * @param button - */ @Override public void release(MouseButton button) { release(button, new Modifier[]{}); } - /** - * - * @param button - * @param modifiers - */ @Override public void release(final MouseButton button, final Modifier... modifiers) { runAction(new Action() { @@ -147,17 +119,10 @@ }); } - /** - * - */ public void move() { move(target.getClickPoint()); } - /** - * - * @param p - */ public void move(final Point p) { runAction(new Action() { @@ -172,24 +137,16 @@ }); } - /** - * - */ public void click() { this.click(1); } - /** - * - * @param count - */ public void click(int count) { this.click(count, null); } /** - * - * @param count + * @param count todo document * @param p Point to click, if null {@linkplain Wrap#getClickPoint() * Wrap.getClickPoint()} method is invoked to get the point to click. */ @@ -199,10 +156,10 @@ /** * - * @param count + * @param count todo document * @param p Point to click, if null {@linkplain Wrap#getClickPoint() * Wrap.getClickPoint()} method is invoked to get the point to click. - * @param button + * @param button todo document */ @Override public void click(int count, Point p, MouseButton button) { @@ -211,11 +168,11 @@ /** * - * @param count + * @param count todo document * @param p Point to click, if null {@linkplain Wrap#getClickPoint() * Wrap.getClickPoint()} method is invoked to get the point to click. - * @param button - * @param modifiers + * @param button todo document + * @param modifiers todo document */ @Override public void click(final int count, final Point p, final MouseButton button, final Modifier... modifiers) { --- old/core/JemmyAWTInput/src/org/jemmy/input/RobotDriver.java 2018-05-20 10:11:40.084479818 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/RobotDriver.java 2018-05-20 10:11:40.026478048 +0200 @@ -44,7 +44,9 @@ /** - * @author Alexandre Iline(alexandre.iline@sun.com), KAM + * @author shura + * @author mrkam + * */ public class RobotDriver { @@ -58,7 +60,7 @@ new Timeout(ROBOT_DELAY_TIMEOUT_NAME, 10)); Environment.getEnvironment().setPropertyIfNotSet( AWTRobotInputFactory.ROBOT_MOUSE_SMOOTHNESS_PROPERTY, - new Integer(Integer.MAX_VALUE).toString()); + Integer.toString(Integer.MAX_VALUE)); smoothness = Integer.parseInt( (String)Environment.getEnvironment().getProperty( AWTRobotInputFactory.ROBOT_MOUSE_SMOOTHNESS_PROPERTY)); @@ -208,9 +210,9 @@ /** * @deprecated Implementation doesn't seem to be correct as it ignores mouseButton and modifiers - * @param point + * @param point todo document * @param mouseButton One of MouseEvent.BUTTON*_MASK - * @param modifiers + * @param modifiers todo document */ public void dragMouse(Point point, int mouseButton, int modifiers) { moveMouse(point); @@ -263,9 +265,9 @@ /** * Turns the wheel. - * @param p + * @param p todo document * @param amount Either positive or negative - * @param modifiers + * @param modifiers a combination of InputEvent.*_MASK fields. */ public void turnWheel(Point p, int amount, Modifier... modifiers) { pressModifiers(modifiers); --- old/core/JemmyAWTInput/src/org/jemmy/input/Version.java 2018-05-20 10:11:40.307486623 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/Version.java 2018-05-20 10:11:40.250484884 +0200 @@ -32,16 +32,10 @@ public static final Version VERSION = new Version(); - /** - * - */ public Version() { super(Version.class.getPackage().getName()); } - /** - * - * @param args - */ + public static void main(String[] args) { System.out.println("JemmyAWTInput version: " + VERSION.getVersion()); } --- old/core/JemmyAWTInput/src/org/jemmy/input/jemmy.properties 2018-05-20 10:11:40.531493459 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/input/jemmy.properties 2018-05-20 10:11:40.473491689 +0200 @@ -1,4 +1,4 @@ -version.major=0 -version.minor=9 -version.mini=5 +version.major=1 +version.minor=0 +version.mini=0 build=${buildnumber} --- old/core/JemmyAWTInput/src/org/jemmy/operators/AWTScreen.java 2018-05-20 10:11:40.769500722 +0200 +++ new/core/JemmyAWTInput/src/org/jemmy/operators/AWTScreen.java 2018-05-20 10:11:40.709498891 +0200 @@ -30,22 +30,14 @@ import org.jemmy.env.Environment; /** - * * @author shura */ public class AWTScreen extends Screen { - /** - * - */ public AWTScreen() { this(Environment.getEnvironment()); } - /** - * - * @param env - */ public AWTScreen(Environment env) { super(env); } --- old/core/JemmyBrowser/pom.xml 2018-05-20 10:11:40.997507680 +0200 +++ new/core/JemmyBrowser/pom.xml 2018-05-20 10:11:40.940505941 +0200 @@ -26,12 +26,12 @@ --> 4.0.0 - org.jemmy.v3 - Jemmy - 1.0 + org.adoptopenjdk + jemmy + ${revision} ../../pom.xml - JemmyBrowser + jemmy-browser jar org/jemmy/browser/jemmy.properties @@ -39,9 +39,9 @@ - org.jemmy.v3 - JemmyCore - 1.0 + org.adoptopenjdk + jemmy-core + ${revision} --- old/core/JemmyBrowser/src/org/jemmy/browser/SearchDialog.java 2018-05-20 10:11:41.239515065 +0200 +++ new/core/JemmyBrowser/src/org/jemmy/browser/SearchDialog.java 2018-05-20 10:11:41.177513173 +0200 @@ -34,7 +34,6 @@ public class SearchDialog extends javax.swing.JDialog { private boolean doSearch; - /** Creates new form SearchDialog */ public SearchDialog(java.awt.Frame parent) { super(parent, true); initComponents(); --- old/core/JemmyBrowser/src/org/jemmy/browser/jemmy.properties 2018-05-20 10:11:41.465521962 +0200 +++ new/core/JemmyBrowser/src/org/jemmy/browser/jemmy.properties 2018-05-20 10:11:41.407520192 +0200 @@ -1,4 +1,4 @@ -version.major=0 -version.minor=9 -version.mini=5 +version.major=1 +version.minor=0 +version.mini=0 build=${buildnumber} --- old/core/JemmyCore/pom.xml 2018-05-20 10:11:41.700529134 +0200 +++ new/core/JemmyCore/pom.xml 2018-05-20 10:11:41.641527333 +0200 @@ -26,11 +26,11 @@ questions. --> - org.jemmy.v3 - Jemmy - 1.0 + org.adoptopenjdk + jemmy + ${revision} ../../pom.xml - JemmyCore + jemmy-core jar --- old/core/JemmyCore/src/org/jemmy/Dimension.java 2018-05-20 10:11:41.930536153 +0200 +++ new/core/JemmyCore/src/org/jemmy/Dimension.java 2018-05-20 10:11:41.872534383 +0200 @@ -30,7 +30,7 @@ /** * Replacement for java.awt.Dimension - * @author Alexander Kouznetsov + * @author mrkam */ public class Dimension implements Serializable { @@ -104,7 +104,6 @@ /** * {@inheritDoc} - * @return */ public double getWidth() { return width; @@ -112,7 +111,6 @@ /** * {@inheritDoc} - * @return */ public double getHeight() { return height; @@ -164,11 +162,6 @@ this.height = height; } - /** - * Checks whether two dimension objects have equal values. - * @param obj - * @return - */ @Override public boolean equals(Object obj) { if (obj instanceof Dimension) { --- old/core/JemmyCore/src/org/jemmy/Point.java 2018-05-20 10:11:42.161543203 +0200 +++ new/core/JemmyCore/src/org/jemmy/Point.java 2018-05-20 10:11:42.103541433 +0200 @@ -31,7 +31,7 @@ /** * Replacement for java.awt.Point - * @author Alexander Kouznetsov + * @author mrkam */ public class Point implements Serializable{ /** @@ -101,7 +101,6 @@ /** * {@inheritDoc} - * @return */ public int getX() { return x; @@ -109,7 +108,6 @@ /** * {@inheritDoc} - * @return */ public int getY() { return y; @@ -128,7 +126,7 @@ /** * Sets the location of the point to the specified location. * @param p a point, the new location for this point - * @return + * @return self * @see org.jemmy.Point#getLocation */ public Point setLocation(Point p) { @@ -142,7 +140,7 @@ * Its behavior is identical with move(int, int). * @param x the X coordinate of the new location * @param y the Y coordinate of the new location - * @return self + * @return self * @see org.jemmy.Point#getLocation * @see org.jemmy.Point#move(int, int) */ @@ -203,8 +201,9 @@ } /** + * Translates the current point according to the given vector. * - * @param v + * @param v vector by which to translate the point * @return self */ public Point translate(Vector v) { @@ -233,7 +232,6 @@ /** * {@inheritDoc} - * @return */ @Override public int hashCode() { --- old/core/JemmyCore/src/org/jemmy/Rectangle.java 2018-05-20 10:11:42.378549825 +0200 +++ new/core/JemmyCore/src/org/jemmy/Rectangle.java 2018-05-20 10:11:42.327548269 +0200 @@ -31,7 +31,7 @@ /** * Replacement for java.awt.Rectangle - * @author Alexander Kouznetsov + * @author mrkam */ public class Rectangle implements Serializable { @@ -898,7 +898,6 @@ /** * {@inheritDoc} - * @return */ public boolean isEmpty() { return (width <= 0) || (height <= 0); @@ -930,7 +929,6 @@ /** * {@inheritDoc } - * @return */ @Override public int hashCode() { --- old/core/JemmyCore/src/org/jemmy/Vector.java 2018-05-20 10:11:42.603556691 +0200 +++ new/core/JemmyCore/src/org/jemmy/Vector.java 2018-05-20 10:11:42.545554921 +0200 @@ -34,55 +34,28 @@ private double x; private double y; - /** - * - * @param x - * @param y - */ public Vector(double x, double y) { this.x = x; this.y = y; } - /** - * - * @param from - * @param to - */ public Vector(Point from, Point to) { x = to.x - from.x; y = to.y - from.y; } - /** - * - * @return - */ public double getX() { return x; } - /** - * - * @return - */ public double getY() { return y; } - /** - * - * @return - */ public double lenght() { return Math.sqrt(x*x + y*y); } - /** - * - * @param newLenght - * @return self - */ public Vector setLenght(double newLenght) { double lenght = lenght(); x = x * newLenght / lenght; @@ -90,10 +63,6 @@ return this; } - /** - * @param multiplier - * @return self - */ public Vector multiply(double multiplier) { x*=multiplier; y*=multiplier; @@ -101,8 +70,7 @@ } /** - * - * @return a clone + * {@inheritDoc} */ @Override public Vector clone() { @@ -110,24 +78,17 @@ } /** - * - * @return + * {@inheritDoc} */ @Override public String toString() { return "(" + x + "," + y + ")"; } - /** - * Adds another vector (x1 + x2, y1 + y2) - * @param v - * @return self - */ public Vector add(Vector v) { x+=v.x; y+=v.y; return this; } - } --- old/core/JemmyCore/src/org/jemmy/Version.java 2018-05-20 10:11:42.821563344 +0200 +++ new/core/JemmyCore/src/org/jemmy/Version.java 2018-05-20 10:11:42.770561788 +0200 @@ -35,9 +35,6 @@ * @author shura */ public class Version { - /** - * - */ public static final Version VERSION = new Version(); private int major; @@ -45,17 +42,10 @@ private int mini; private String build; - /** - * - */ public Version() { this(Version.class.getPackage().getName()); } - /** - * - * @param pkg - */ public Version(String pkg) { try { Properties props = new Properties(); @@ -74,51 +64,27 @@ } } - /** - * - * @return - */ public int getMajor() { return major; } - /** - * - * @return - */ + public int getMini() { return mini; } - /** - * - * @return - */ public int getMinor() { return minor; } - /** - * - * @return - */ public String getVersion() { return major + "." + minor + "." + mini; } - /** - * - * @return - */ public String getBuild() { return build; } - /** - * - * @param old - * @return - */ public boolean newer(String old) { StringTokenizer tn = new StringTokenizer(old, "."); if(major >= Integer.parseInt(tn.nextToken())) { @@ -131,10 +97,6 @@ return false; } - /** - * - * @param args - */ public static void main(String[] args) { System.out.println("JemmyCore version: " + VERSION.getVersion() + "." + VERSION.build); } --- old/core/JemmyCore/src/org/jemmy/action/AbstractExecutor.java 2018-05-20 10:11:43.042570089 +0200 +++ new/core/JemmyCore/src/org/jemmy/action/AbstractExecutor.java 2018-05-20 10:11:42.983568288 +0200 @@ -59,9 +59,6 @@ public static final String QUEUE_ACTION_OUTPUT = "org.jemmy.action.AbstractExecutor.QUEUE_ACTION_OUTPUT"; private ActionQueue queue; - /** - * - */ public AbstractExecutor() { queue = new ActionQueue(); } @@ -72,21 +69,18 @@ Environment.getEnvironment().initOutput(NON_QUEUE_ACTION_OUTPUT, TestOut.getNullOutput()); } - /** - * - * @return - */ protected int actionsInQueue() { return queue.actionsInQueue(); } /** * {@inheritDoc } + * * Prints out what action is executed into output * specified by either NON_QUEUE_ACTION_OUTPUT or QUEUE_ACTION_OUTPUT * depending whether the action is called on queue or not. No output provided for * nested actions - only the top level ones are printed. - * @see TestOut#getOutput(java.lang.String) + * @see TestOut#getOutput() */ public final void execute(Environment env, boolean dispatch, final Action action, Object... parameters) { printStrace(env, "Action: ", action); @@ -104,11 +98,12 @@ /** * {@inheritDoc } + * * Prints out what action is executed into output * specified by either NON_QUEUE_ACTION_OUTPUT or QUEUE_ACTION_OUTPUT * depending whether the action is called on queue or not. No output provided for * nested actions - only the top level ones are printed. - * @see TestOut#getOutput(java.lang.String) + * @see TestOut#getOutput() */ public final void executeDetached(Environment env, boolean dispatch, final Action action, final Object... parameters) { printStrace(env, "Action detached: ", action); --- old/core/JemmyCore/src/org/jemmy/action/Action.java 2018-05-20 10:11:43.293577749 +0200 +++ new/core/JemmyCore/src/org/jemmy/action/Action.java 2018-05-20 10:11:43.237576040 +0200 @@ -28,7 +28,6 @@ /** - * * @author shura, KAM */ public abstract class Action { @@ -41,8 +40,7 @@ * Executes {@linkplain #run(java.lang.Object[]) run()} method of this * Action, saving the duration of its execution and storing any * RuntimeException and Error which may occur during its work. - * @param parameters Parameters to pass to {@linkplain - * #run(java.lang.Object[]) run()} method + * @param parameters Parameters to pass to {@linkplain #run(java.lang.Object[]) run()} method * @see #getThrowable() * @see #failed() */ @@ -64,24 +62,16 @@ } } - /** - * - * @return - */ public long getEndTime() { return endTime; } - /** - * - * @return - */ public long getStartTime() { return startTime; } /** - * Should be used from {@linkplain #run(java.lang.Object[]) run()) method + * Should be used from {@linkplain #run(java.lang.Object[]) run()} method * to check whether execution time is withing allowed time * @return true if difference between current time and start time is less * then allowed time; false otherwice @@ -90,39 +80,20 @@ return System.currentTimeMillis() - startTime < allowedTime; } - /** - * - * @return - */ public long getAllowedTime() { return allowedTime; } - /** - * - * @param allowedTime - */ public void setAllowedTime(long allowedTime) { this.allowedTime = allowedTime; } - /** - * - * @param parameters - */ public abstract void run(Object... parameters) throws Exception; - /** - * - * @return - */ public boolean isInterrupted() { return interrupted; } - /** - * - */ public void interrupt() { this.interrupted = true; } --- old/core/JemmyCore/src/org/jemmy/action/ActionExecutor.java 2018-05-20 10:11:43.504584188 +0200 +++ new/core/JemmyCore/src/org/jemmy/action/ActionExecutor.java 2018-05-20 10:11:43.445582387 +0200 @@ -36,9 +36,6 @@ */ public interface ActionExecutor { - /** - * - */ public static final String ACTION_EXECUTOR_PROPERTY = "action.executor"; /** @@ -69,7 +66,6 @@ /** * Checks whether the current thread is already performing an action. * @return true if the current thread is already performing an action. - * @see AbstractExecutor#isDispatchThread() */ public boolean isInAction(); --- old/core/JemmyCore/src/org/jemmy/action/GetAction.java 2018-05-20 10:11:43.724590902 +0200 +++ new/core/JemmyCore/src/org/jemmy/action/GetAction.java 2018-05-20 10:11:43.666589132 +0200 @@ -30,7 +30,7 @@ /** * An action to get some value. - * @param + * * @author shura */ public abstract class GetAction extends Action { @@ -38,32 +38,17 @@ private boolean finished = false; private T result = null; - /** - * - */ public GetAction() { } - /** - * - * @return - */ public boolean isFinished() { return finished; } - /** - * - * @return - */ public T getResult() { return result; } - /** - * - * @param result - */ protected void setResult(T result) { this.result = result; finished = true; --- old/core/JemmyCore/src/org/jemmy/control/AbstractWrapper.java 2018-05-20 10:11:43.941597524 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/AbstractWrapper.java 2018-05-20 10:11:43.889595937 +0200 @@ -39,32 +39,16 @@ private Environment env; - /** - * - * @param env - */ - @SuppressWarnings("unchecked") public AbstractWrapper(Environment env) { this.env = env; } - /** - * - * @return - */ public Environment getEnvironment() { return env; } protected abstract Class getWrapClass(Class controlClass); - /** - * - * @param - * @param controlClass - * @param control - * @return Wrap - */ public Wrap wrap(Class controlClass, T control) { Class cls = control.getClass(); Class wrp; @@ -91,20 +75,6 @@ throw new WrapperException(control); } - /** - * - * @param - * @param control - * @param controlClass - * @param wrapperClass - * @return Wrap - * @throws java.lang.NoSuchMethodException - * @throws java.lang.InstantiationException - * @throws java.lang.IllegalAccessException - * @throws java.lang.IllegalArgumentException - * @throws java.lang.reflect.InvocationTargetException - */ - @SuppressWarnings("unchecked") protected Wrap doWrap(T control, Class controlClass, Class wrapperClass) throws NoSuchMethodException, InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException { Constructor cns = null; Class cls = controlClass; --- old/core/JemmyCore/src/org/jemmy/control/As.java 2018-05-20 10:11:44.156604085 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/As.java 2018-05-20 10:11:44.101602407 +0200 @@ -45,7 +45,7 @@ * This should point out what is the encapsulated type for * TypeControlInterface * @see TypeControlInterface - * @return + * @return todo document */ Class value() default Void.class; } --- old/core/JemmyCore/src/org/jemmy/control/ControlInterfaces.java 2018-05-20 10:11:44.373610708 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/ControlInterfaces.java 2018-05-20 10:11:44.317608999 +0200 @@ -48,6 +48,7 @@ /** * List of interfaces supported by this wrap. * @see ControlInterface + * @return todo document */ Class[] value(); /** @@ -57,11 +58,13 @@ * provide anything for a ControlInterface which is not a * TypeControlInterface * @see TypeControlInterface + * @return todo document */ Class[] encapsulates() default {}; /** * This provides names for the dock methods which would be generated. If the array * does not have enough elements, the method would be named as "as" + value[i].getName(). + * @return todo document */ String[] name() default {}; } --- old/core/JemmyCore/src/org/jemmy/control/ControlType.java 2018-05-20 10:11:44.582617086 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/ControlType.java 2018-05-20 10:11:44.527615407 +0200 @@ -35,9 +35,6 @@ @Target(ElementType.TYPE) @Documented public @interface ControlType { - /** - * - * @return - */ + Class[] value(); } --- old/core/JemmyCore/src/org/jemmy/control/DefaultWrapper.java 2018-05-20 10:11:44.791623464 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/DefaultWrapper.java 2018-05-20 10:11:44.738621846 +0200 @@ -27,36 +27,19 @@ import org.jemmy.env.Environment; /** - * * @author shura */ public class DefaultWrapper extends WrapperImpl { - /** - * - * @param env - * @param wrapList - */ - @SuppressWarnings("unchecked") public DefaultWrapper(Environment env, Class... wrapList) { super(env); addAnnotated(wrapList); } - /** - * - * @param env - */ - @SuppressWarnings("unchecked") public DefaultWrapper(Environment env) { super(env); } - /** - * - * @param list - */ - @SuppressWarnings("unchecked") public final void addAnnotated(Class... list) { for (Class cn : list) { if (!cn.isAnnotationPresent(ControlType.class)) { --- old/core/JemmyCore/src/org/jemmy/control/FieldProperties.java 2018-05-20 10:11:45.015630300 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/FieldProperties.java 2018-05-20 10:11:44.955628469 +0200 @@ -44,16 +44,16 @@ @Documented public @interface FieldProperties { /** - * List of field names. + * @return list of field names. */ String[] value(); /** - * List of field types. In case this list is shorter, all the unmatched ones + * @return list of field types. In case this list is shorter, all the unmatched ones * from value are considered to be of type Object. */ Class[] types() default {}; /** - * Are the properties worth a waiter. + * @return are the properties worth a waiter. */ boolean[] waitable() default {}; } --- old/core/JemmyCore/src/org/jemmy/control/LazyWrapper.java 2018-05-20 10:11:45.224636678 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/LazyWrapper.java 2018-05-20 10:11:45.170635030 +0200 @@ -40,22 +40,13 @@ private final HashMap theWrappers = new HashMap(); private final ClassLoader loader; - /** - * - * @param env - */ - @SuppressWarnings("unchecked") + public LazyWrapper(ClassLoader loader, Environment env) { super(env); this.loader = loader; } - /** - * - * @param

    - * @param controlClass - * @param wrapperClass - */ + public

    void add(String controlClass, String wrapperClass) { theWrappers.put(controlClass, wrapperClass); } --- old/core/JemmyCore/src/org/jemmy/control/MethodProperties.java 2018-05-20 10:11:45.446643453 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/MethodProperties.java 2018-05-20 10:11:45.387641652 +0200 @@ -45,16 +45,16 @@ @Documented public @interface MethodProperties { /** - * List of method names + * @return list of method names */ String[] value(); /** - * List of field types. In case this list is shorter, all the unmatched ones + * @return list of field types. In case this list is shorter, all the unmatched ones * from value are considered to be of type Object. */ Class[] types() default {}; /** - * Are the properties worth a waiter. + * @return are the properties worth a waiter. */ boolean[] waitable() default {}; } --- old/core/JemmyCore/src/org/jemmy/control/Property.java 2018-05-20 10:11:45.660649984 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/Property.java 2018-05-20 10:11:45.604648275 +0200 @@ -49,7 +49,7 @@ */ String value(); /** - * Are the properties worth a waiter. + * @return are the properties worth a waiter. */ boolean waitable() default false; } --- old/core/JemmyCore/src/org/jemmy/control/ScreenArea.java 2018-05-20 10:11:45.882656759 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/ScreenArea.java 2018-05-20 10:11:45.826655050 +0200 @@ -34,9 +34,6 @@ * @author shura */ public interface ScreenArea { - /** - * - * @return - */ + public abstract Rectangle getScreenBounds(); } --- old/core/JemmyCore/src/org/jemmy/control/SelectorImpl.java 2018-05-20 10:11:46.100663411 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/SelectorImpl.java 2018-05-20 10:11:46.045661733 +0200 @@ -31,9 +31,6 @@ import org.jemmy.timing.State; /** - * - * @param - * @param * @author shura */ public class SelectorImpl implements Selector { @@ -41,21 +38,11 @@ Wrap target; Selectable selectable; - /** - * - * @param target - * @param selectable - */ public SelectorImpl(Wrap target, Selectable selectable) { this.target = target; this.selectable = selectable; } - /** - * - * @param state - */ - @SuppressWarnings("unchecked") public void select(final STATE state) { if (target.is(Showable.class)) { target.as(Showable.class).shower().show(); --- old/core/JemmyCore/src/org/jemmy/control/Wrap.java 2018-05-20 10:11:46.319670095 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/Wrap.java 2018-05-20 10:11:46.259668264 +0200 @@ -53,65 +53,21 @@ @ControlInterfaces({Mouse.class, Keyboard.class, Drag.class}) public abstract class Wrap { - /** - * - */ public static final String BOUNDS_PROP_NAME = "bounds"; - /** - * - */ public static final String CLICKPOINT_PROP_NAME = "clickPoint"; - /** - * - */ public static final String CONTROL_CLASS_PROP_NAME = "control.class"; - /** - * - */ public static final String CONTROL_PROP_NAME = "control"; - /** - * - */ public static final String INPUT_FACTORY_PROPERTY = "input.control.interface.factory"; - /** - * - */ public static final String IMAGE_LOADER_PROPERTY = "image.loader"; - /** - * - */ public static final String IMAGE_CAPTURER_PROPERTY = "image.capturer"; - /** - * - */ public static final String TEXT_PROP_NAME = "text"; - /** - * - */ public static final String POSITION_PROP_NAME = "position"; - /** - * - */ public static final String VALUE_PROP_NAME = "value"; - /** - * - */ public static final String WRAPPER_CLASS_PROP_NAME = "wrapper.class"; - /** - * - */ public static final String TOOLTIP_PROP_NAME = "tooltip"; - /** - * - */ public static final String NAME_PROP_NAME = "name"; - /** - * - */ public static final Timeout WAIT_STATE_TIMEOUT = new Timeout("wait.state", 1000); - /** - * - */ + public static final String OUTPUT = Wrap.class.getName() + ".OUTPUT"; private static DefaultWrapper theWrapper = new DefaultWrapper(Environment.getEnvironment()); @@ -125,10 +81,6 @@ Environment.getEnvironment().initTimeout(Keyboard.PUSH); } - /** - * - * @return - */ public static DefaultWrapper getWrapper() { return theWrapper; } @@ -184,7 +136,7 @@ * Return default point to click, drag. This implementation returns the * center must be overriden if something different is desired. * - * @return + * @return the default click point */ @Property(CLICKPOINT_PROP_NAME) public Point getClickPoint() { @@ -207,8 +159,8 @@ * Transforms point in local control coordinate system to screen * coordinates. * - * @param local - * @return + * @param local the local coordinate + * @return a absolute translated point * @see #toLocal(org.jemmy.Point) */ public Point toAbsolute(Point local) { @@ -220,7 +172,7 @@ * Transforms point in screen coordinates to local control coordinate * system. * - * @param local + * @param local the local coordinate * @return coordinates which should be used for mouse operations. * @see #toAbsolute(org.jemmy.Point) */ @@ -258,7 +210,7 @@ * Waits for a portion of image to be exact the same as the parameter. * * @see Wrap#as(java.lang.Class) - * @param golden + * @param golden the image to match against * @param rect A portion of control to compare. * @param resID ID of a result image to save in case of failure. No image * saved if null. @@ -296,7 +248,7 @@ * Waits for image to be exact the same as the parameter. * * @see Wrap#as(java.lang.Class) - * @param golden + * @param golden the image to match against * @param resID ID of a result image to save in case of failure. No image * saved if null. * @param diffID ID of a diff image to save in case of failure. No image @@ -310,9 +262,9 @@ /** * TODO javadoc * - * @param - * @param state - * @param value + * @param the states type + * @param state the state + * @param value the state value * @return last returned State value * @throws TimeoutExpiredException in case the wait is unsuccessful. */ @@ -321,10 +273,8 @@ } /** - * TODO javadoc - * - * @param - * @param state + * @param the states type + * @param state the state * @return last returned State value * @throws TimeoutExpiredException in case the wait is unsuccessful. */ @@ -370,9 +320,10 @@ * as(java.lang.Class) * * @see Wrap#is(java.lang.Class) - * @param - * @param interfaceClass - * @return + * @param the control interface + * @param interfaceClass the interface class + * @return true if the control is an ControlInterface, + * false otherwise */ public boolean is(Class interfaceClass) { if (interfaceClass.isInstance(this)) { @@ -393,11 +344,12 @@ * as(java.lang.Class) * * @see Wrap#is(java.lang.Class) - * @param - * @param - * @param interfaceClass + * @param the type interface + * @param the control interface + * @param interfaceClass the interface class * @param type The parameter class. - * @return + * @return true if the control is an ControlInterface, + * false otherwise */ public > boolean is(Class interfaceClass, Class type) { if (interfaceClass.isInstance(this)) { @@ -435,9 +387,9 @@ * it checks * * @see Wrap#is(java.lang.Class) - * @param - * @param interfaceClass - * @return + * @param the control interface + * @param interfaceClass the interface class + * @return the control interface instance */ public INTERFACE as(Class interfaceClass) { if (interfaceClass.isInstance(this)) { @@ -456,11 +408,11 @@ * Returns an implementation of interface associated with the object. * * @see Wrap#is(java.lang.Class) - * @param - * @param - * @param interfaceClass + * @param the type interface + * @param the control interface + * @param interfaceClass the interface class * @param type The parameter class. - * @return + * @return the type control instance */ public > INTERFACE as(Class interfaceClass, Class type) { if (interfaceClass.isInstance(this)) { @@ -490,10 +442,9 @@ private Keyboard keyboard = null; /** - * A shortcut to - * as(MouseTarget.class).mouse() + * A shortcut to as(MouseTarget.class).mouse() * - * @return + * @return the mouse target */ @As(Mouse.class) public Mouse mouse() { @@ -504,10 +455,9 @@ } /** - * A shortcut to - * as(MouseTarget.class).drag() + * A shortcut to as(MouseTarget.class).drag() * - * @return + * @return the drag target */ @As(Drag.class) public Drag drag() { @@ -518,10 +468,9 @@ } /** - * A shortcut to - * as(KeyTarget.class).wrap() + * A shortcut to as(KeyTarget.class).wrap() * - * @return + * @return the wrap target */ @As(Keyboard.class) public Keyboard keyboard() { @@ -541,10 +490,6 @@ */ private HashMap properties = new HashMap(); - /** - * - * @return - */ @Property(CONTROL_CLASS_PROP_NAME) public Class getControlClass() { return getControl().getClass(); @@ -667,13 +612,12 @@ } /** - * Get property of the wrapped object. Uses first available from - *

  • methods annotated by - * org.jemmy.control.Property
  • wrapped object methods - * listed in - * org.jemmy.control.MethodProperties
  • wrapped object - * fields listed in - * org.jemmy.control.FieldProperties
  • + * Get property of the wrapped object. Uses first available from + *
      + *
    1. methods annotated by org.jemmy.control.Property
    2. + *
    3. wrapped object methods listed in org.jemmy.control.MethodProperties
    4. + *
    5. wrapped object fields listed in org.jemmy.control.FieldProperties
    6. + *
    * * @param name property name * @throws JemmyException if no property found @@ -711,10 +655,10 @@ * Get property out of the control interface. Refer to the interface doc to * find out what properties are provided. * - * @param - * @param name - * @param intrfc - * @return + * @param the control interface + * @param name the property name + * @param intrfc the interface class + * @return the control property */ public Object getProperty(String name, Class intrfc) { return getInterfaceProperty(intrfc, as(intrfc), name); @@ -724,12 +668,12 @@ * Get property out of the control interface. Refer to the interface doc to * find out what properties are provided. * - * @param - * @param - * @param name - * @param intrfc - * @param type - * @return + * @param the type interface + * @param the control interface + * @param name the property name + * @param intrfc the interface class + * @param type The parameter class. + * @return the control property */ public > Object getProperty(String name, Class intrfc, Class type) { return getInterfaceProperty(intrfc, as(intrfc, type), name); @@ -762,10 +706,10 @@ * property of control interface to get the specified value. * WAIT_STATE_TIMOUT timeout is used * - * @param - * @param property - * @param intrfc - * @param value + * @param the control interface + * @param property the property name + * @param intrfc the interface class + * @param value the new parameter value */ public void waitProperty(final String property, final Class intrfc, final Object value) { Object instance = as(intrfc); @@ -787,12 +731,12 @@ * property of control interface to get the specified value. * WAIT_STATE_TIMOUT timeout is used * - * @param - * @param - * @param property - * @param intrfc - * @param type - * @param value + * @param the type interface + * @param the control interface + * @param property the property name + * @param intrfc the interface class + * @param type the parameter class. + * @param value the new parameter value */ public > void waitProperty(final String property, final Class intrfc, final Class type, final Object value) { getEnvironment().getWaiter(WAIT_STATE_TIMEOUT).ensureValue(value, new State() { @@ -808,11 +752,6 @@ }); } - /** - * - * @param name - * @return - */ public boolean hasFieldProperty(String name) { Class cls = getClass(); do { @@ -826,11 +765,6 @@ return false; } - /** - * - * @param name - * @return - */ public boolean hasMethodProperty(String name) { Class cls = getClass(); do { @@ -854,11 +788,6 @@ return false; } - /** - * - * @param name - * @return - */ public Object getFieldProperty(final String name) { if (!hasFieldProperty(name)) { throw new JemmyException("No \"" + name + "\" field property specified on " + getClass().getName()); @@ -877,11 +806,6 @@ return result; } - /** - * - * @param name - * @return - */ public Object getMethodProperty(final String name) { if (!hasMethodProperty(name)) { throw new JemmyException("No \"" + name + "\" method property specified on " + getClass().getName()); @@ -905,13 +829,6 @@ return result; } - /** - * - * @param

    - * @param valueClass - * @param name - * @return - */ public

    P getProperty(Class

    valueClass, String name) { return valueClass.cast(getProperty(name)); } @@ -924,7 +841,7 @@ * FieldProperties correspondingly. * * @return a map of properties - * @throws Runtime exception should there be an exception thrown while + * @throws RuntimeException should there be an exception thrown while * getting a property */ public HashMap getProperties() { --- old/core/JemmyCore/src/org/jemmy/control/Wrapper.java 2018-05-20 10:11:46.549677114 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/Wrapper.java 2018-05-20 10:11:46.488675252 +0200 @@ -30,12 +30,6 @@ * @author shura */ public interface Wrapper { - /** - * - * @param - * @param controlClass - * @param control - * @return Wrap - */ + public Wrap wrap(Class controlClass, T control); } --- old/core/JemmyCore/src/org/jemmy/control/WrapperException.java 2018-05-20 10:11:46.756683431 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/WrapperException.java 2018-05-20 10:11:46.702681783 +0200 @@ -33,32 +33,14 @@ */ public class WrapperException extends JemmyException { - /** - * - * @param - * @param controlClass - * @param wrapperClass - * @param e - */ public WrapperException(Class controlClass, Class wrapperClass, Exception e) { super(controlClass.getName() + " is not accepted by " + wrapperClass.getName(), e); } - /** - * - * @param - * @param controlClass - * @param wrapperClass - */ public WrapperException(Class controlClass, Class wrapperClass) { super(controlClass.getName() + " is not accepted by " + wrapperClass.getName()); } - /** - * - * @param - * @param control - */ public WrapperException(Object control) { super("Unable to find a wrapper", control); } --- old/core/JemmyCore/src/org/jemmy/control/WrapperImpl.java 2018-05-20 10:11:46.975690114 +0200 +++ new/core/JemmyCore/src/org/jemmy/control/WrapperImpl.java 2018-05-20 10:11:46.917688344 +0200 @@ -38,22 +38,11 @@ private HashMap> theWrappers; - /** - * - * @param env - */ - @SuppressWarnings("unchecked") public WrapperImpl(Environment env) { super(env); theWrappers = new HashMap>(); } - /** - * - * @param

    - * @param controlClass - * @param wrapperClass - */ public

    void add(Class controlClass, Class wrapperClass) { theWrappers.put(controlClass, wrapperClass); // TODO: Improve output --- old/core/JemmyCore/src/org/jemmy/dock/DefaultParent.java 2018-05-20 10:11:47.190696676 +0200 +++ new/core/JemmyCore/src/org/jemmy/dock/DefaultParent.java 2018-05-20 10:11:47.138695089 +0200 @@ -42,7 +42,7 @@ @Documented public @interface DefaultParent { /** - * Description of a parent represented by the annotated method. + * @return description of a parent represented by the annotated method. */ String value(); } --- old/core/JemmyCore/src/org/jemmy/dock/DefaultWrapper.java 2018-05-20 10:11:47.407703298 +0200 +++ new/core/JemmyCore/src/org/jemmy/dock/DefaultWrapper.java 2018-05-20 10:11:47.349701528 +0200 @@ -34,7 +34,6 @@ * do not take parent as a parameter, taking whatever is coming from this method * instead. An annotated method should take one parameter - class of the controls * to by looked for and should return parent for that type. - * @see Parent * @author shura */ @Target(ElementType.METHOD) --- old/core/JemmyCore/src/org/jemmy/dock/Dock.java 2018-05-20 10:11:47.613709584 +0200 +++ new/core/JemmyCore/src/org/jemmy/dock/Dock.java 2018-05-20 10:11:47.556707845 +0200 @@ -65,12 +65,12 @@ /** * Method which at the end actually get called from all dock lookup * constructors. - * @param - * @param parent - * @param controlType - * @param index - * @param criteria - * @return + * @param todo document + * @param parent todo document + * @param controlType todo document + * @param index todo document + * @param criteria todo document + * @return todo document */ protected static Wrap lookup(Parent parent, Class controlType, int index, LookupCriteria... criteria) { Lookup lookup; @@ -94,7 +94,6 @@ } /** - * * @return Wrap instance obtainer through lookup */ public Object control() { @@ -102,28 +101,28 @@ } /** - * Shortcut to wrap().mouse() + * @return Shortcut to wrap().mouse() */ public Mouse mouse() { return wrap.mouse(); } /** - * Shortcut to wrap().keyboard() + * @return Shortcut to wrap().keyboard() */ public Keyboard keyboard() { return wrap.keyboard(); } /** - * Shortcut to wrap().drag() + * @return Shortcut to wrap().drag() */ public Drag drag() { return wrap.drag(); } /** - * Shortcut to wrap().getScreenBounds() + * @return Shortcut to wrap().getScreenBounds() */ public Rectangle bounds() { return wrap.getScreenBounds(); @@ -135,7 +134,6 @@ } /** - * * @return wrap().getEnvironment(). */ public Environment environment() { @@ -145,6 +143,10 @@ /** * Loads image with goldenId id waits for the control to match it. * @see Wrap#waitImage(org.jemmy.image.Image, org.jemmy.Rectangle, java.lang.String, java.lang.String) + * @param goldenId todo document + * @param rect todo document + * @param resID todo document + * @param diffID todo document */ public void waitImage(String goldenId, Rectangle rect, String resID, String diffID) { wrap.waitImage(environment().getImageLoader().load(goldenId), rect, resID, diffID); @@ -158,6 +160,8 @@ * @see #waitImage(java.lang.String, org.jemmy.Rectangle, java.lang.String, java.lang.String) * @see #DEFAULT_DIFF_IMAGE_SUFFIX * @see #DEFAULT_RESULT_IMAGE_SUFFIX + * @param goldenId todo document + * @param rect todo document */ public void waitImage(String goldenId, Rectangle rect) { waitImage(goldenId, @@ -169,6 +173,9 @@ /** * Loads image with goldenId id waits for the control to match it. * @see Wrap#waitImage(org.jemmy.image.Image, java.lang.String, java.lang.String) + * @param goldenId todo document + * @param resID todo document + * @param diffID todo document */ public void waitImage(String goldenId, String resID, String diffID) { wrap.waitImage(environment().getImageLoader().load(goldenId), resID, diffID); @@ -182,6 +189,7 @@ * @see #waitImage(java.lang.String, java.lang.String, java.lang.String) * @see #DEFAULT_DIFF_IMAGE_SUFFIX * @see #DEFAULT_RESULT_IMAGE_SUFFIX + * @param goldenId todo document */ public void waitImage(String goldenId) { waitImage(goldenId, --- old/core/JemmyCore/src/org/jemmy/dock/DockInfo.java 2018-05-20 10:11:47.831716237 +0200 +++ new/core/JemmyCore/src/org/jemmy/dock/DockInfo.java 2018-05-20 10:11:47.779714650 +0200 @@ -43,6 +43,7 @@ * Desired name of the dock class, should one be generated. * Usually empty ("", as nulls are not allowed) in which case the calculated value * is taken whatever logic annotation processor decides to use. + * @return todo document */ String name() default ""; @@ -50,19 +51,22 @@ * Should there be extra constructors which take another lookup criteria - a class * of a desired control? That class must be a subtype of the one wrapped by the wrap * class annotated with this annotation. + * @return todo document */ boolean generateSubtypeLookups() default false; /** * Should generated wrap() method return this class or - * Wrap and also should there be a constructor with + * Wrap<? extends ...> and also should there be a constructor with * one parameter - the wrap. + * @return todo document */ boolean anonymous() default false; /** * Should the lookup constructors have LookupCriteria<Type>... * parameter or the LookupCriteria<Type> parameter. + * @return todo document */ boolean multipleCriteria() default true; } --- old/core/JemmyCore/src/org/jemmy/dock/ObjectLookup.java 2018-05-20 10:11:48.046722799 +0200 +++ new/core/JemmyCore/src/org/jemmy/dock/ObjectLookup.java 2018-05-20 10:11:47.989721059 +0200 @@ -43,7 +43,7 @@ @Documented public @interface ObjectLookup { /** - * Description of the method parameters. To be used in the javadoc later. + * @return description of the method parameters. To be used in the javadoc later. */ String value(); } --- old/core/JemmyCore/src/org/jemmy/dock/Shortcut.java 2018-05-20 10:11:48.263729421 +0200 +++ new/core/JemmyCore/src/org/jemmy/dock/Shortcut.java 2018-05-20 10:11:48.203727590 +0200 @@ -39,7 +39,7 @@ @Retention(RetentionPolicy.CLASS) public @interface Shortcut { /** - * If name is not specified, then shortcut's name will same as name of interface method + * @return if name is not specified, then shortcut's name will same as name of interface method */ String name() default ""; } --- old/core/JemmyCore/src/org/jemmy/env/Environment.java 2018-05-20 10:11:48.482736104 +0200 +++ new/core/JemmyCore/src/org/jemmy/env/Environment.java 2018-05-20 10:11:48.423734304 +0200 @@ -43,14 +43,9 @@ import org.jemmy.timing.Waiter; /** - * * @author shura, mrkam, erikgreijus */ public class Environment { - - /** - * - */ public static final String JEMMY_PROPERTIES_FILE_PROPERTY = "jemmy.properties"; public static final String TIMEOUTS_FILE_PROPERTY = "timeouts"; /** @@ -59,10 +54,6 @@ public static final String OUTPUT = Environment.class.getName() + ".OUTPUT"; private final static Environment env = new Environment(null); - /** - * - * @return - */ public static Environment getEnvironment() { return env; } @@ -74,10 +65,6 @@ private HashMap environment = new HashMap(); private Environment parent; - /** - * - * @param parent - */ public Environment(Environment parent) { this.parent = parent; environment = new HashMap(); @@ -86,25 +73,14 @@ } } - /** - * - */ public Environment() { this(getEnvironment()); } - /** - * - * @return - */ public Environment getParentEnvironment() { return parent; } - /** - * - * @param parent - */ public void setParentEnvironment(Environment parent) { this.parent = parent; } @@ -151,11 +127,6 @@ } } - /** - * - * @param cls - * @return - */ public List get(Class cls) { Set all = environment.keySet(); ArrayList result = new ArrayList(); @@ -167,19 +138,10 @@ return result; } - /** - * - * @param defaultExecutor - * @return - */ public ActionExecutor setExecutor(ActionExecutor defaultExecutor) { return (ActionExecutor) setProperty(ActionExecutor.class, defaultExecutor); } - /** - * - * @return - */ public ActionExecutor getExecutor() { ActionExecutor res = (ActionExecutor) getProperty(ActionExecutor.class); if (res == null) { @@ -225,72 +187,46 @@ } /** - * - * @param - * @param cls + * @param todo document + * @param cls todo document * @param obj if null then property is removed - * @return + * @return todo document */ public T setProperty(Class cls, T obj) { return setProperty(cls, null, obj); } /** - * - * @param - * @param cls + * @param todo document + * @param cls todo document * @param obj if null then property is removed - * @return + * @return todo document */ public T setPropertyIfNotSet(Class cls, T obj) { return setPropertyIfNotSet(cls, null, obj); } - /** - * - * @param - * @param cls - * @return - */ public T getProperty(Class cls) { return getProperty(cls, null); } /** - * - * @param name + * @param name todo document * @param obj if null then property is removed - * @return + * @return todo document */ public Object setProperty(String name, Object obj) { return setProperty(Object.class, name, obj); } - /** - * - * @param name - * @param obj - * @return - */ public Object setPropertyIfNotSet(String name, Object obj) { return setPropertyIfNotSet(Object.class, name, obj); } - /** - * - * @param name - * @return - */ public Object getProperty(String name) { return getProperty(Object.class, name); } - /** - * - * @param name - * @param defaultValue - * @return - */ public Object getProperty(String name, Object defaultValue) { return getProperty(Environment.class, name, defaultValue); } @@ -322,19 +258,10 @@ return environment.get(key); } - /** - * - * @param out - * @return - */ public TestOut setOutput(TestOut out) { return (TestOut) setProperty(TestOut.class, out); } - /** - * - * @return - */ public TestOut getOutput() { return (TestOut) getProperty(TestOut.class); } @@ -344,9 +271,9 @@ * some specific outputs. Please consult javadoc for a class in question. * Use null to unset the property. * - * @param outputName - * @param out - * @return + * @param outputName todo document + * @param out todo document + * @return todo document */ public TestOut setOutput(String outputName, TestOut out) { return (TestOut) setProperty(TestOut.class, outputName, out); @@ -355,9 +282,9 @@ /** * Initializes some specific output only if it is not yet set. * - * @param outputName - * @param out - * @return + * @param outputName todo document + * @param out todo document + * @return todo document */ public TestOut initOutput(String outputName, TestOut out) { TestOut res = (TestOut) getProperty(TestOut.class, outputName); @@ -372,46 +299,26 @@ * Get's a specific output. If nothing assigned, returns * getOutput() * - * @param outputName - * @return + * @param outputName todo document + * @return todo document */ public TestOut getOutput(String outputName) { TestOut res = (TestOut) getProperty(TestOut.class, outputName); return (res != null) ? res : getOutput(); } - /** - * - * @param timeout - * @return - */ public Waiter getWaiter(Timeout timeout) { return getWaiter(timeout.getName()); } - /** - * - * @param timeoutName - * @return - */ public Waiter getWaiter(String timeoutName) { return new Waiter(getTimeout(timeoutName)); } - /** - * - * @param timeout - * @return - */ public Timeout getTimeout(Timeout timeout) { return getTimeout(timeout.getName()); } - /** - * - * @param name - * @return - */ public Timeout getTimeout(String name) { return (Timeout) getProperty(Timeout.class, name); } @@ -462,27 +369,14 @@ return setTimeout(new Timeout(name, value)); } - /** - * - * @return - */ public CharBindingMap getBindingMap() { return (CharBindingMap) getProperty(CharBindingMap.class); } - /** - * - * @param map - * @return - */ public CharBindingMap setBindingMap(CharBindingMap map) { return (CharBindingMap) setProperty(CharBindingMap.class, map); } - /** - * - * @return - */ public ImageLoader getImageLoader() { ImageLoader res = (ImageLoader) getProperty(ImageLoader.class); if (res == null) { @@ -504,10 +398,6 @@ return res; } - /** - * - * @return - */ public ImageCapturer getImageCapturer() { ImageCapturer res = (ImageCapturer) getProperty(ImageCapturer.class); if (res == null) { @@ -529,29 +419,15 @@ return res; } - /** - * - * @param imageLoader - * @return - */ public ImageLoader setImageLoader(ImageLoader imageLoader) { return (ImageLoader) setProperty(ImageLoader.class, imageLoader); } - /** - * - * @param imageCapturer - * @return - */ public ImageCapturer setImageCapturer(ImageCapturer imageCapturer) { getOutput(OUTPUT).println("ImageCapturer set to " + imageCapturer); return (ImageCapturer) setProperty(ImageCapturer.class, imageCapturer); } - /** - * - * @return - */ public ControlInterfaceFactory getInputFactory() { ControlInterfaceFactory res = (ControlInterfaceFactory) getProperty(ControlInterfaceFactory.class); if (res == null) { @@ -572,11 +448,6 @@ return res; } - /** - * - * @param factory - * @return - */ public ControlInterfaceFactory setInputFactory(ControlInterfaceFactory factory) { getOutput(OUTPUT).println("Input factory set to " + factory); return (ControlInterfaceFactory) setProperty(ControlInterfaceFactory.class, factory); --- old/core/JemmyCore/src/org/jemmy/env/TestOut.java 2018-05-20 10:11:48.714743184 +0200 +++ new/core/JemmyCore/src/org/jemmy/env/TestOut.java 2018-05-20 10:11:48.655741384 +0200 @@ -128,7 +128,7 @@ /** * Read one byte from input. * @return an int from input stream. - * @exception IOException + * @exception IOException todo document */ public int read() throws IOException { if (input != null) { @@ -141,7 +141,7 @@ /** * Read a line from input. * @return a line from input stream. - * @exception IOException + * @exception IOException todo document */ public String readln() throws IOException { if (buffInput != null) { --- old/core/JemmyCore/src/org/jemmy/env/Timeout.java 2018-05-20 10:11:48.933749868 +0200 +++ new/core/JemmyCore/src/org/jemmy/env/Timeout.java 2018-05-20 10:11:48.881748281 +0200 @@ -109,10 +109,6 @@ } } - /** - * - * @return - */ @Override public String toString() { return "Timeout [" + name + ", " + value + "]"; --- old/core/JemmyCore/src/org/jemmy/image/Image.java 2018-05-20 10:11:49.147756398 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/Image.java 2018-05-20 10:11:49.090754659 +0200 @@ -33,7 +33,7 @@ /** * Compares the image with the other one. * - * @param img + * @param img todo document * @return difference of the images, null if identical */ public Image compareTo(Image img); --- old/core/JemmyCore/src/org/jemmy/image/ImageCapturer.java 2018-05-20 10:11:49.361762929 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/ImageCapturer.java 2018-05-20 10:11:49.300761068 +0200 @@ -36,9 +36,9 @@ /** * Obtains a screen image for the control. - * @param control + * @param control todo document * @param rect a control area to be captured - * @return + * @return todo document */ Image capture(Wrap control, Rectangle rect); --- old/core/JemmyCore/src/org/jemmy/image/ImageComparator.java 2018-05-20 10:11:49.573769399 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/ImageComparator.java 2018-05-20 10:11:49.520767781 +0200 @@ -28,7 +28,7 @@ /** * Interface for all classes performing image comparison. * - * @author Alexandre Iline (alexandre.iline@sun.com) + * @author shura */ public interface ImageComparator { @@ -43,7 +43,7 @@ /** * A string qualifying an image comparison algorithm. To be used in logs and * tools. - * @return + * @return todo document */ public String getID(); } --- old/core/JemmyCore/src/org/jemmy/image/ImageLoader.java 2018-05-20 10:11:49.794776143 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/ImageLoader.java 2018-05-20 10:11:49.742774556 +0200 @@ -35,8 +35,8 @@ * Loads image by ID. Depending on an implementation, ID could be file name, * class name, image name, etc. * @see Image#save(java.lang.String) - * @param ID - * @return + * @param ID todo document + * @return todo document */ Image load(String ID); --- old/core/JemmyCore/src/org/jemmy/image/pixel/AverageDistanceComparator.java 2018-05-20 10:11:50.022783101 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/AverageDistanceComparator.java 2018-05-20 10:11:49.960781209 +0200 @@ -32,21 +32,11 @@ */ public class AverageDistanceComparator extends ThresholdComparator { - /** - * - * @param threshold - */ public AverageDistanceComparator(double threshold) { super(0, Math.sqrt(3)); setThreshold(threshold); } - /** - * - * @param image1 - * @param image2 - * @return - */ public boolean compare(Raster image1, Raster image2) { Dimension size = PixelImageComparator.computeDiffSize(image1, image2); if (size == null) { @@ -69,6 +59,7 @@ public static final Raster.Component[] DISTANCE_COMPONENTS = { Raster.Component.RED, Raster.Component.BLUE, Raster.Component.GREEN }; + static double distance(Raster.Component[] comps1, double[] colors1, Raster.Component[] comps2, double[] colors2) { double res = 0; double diff; @@ -80,10 +71,6 @@ return Math.sqrt(res); } - /** - * - * @return - */ public String getID() { return AverageDistanceComparator.class.getName() + ":" + getThreshold(); } --- old/core/JemmyCore/src/org/jemmy/image/pixel/BaseCountingRasterComparator.java 2018-05-20 10:11:50.244789876 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/BaseCountingRasterComparator.java 2018-05-20 10:11:50.184788045 +0200 @@ -28,26 +28,15 @@ import org.jemmy.Dimension; /** - * * @author shura */ public abstract class BaseCountingRasterComparator extends ThresholdComparator { - /** - * - * @param threshold - */ public BaseCountingRasterComparator(double threshold) { super(0, 1); setThreshold(threshold); } - /** - * - * @param image1 - * @param image2 - * @return - */ public boolean compare(Raster image1, Raster image2) { Dimension size = PixelImageComparator.computeDiffSize(image1, image2); if(size == null) { @@ -70,14 +59,6 @@ return (offPixels <= totalPixels * getThreshold()); } - /** - * - * @param comps1 - * @param colors1 - * @param comps2 - * @param colors2 - * @return - */ protected abstract boolean compare(Raster.Component[] comps1, double[] colors1, Raster.Component[] comps2, double[] colors2); } --- old/core/JemmyCore/src/org/jemmy/image/pixel/ColorMappingComparator.java 2018-05-20 10:11:50.480797078 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/ColorMappingComparator.java 2018-05-20 10:11:50.423795339 +0200 @@ -29,7 +29,6 @@ import org.jemmy.image.pixel.Raster.Component; /** - * * @author shura */ public abstract class ColorMappingComparator implements RasterComparator { @@ -38,12 +37,6 @@ final private ColorMap right; private RasterComparator subComparator; - /** - * - * @param left - * @param right - * @param subComparator - */ public ColorMappingComparator(ColorMap left, ColorMap right, RasterComparator subComparator) { this.subComparator = subComparator; @@ -51,47 +44,22 @@ this.right = right; } - /** - * - * @return - */ public RasterComparator getSubComparator() { return subComparator; } - /** - * - * @param subComparator - */ public void setSubComparator(RasterComparator subComparator) { this.subComparator = subComparator; } - /** - * - * @param both - * @param subComparator - */ public ColorMappingComparator(ColorMap both, RasterComparator subComparator) { this(both, both, subComparator); } - /** - * - * @param image1 - * @param image2 - * @return - */ public boolean compare(Raster image1, Raster image2) { return subComparator.compare(map(image1, left), map(image2, right)); } - /** - * - * @param image - * @param map - * @return - */ public WriteableRaster map(Raster image, ColorMap map) { WriteableRaster res = createView(image.getSize()); double[] colors = new double[image.getSupported().length]; @@ -106,39 +74,18 @@ return res; } - /** - * - * @param size - * @return - */ protected abstract WriteableRaster createView(Dimension size); - /** - * - * @return - */ public String getID() { return ColorMappingComparator.class.getName() + ":" + left.getID() + "," + right.getID() + "(" + subComparator.getID() + ")"; } - /** - * - */ public interface ColorMap { - /** - * - * @param components - * @param values - * @param newValues - */ public void map(Component[] components, double[] values, double[] newValues); - /** - * - * @return - */ + public String getID(); } } --- old/core/JemmyCore/src/org/jemmy/image/pixel/MaxDistanceComparator.java 2018-05-20 10:11:50.695803640 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/MaxDistanceComparator.java 2018-05-20 10:11:50.635801809 +0200 @@ -27,26 +27,15 @@ import org.jemmy.Dimension; /** - * * @author shura */ public class MaxDistanceComparator extends ThresholdComparator{ - /** - * - * @param threshold - */ public MaxDistanceComparator(double threshold) { super(0, Math.sqrt(3)); setThreshold(threshold); } - /** - * - * @param image1 - * @param image2 - * @return - */ public boolean compare(Raster image1, Raster image2) { Dimension size = PixelImageComparator.computeDiffSize(image1, image2); if (size == null) { @@ -65,10 +54,6 @@ return distance <= getThreshold(); } - /** - * - * @return - */ public String getID() { return MaxDistanceComparator.class.getName() + ":" + getThreshold(); } --- old/core/JemmyCore/src/org/jemmy/image/pixel/PNGLoader.java 2018-05-20 10:11:50.905810048 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/PNGLoader.java 2018-05-20 10:11:50.856808553 +0200 @@ -79,7 +79,7 @@ /** * Decodes image from an input stream passed into constructor. * @return a BufferedImage object - * @throws IOException + * @throws IOException todo document */ public Raster decode() throws IOException { return decode(true); @@ -91,7 +91,7 @@ * Decodes image from an input stream passed into constructor. * @return a BufferedImage object * @param closeStream requests method to close the stream after the image is read - * @throws IOException + * @throws IOException todo document */ public Raster decode(boolean closeStream) throws IOException { --- old/core/JemmyCore/src/org/jemmy/image/pixel/PNGSaver.java 2018-05-20 10:11:51.119816579 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/PNGSaver.java 2018-05-20 10:11:51.065814931 +0200 @@ -52,11 +52,6 @@ CRC32 crc; byte mode; - /** - * - * @param file - * @throws java.io.FileNotFoundException - */ public PNGSaver(File file) throws FileNotFoundException { this(new FileOutputStream(file)); } --- old/core/JemmyCore/src/org/jemmy/image/pixel/PixelEqualityRasterComparator.java 2018-05-20 10:11:51.338823262 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/PixelEqualityRasterComparator.java 2018-05-20 10:11:51.281821523 +0200 @@ -29,27 +29,14 @@ import org.jemmy.image.pixel.Raster.Component; /** - * * @author shura */ public class PixelEqualityRasterComparator extends BaseCountingRasterComparator { - /** - * - * @param treshold - */ public PixelEqualityRasterComparator(double treshold) { super(treshold); } - /** - * - * @param comps1 - * @param colors1 - * @param comps2 - * @param colors2 - * @return - */ @Override protected boolean compare(Raster.Component[] comps1, double[] colors1, Raster.Component[] comps2, double[] colors2) { @@ -65,10 +52,6 @@ } } - /** - * - * @return - */ public String getID() { return PixelEqualityRasterComparator.class.getName() + ":" + getThreshold(); } --- old/core/JemmyCore/src/org/jemmy/image/pixel/PixelImageComparator.java 2018-05-20 10:11:51.550829732 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/PixelImageComparator.java 2018-05-20 10:11:51.495828054 +0200 @@ -32,7 +32,6 @@ import org.jemmy.image.pixel.Raster.Component; /** - * * @author shura */ public abstract class PixelImageComparator implements ImageComparator { @@ -46,10 +45,6 @@ private RasterComparator comparator = null; private Environment env = null; - /** - * - * @param comparator - */ public PixelImageComparator(RasterComparator comparator) { this.comparator = comparator; } @@ -66,12 +61,6 @@ } } - /** - * - * @param one - * @param two - * @return - */ public static Dimension computeDiffSize(Raster one, Raster two) { if (one.getSize().equals(two.getSize())) { return one.getSize(); @@ -90,12 +79,6 @@ } } - /** - * - * @param image1 - * @param image2 - * @return - */ public WriteableRaster computeDifference(Raster image1, Raster image2) { Dimension size = computeDiffSize(image1, image2); if (size == null) { @@ -129,15 +112,7 @@ private static final Component[] diffComponents = { Component.RED, Component.BLUE, Component.GREEN }; - /** - * - * @param comps1 - * @param colors1 - * @param comps2 - * @param colors2 - * @param compsRes - * @param colorsRes - */ + protected void calcDiffColor(Raster.Component[] comps1, double[] colors1, Raster.Component[] comps2, double[] colors2, Raster.Component[] compsRes, double[] colorsRes) { double square1, square2; @@ -158,34 +133,12 @@ return getRasterComparator().getID(); } - /** - * - * @param image - * @return - */ protected abstract Image toImage(Raster image); - /** - * - * @param image - * @return - */ protected abstract Raster toRaster(Image image); - /** - * - * @param r1 - * @param r2 - * @return - */ protected abstract WriteableRaster createDiffRaster(Raster r1, Raster r2); - /** - * - * @param comps - * @param comp - * @return - */ public static int arrayIndexOf(Raster.Component[] comps, Raster.Component comp) { for (int i = 0; i < comps.length; i++) { if (comp == comps[i]) { --- old/core/JemmyCore/src/org/jemmy/image/pixel/Raster.java 2018-05-20 10:11:51.761836171 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/Raster.java 2018-05-20 10:11:51.709834584 +0200 @@ -32,47 +32,27 @@ */ public interface Raster { - /** - * - */ public enum Component { - - /** - * - */ RED, - /** - * - */ GREEN, - /** - * - */ BLUE, - /** - * - */ ALPHA; }; - /** - * - * @return - */ public Dimension getSize(); /** * Supposed to write pixel component value in the RGBA order. - * @param x - * @param y - * @param colors + * @param x todo document + * @param y todo document + * @param colors todo document */ public void getColors(int x, int y, double[] colors); /** * Returns a list of supported components. * - * @return + * @return todo document */ public Component[] getSupported(); --- old/core/JemmyCore/src/org/jemmy/image/pixel/RasterComparator.java 2018-05-20 10:11:51.975842702 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/RasterComparator.java 2018-05-20 10:11:51.921841054 +0200 @@ -31,16 +31,8 @@ * @author shura */ public interface RasterComparator { - /** - * - * @param image1 - * @param image2 - * @return - */ + public boolean compare(Raster image1, Raster image2); - /** - * - * @return - */ + public String getID(); } --- old/core/JemmyCore/src/org/jemmy/image/pixel/ResizeComparator.java 2018-05-20 10:11:52.195849416 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/ResizeComparator.java 2018-05-20 10:11:52.141847768 +0200 @@ -29,7 +29,6 @@ import org.jemmy.image.ImageComparator; /** - * * @author shura */ public abstract class ResizeComparator implements ImageComparator { @@ -37,65 +36,29 @@ private ImageComparator subComparator; private Mode mode; - /** - * - */ public enum Mode { - - /** - * - */ LEFT, - /** - * - */ RIGTH, - /** - * - */ MAX }; - /** - * - * @param subComparator - * @param mode - */ public ResizeComparator(ImageComparator subComparator, Mode mode) { this.subComparator = subComparator; this.mode = mode; } - /** - * - * @param subComparator - */ public ResizeComparator(ImageComparator subComparator) { this(subComparator, Mode.MAX); } - /** - * - * @return - */ public ImageComparator getSubComparator() { return subComparator; } - /** - * - * @param subComparator - */ public void setSubComparator(ImageComparator subComparator) { this.subComparator = subComparator; } - /** - * - * @param image1 - * @param image2 - * @return - */ public Image compare(Image image1, Image image2) { if(image1 == null || image2 == null) { return (image1 == null) ? image2 : image1; @@ -128,25 +91,10 @@ return subComparator.compare(r1, r2); } - /** - * - * @param image - * @param size - * @return - */ abstract public Image resize(Image image, Dimension size); - /** - * - * @param image - * @return - */ abstract public Dimension getSize(Image image); - /** - * - * @return - */ public String getID() { return ResizeComparator.class.getName() + "(" + subComparator.getID() + ")"; } --- old/core/JemmyCore/src/org/jemmy/image/pixel/ThresholdComparator.java 2018-05-20 10:11:52.414856099 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/ThresholdComparator.java 2018-05-20 10:11:52.356854329 +0200 @@ -39,10 +39,7 @@ this.max = max; } - /** - * - * @param threshold - */ + public final void setThreshold(double threshold) { if (threshold < min || threshold > max) { throw new IllegalArgumentException("Treshold expected to be withing (" --- old/core/JemmyCore/src/org/jemmy/image/pixel/WriteableRaster.java 2018-05-20 10:11:52.626862569 +0200 +++ new/core/JemmyCore/src/org/jemmy/image/pixel/WriteableRaster.java 2018-05-20 10:11:52.574860982 +0200 @@ -25,15 +25,9 @@ package org.jemmy.image.pixel; /** - * * @author shura */ public interface WriteableRaster extends Raster { - /** - * - * @param x - * @param y - * @param values - */ + public void setColors(int x, int y, double[] values); } --- old/core/JemmyCore/src/org/jemmy/input/AbstractCaretOwner.java 2018-05-20 10:11:52.832868856 +0200 +++ new/core/JemmyCore/src/org/jemmy/input/AbstractCaretOwner.java 2018-05-20 10:11:52.780867269 +0200 @@ -84,10 +84,6 @@ return caret; } - /** - * - * @return - */ protected double position() { return value; } --- old/core/JemmyCore/src/org/jemmy/input/CaretImpl.java 2018-05-20 10:11:53.047875417 +0200 +++ new/core/JemmyCore/src/org/jemmy/input/CaretImpl.java 2018-05-20 10:11:52.989873647 +0200 @@ -58,8 +58,7 @@ private ArrayList actions; /** - * - * @param wrap + * @param wrap the control wrapper instance * @param caretOwner only position() is used */ public CaretImpl(Wrap wrap, CaretOwner caretOwner) { @@ -68,18 +67,10 @@ actions = new ArrayList(); } - /** - * - * @return - */ public Wrap getWrap() { return wrap; } - /** - * - * @param action - */ protected void addScrollAction(ScrollAction action) { actions.add(0, action); } @@ -145,11 +136,6 @@ private CaretOwner caret; private double precision; - /** - * - * @param caret - * @param value - */ public DirectionToPosition(CaretOwner caret, double value, double precision) { this.value = value; this.caret = caret; @@ -165,19 +151,11 @@ return (diff == 0) ? 0 : ((diff > 0) ? 1 : -1); } - /** - * - * @return - */ @Override public String toString() { return "value == " + position(); } - /** - * - * @return - */ protected double position() { return value; } @@ -188,28 +166,14 @@ */ protected static interface ScrollAction { - /** - * - * @param direction - */ public void scrollTo(int direction); } - /** - * - */ protected class MouseScrollAction implements ScrollAction { Point up, down; KeyboardModifier[] upMods, downMods; - /** - * - * @param down - * @param downMods - * @param up - * @param upMods - */ public MouseScrollAction(Point down, KeyboardModifier[] downMods, Point up, KeyboardModifier[] upMods) { this.up = up; this.down = down; @@ -217,40 +181,21 @@ this.downMods = downMods; } - /** - * - * @param down - * @param up - */ public MouseScrollAction(Point down, Point up) { this(up, new KeyboardModifier[0], up, new KeyboardModifier[0]); } - /** - * - * @param direction - */ public void scrollTo(int direction) { wrap.mouse().click(1, (direction > 0) ? up : down, MouseButtons.BUTTON1, (direction > 0) ? upMods : downMods); } } - /** - * - */ protected class KeyboardScrollAction implements ScrollAction { KeyboardButton down, up; KeyboardModifier[] downMods, upMods; - /** - * - * @param down - * @param downMods - * @param up - * @param upMods - */ public KeyboardScrollAction(KeyboardButton down, KeyboardModifier[] downMods, KeyboardButton up, KeyboardModifier[] upMods) { this.down = down; this.up = up; @@ -258,19 +203,10 @@ this.upMods = upMods; } - /** - * - * @param down - * @param up - */ public KeyboardScrollAction(KeyboardButton down, KeyboardButton up) { this(down, new KeyboardModifier[0], up, new KeyboardModifier[0]); } - /** - * - * @param direction - */ public void scrollTo(int direction) { wrap.keyboard().pushKey((direction > 0) ? up : down, (direction > 0) ? upMods : downMods); --- old/core/JemmyCore/src/org/jemmy/input/CaretText.java 2018-05-20 10:11:53.261881948 +0200 +++ new/core/JemmyCore/src/org/jemmy/input/CaretText.java 2018-05-20 10:11:53.211880422 +0200 @@ -45,10 +45,6 @@ TextImpl text; Wrap wrap; - /** - * - * @param wrap - */ public CaretText(Wrap wrap) { this.wrap = wrap; text = new TextImpl(wrap) { @@ -69,10 +65,6 @@ caret = new TextCaret(wrap, this); } - /** - * - * @return - */ protected int getFlags() { return (Integer)wrap.getEnvironment(). getProperty(RegexCaretDirection.REGEX_FLAGS, 0); @@ -88,9 +80,10 @@ /** * Moves caret to a beginning/end of an index'th occurance of the regex. - * @param regex - * @param front - * @param index + * + * @param regex the regular expression to search for + * @param front todo document + * @param index todo document */ public void to(String regex, boolean front, int index) { caret().to(new RegexCaretDirection(this, this, regex, getFlags(), front, index)); @@ -98,8 +91,9 @@ /** * Moves caret to a beginning/end of the first occurance of the regex. - * @param regex - * @param front + * + * @param regex the regular expression to search for + * @param front todo document */ public void to(String regex, boolean front) { to(regex, front, 0); @@ -107,29 +101,18 @@ /** * Moves caret to a beginning the first occurance of the regex. - * @param regex + * + * @param regex the regular expression to search for */ public void to(String regex) { to(regex, true); } - /** - * - * @param left - * @param leftMods - * @param right - * @param rightMods - */ public void addNavKeys(KeyboardButton left, KeyboardModifier[] leftMods, KeyboardButton right, KeyboardModifier[] rightMods) { caret().addNavKeys(left, leftMods, right, rightMods); } - /** - * - * @param left - * @param right - */ public void addNavKeys(KeyboardButton left, KeyboardButton right) { addNavKeys(left, new KeyboardModifier[0], right, new KeyboardModifier[0]); } --- old/core/JemmyCore/src/org/jemmy/input/CharBindingMap.java 2018-05-20 10:11:53.473888417 +0200 +++ new/core/JemmyCore/src/org/jemmy/input/CharBindingMap.java 2018-05-20 10:11:53.416886678 +0200 @@ -36,8 +36,8 @@ * communicate what modifiers and primary key are required to generate * a given symbol. * - * @param