Skip to content
This repository has been archived by the owner on Nov 1, 2020. It is now read-only.

Commit

Permalink
remove unused private field
Browse files Browse the repository at this point in the history
Test: CtsGraphicsTests

Change-Id: I611f5dc127636f67dc219ef15adc0a591a72a95e
  • Loading branch information
reed-at-google committed May 1, 2017
1 parent 926d4a4 commit 287a131
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
10 changes: 0 additions & 10 deletions core/jni/com_google_android_gles_jni_EGLImpl.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,6 @@ static jmethodID gConfig_ctorID;
static jfieldID gDisplay_EGLDisplayFieldID;
static jfieldID gContext_EGLContextFieldID;
static jfieldID gSurface_EGLSurfaceFieldID;
static jfieldID gSurface_NativePixelRefFieldID;
static jfieldID gConfig_EGLConfigFieldID;

static inline EGLDisplay getDisplay(JNIEnv* env, jobject o) {
Expand Down Expand Up @@ -84,7 +83,6 @@ static void nativeClassInit(JNIEnv *_env, jclass eglImplClass)

jclass surface_class = _env->FindClass("com/google/android/gles_jni/EGLSurfaceImpl");
gSurface_EGLSurfaceFieldID = _env->GetFieldID(surface_class, "mEGLSurface", "J");
gSurface_NativePixelRefFieldID = _env->GetFieldID(surface_class, "mNativePixelRef", "J");
}

static const jint gNull_attrib_base[] = {EGL_NONE};
Expand Down Expand Up @@ -398,14 +396,6 @@ static jboolean jni_eglDestroySurface(JNIEnv *_env, jobject _this, jobject displ
}
EGLDisplay dpy = getDisplay(_env, display);
EGLSurface sur = getSurface(_env, surface);

if (sur) {
SkPixelRef* ref = (SkPixelRef*)(_env->GetLongField(surface,
gSurface_NativePixelRefFieldID));
if (ref) {
SkSafeUnref(ref);
}
}
return EglBoolToJBool(eglDestroySurface(dpy, sur));
}

Expand Down
3 changes: 0 additions & 3 deletions opengl/java/com/google/android/gles_jni/EGLSurfaceImpl.java
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,11 @@

public class EGLSurfaceImpl extends EGLSurface {
long mEGLSurface;
private long mNativePixelRef;
public EGLSurfaceImpl() {
mEGLSurface = 0;
mNativePixelRef = 0;
}
public EGLSurfaceImpl(long surface) {
mEGLSurface = surface;
mNativePixelRef = 0;
}

@Override
Expand Down

0 comments on commit 287a131

Please sign in to comment.