[rfc][icedtea-web] AWTHelper small modification

[Jiri Vanek]

On 04/30/2013 06:04 PM, Jana Fabrikova wrote:
> Hello, I am sending only a small modification to the file AWTHelper,
> part of AWTFramework, in dealing with an initialisation string (it can
> be null or given),
> thanks for any comments,
> Jana
>
>
> ChangeLog:
>
> 2013-04-30  Jana Fabrikova<jfabriko at redhat.com>
>
>          * tests/test-extensions/net/sourceforge/jnlp/awt/AWTHelper.java:
>          modifying the (charrReaded) method
>
>
> modifying_AWTHelper_null_initStr.patch
>
Hmm this does not seems to be correct.
a) This should allow awt helper to work with null "magic string"
b) this should allow awt helper to run without need of magic string.

The (a) looks nearly ok, but I'm confused by initStrGiven variable. It jus disappeared from condition. So it is not needed any more? Then please remove it.
Or have it just slipped from condition accidentally?

for (b) I'm not sure if it is enough - is it?
>
> diff -r 3fa3d0fdce30 tests/test-extensions/net/sourceforge/jnlp/awt/AWTHelper.java
> --- a/tests/test-extensions/net/sourceforge/jnlp/awt/AWTHelper.java	Tue Apr 30 11:31:28 2013 -0400
> +++ b/tests/test-extensions/net/sourceforge/jnlp/awt/AWTHelper.java	Tue Apr 30 18:00:55 2013 +0200
> @@ -60,7 +60,7 @@
>   public abstract class AWTHelper extends RulesFolowingClosingListener implements Runnable{
>
>       //attributes possibly set by user
> -    private String initStr = "";
> +    private String initStr = null;
>       private Color appletColor;
>       private BufferedImage marker;
>       private Point markerPosition;
> @@ -171,6 +171,7 @@
>           } catch (IOException e) {
>               throw new RuntimeException("AWTHelper could not read marker.png.",e);
>           }
> +
>
>           this.appletWidth = appletWidth;
>           this.appletHeight = appletHeight;
> @@ -206,7 +207,7 @@
>       public void charReaded(char ch) {
>           sb.append(ch);
>           //is applet ready to start clicking?
> -        if (initStrGiven&&  !actionStarted&&  appletIsReady(sb.toString())) {
> +        if ((initStr != null)&&  !actionStarted&&  appletIsReady(sb.toString())) {
also pelase keep code according to rules  -  spaces on both sides of logical operators

>               try{
>                   actionStarted = true;
>                   this.findAndActivateApplet();

Thank you for keeping teh awt helper classes more usable - it was not possible without second person to try it.

   Thank you
   J.