From a90086a914f2fad9686e8e3d23dcdf65f38360eb Mon Sep 17 00:00:00 2001 From: Narayan Kamath Date: Thu, 27 Feb 2014 14:12:55 +0000 Subject: [PATCH] API changes for 64 bit platforms. EGLObjectHandle.(int) and int EGLObjectHandle.getHandle() have now been deprecated and replaced with variants that take and return java longs. bug: 13181704 Change-Id: Ie88e591c288d6de5655364b4cd673f61cce68d7d --- .../glgen/static/egl/EGLObjectHandle.java | 33 +++++++++---------- 1 file changed, 16 insertions(+), 17 deletions(-) diff --git a/opengl/tools/glgen/static/egl/EGLObjectHandle.java b/opengl/tools/glgen/static/egl/EGLObjectHandle.java index e6e397602..113f867c4 100644 --- a/opengl/tools/glgen/static/egl/EGLObjectHandle.java +++ b/opengl/tools/glgen/static/egl/EGLObjectHandle.java @@ -24,18 +24,28 @@ package android.opengl; public abstract class EGLObjectHandle { private final long mHandle; - // TODO Deprecate EGLObjectHandle(int) method + /** + * @deprecated Use {@link EGLObjectHandle(long)} instead. Handles + * on 64 bit platforms will be wider than java ints. + */ + @Deprecated protected EGLObjectHandle(int handle) { mHandle = handle; } - // TODO Unhide the EGLObjectHandle(long) method - /** - * {@hide} - */ protected EGLObjectHandle(long handle) { mHandle = handle; } - // TODO Deprecate getHandle() method in favor of getNativeHandle() + /** + * @deprecated Use {@link #getNativeHandle()} instead. Handles on + * 64 bit platforms will be wider than java ints. + */ + @Deprecated + public int getHandle() { + if ((mHandle & 0xffffffffL) != mHandle) { + throw new UnsupportedOperationException(); + } + return (int)mHandle; + } /** * Returns the native handle of the wrapped EGL object. This handle can be * cast to the corresponding native type on the native side. @@ -44,17 +54,6 @@ public abstract class EGLObjectHandle { * * @return the native handle of the wrapped EGL object. */ - public int getHandle() { - if ((mHandle & 0xffffffffL) != mHandle) { - throw new UnsupportedOperationException(); - } - return (int)mHandle; - } - - // TODO Unhide getNativeHandle() method - /** - * {@hide} - */ public long getNativeHandle() { return mHandle; }