ok. looks good to me.

Regards
Prasanta
On 4/14/2016 1:59 PM, Muneer Kolarkunnu wrote:

I added the @run tag and updated the review: http://cr.openjdk.java.net/~ntv/muneer/8025430/webrev.01/ <http://cr.openjdk.java.net/%7Entv/muneer/8025430/webrev.01/>

I tested with latest jtreg with and without @run tag, both are working.

But there is a difference in the logs in the following line:

*Without @run flag:*

command: main bug5076514

reason: Assumed action based on file name: run main bug5076514

*With @run flag:*

command: main bug5076514
reason: User specified action: run main bug5076514

Regards,

Muneer

*From:*prasanta sadhukhan
*Sent:* Wednesday, April 13, 2016 7:49 PM
*To:* Muneer Kolarkunnu; [email protected]
*Subject:* Re: Review request for 8025430: [TEST_BUG] javax/swing/JEditorPane/5076514/bug5076514.java failed since jdk8b108

You need to add
@run main <test> for latest jtreg to run. Have you run against latest jtreg?

Regards
Prasanta

On 4/13/2016 6:10 PM, Muneer Kolarkunnu wrote:

    Hi All,

                    Please review the fix for test bug 8025430,

                    Webrev:
    http://cr.openjdk.java.net/~ntv/muneer/8025430/webrev.00/
    <http://cr.openjdk.java.net/%7Entv/muneer/8025430/webrev.00/>

                    Bug: https://bugs.openjdk.java.net/browse/JDK-8025430

    Issue:

    It is a test bug, API checkSystemClipboardAccess() is deprecated.
    Deprecated message from java doc for your reference : "The
    dependency on AWTPermission creates an impediment to future
    modularization of the Java platform. Users of this method should
    instead invoke checkPermission(java.security.Permission) directly.
    This method will be changed in a future release to check the
    permission java.security.AllPermission."

    Fix:

                  API  checkPermission(java.security.Permission) is
    used instead of checkSystemClipboardAccess()

    Regards,

    Muneer


Reply via email to