Kill stale assert, fix for 4310384

There was an old assert in dvmPopFrame that did a sanity check on
the JNI local reference table it was popping.  In the asserting case,
we had internal native method calls and no JNI - and thus no sane
local reference table.  Further, the localRefTable code has changed enough
that this particular sanity check is no longer useful.  Assert deleted,
along with another related (and previously-commented out) assert.

Change-Id: I3a806838fa2b5909c3e3b3ad27aeed29e3e467ab

diff --git a/vm/interp/Stack.c b/vm/interp/Stack.c
index 97fbeb5..db06866 100644 (file)
--- a/vm/interp/Stack.c
+++ b/vm/interp/Stack.c
@@ -323,10 +323,6 @@ static bool dvmPopFrame(Thread* self)
                 saveBlock->method->name,
                 (SAVEAREA_FROM_FP(saveBlock->prevFrame)->method == NULL) ?
                 "" : " (JNI local)");
-            assert(saveBlock->xtra.localRefCookie != 0);
-            //assert(saveBlock->xtra.localRefCookie >= self->jniLocalRefTable.table &&
-            //    saveBlock->xtra.localRefCookie <=self->jniLocalRefTable.nextEntry);
-
             dvmPopJniLocals(self, saveBlock);
         }