Change looks correct.

Approved.

        Dave

On 7/5/2013 2:36 PM, Tim Bell wrote:
Hello everyone

Unfortunately there is no public link for 8012925

This is a one-line change in jdk/src/macosx/native/sun/awt/AWTEvent.m to return a value (0) instead of returning nothing. Returning 0 is consistent with the code at line 418 in the same method:

   http://cr.openjdk.java.net/~tbell/8012925/webrev.00/

I came across this during my work on 8019470:
   http://bugs.sun.com/view_bug.do?bug_id=8019470

When building with clang on MacOS, I get a fatal error over this 'warning', so I needed to fix it before I could continue working.

Thanks in advance-

Tim


Reply via email to