On 03/02/2014 20:17, roger riggs wrote:
Please re-review.

I missed a warning that the CHECK_NULL macros was being redefined.

Retaining the previous changes to java/util/jar/pack that removed the redefinition
addresses the issue.

The webrev now only has a few small changes to cleanup the name and use of CHECK macros. The CHECK_NULL_* macros are more generally useful without the JNU_ prefix. The JNU_CHECK_EXCEPTION* macros apply only when there is an jni environment.

Webrev:
    http://cr.openjdk.java.net/~rriggs/webrev-check-cleanup-8031737/
Thumbs up from me too.

-Alan

Reply via email to