summaryrefslogtreecommitdiffstats
path: root/opengl/java
diff options
context:
space:
mode:
authorJack Palevich <jackpal@google.com>2009-11-07 07:49:15 +0800
committerJack Palevich <jackpal@google.com>2009-11-07 07:49:15 +0800
commita44249500f6c1c7a75b5c420fa2e81c2aa317a10 (patch)
tree7a793f8e3e1cb0abc67246af04c195dbbcd6d2b8 /opengl/java
parent8d51f7101d2427953dbff9b4b8f8c9fdc0dc4a91 (diff)
downloadframeworks_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.java2
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) {