diff options
author | Jack Palevich <jackpal@google.com> | 2009-11-07 07:49:15 +0800 |
---|---|---|
committer | Jack Palevich <jackpal@google.com> | 2009-11-07 07:49:15 +0800 |
commit | a44249500f6c1c7a75b5c420fa2e81c2aa317a10 (patch) | |
tree | 7a793f8e3e1cb0abc67246af04c195dbbcd6d2b8 /opengl/java | |
parent | 8d51f7101d2427953dbff9b4b8f8c9fdc0dc4a91 (diff) | |
download | frameworks_base-a44249500f6c1c7a75b5c420fa2e81c2aa317a10.zip frameworks_base-a44249500f6c1c7a75b5c420fa2e81c2aa317a10.tar.gz frameworks_base-a44249500f6c1c7a75b5c420fa2e81c2aa317a10.tar.bz2 |
Fix stupid bug in GLThreadManager implementation.
The code intended to copy the old value of a field, before the
field was updated. However, what the code was actually doing was
copying the new value of the field, after the field was updated.
The fix was to move the copy to before the update.
This is work towards fixing bug 2228262
Diffstat (limited to 'opengl/java')
-rw-r--r-- | opengl/java/android/opengl/GLSurfaceView.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/opengl/java/android/opengl/GLSurfaceView.java b/opengl/java/android/opengl/GLSurfaceView.java index 952eff2..ac27a2d 100644 --- a/opengl/java/android/opengl/GLSurfaceView.java +++ b/opengl/java/android/opengl/GLSurfaceView.java @@ -1281,8 +1281,8 @@ public class GLSurfaceView extends SurfaceView implements SurfaceHolder.Callback public void start(GLThread thread) throws InterruptedException { GLThread oldThread = null; synchronized(this) { - mMostRecentGLThread = thread; oldThread = mMostRecentGLThread; + mMostRecentGLThread = thread; } if (oldThread != null) { synchronized(oldThread) { |