am 82ebe52a: Merge "Javadocs for JSONObject."

Merge commit '82ebe52a3e4ee91ccbef38d241b2b1fd9409a03e' into dalvik-dev

* commit '82ebe52a3e4ee91ccbef38d241b2b1fd9409a03e':
  Javadocs for JSONObject.

Merge "Javadocs for JSONObject."

Javadocs for JSONObject.

Change-Id: I5ec9df6a3a9baac8f4f498890cd35feff774737a

Merge "misc DalvikRunner changes" into dalvik-dev

Merge "Add Java 6's PrintStream.clearError/PrintWriter.clearError methods." into dalvik-dev

Add Java 6's PrintStream.clearError/PrintWriter.clearError methods.

Also bring back the latest harmony java6 branch tests.

Bug: 2497395
Change-Id: I242e9a7d0446b46faaa5b28e0348806e999d8fdf

Fix "run-core-tests.sh" to work again.

cshapiro was complaining about this the other week, and I needed it today.
We should fix the duplication, but Android's crippled shell makes that
awkward, and this is the simplest thing that lets us carry on working...

Change-Id: I4c1a764d6069ba55a021631acdc4d6a665df0089

Merge "Add Java 6's java.io.Console." into dalvik-dev

Add Java 6's java.io.Console.

This is actually functional, if you're in the mood to "adb shell".

The implementation is based on harmony's, but with the initialization and
native code rewritten, with readPassword responsible for echoing a newline (so
we don't have to play silly tricks with the ECHONL flag), and a vastly
simplified ConsoleReader class. I've also rewritten the documentation.

Change-Id: I902b47fb27a8fdb2d6f067bb905ee02c6a10e454

misc DalvikRunner changes

Summary:
- "vogar --debug-port <port>" now overrides --timeout-seconds to be zero
  to prevent timeouts during interactive debugging
- "vogar --timeout-seconds 0" now disables timeout
  as a side-effect of the previous change
- "vogar" no longer defaults to "javac -Xmaxerrs 1"
  This is to make using vogar's output more useful when iteratively developing tests.
  continuous build needs to use "vogar --javac-arg -Xmaxerrs --javac-arg 1"
- "vogar --ident ''" allows disabling of output formating
  This prevents output parsing problems for Emacs *compilation* buffers

Change-Id: I599590c9c4fe3c57524af5c4d7f2be2e30ad4685

am 9fa39c91: Add missing macro SIGNATURE_BREAKPOINT.

Merge commit '9fa39c91c885a05ce544d0899089212321413c13' into dalvik-dev

* commit '9fa39c91c885a05ce544d0899089212321413c13':
  Add missing macro SIGNATURE_BREAKPOINT.

Add missing macro SIGNATURE_BREAKPOINT.

Change-Id: I9e450b7b2ff34a3609a99bf8fe11e3efa88ccc30

am 79cea5c9: Merge "Implement signature-based breakpoint for the JIT compiler."

Merge commit '79cea5c97267e0b3e35e0c2b70e2662aa533f200' into dalvik-dev

* commit '79cea5c97267e0b3e35e0c2b70e2662aa533f200':
  Implement signature-based breakpoint for the JIT compiler.

Merge "Implement signature-based breakpoint for the JIT compiler."

am 288de8e6: Merge "Replace a use of index with strchr."

Merge commit '288de8e67b5a2c2e6a19372a2b99f17ca17de2fa' into dalvik-dev

* commit '288de8e67b5a2c2e6a19372a2b99f17ca17de2fa':
  Replace a use of index with strchr.

Merge "Replace a use of index with strchr."

Implement signature-based breakpoint for the JIT compiler.

The JIT compiler will replay itself with verbose printing if the compiled
instruction stream matches the customized signature from memory dump in the
bugreports.

Change-Id: I024082c5744903273b24a0a73468b9c0d3588ad5

Import the heap verification code from the copying collector.  The
reference verification routine adds an extra argument so the base
address of an object can be passed to the verification code without
provoking a warning from GCC about breaking alias analysis.

Change-Id: Idd921bcc0e084c18bff1e209a8591ef55f57843a

Replace a use of index with strchr.

Change-Id: I2c1238bc1bd0945533712d0d74a761721b8408a3

Merge "Add some certimport.sh documention" into dalvik-dev

Add some certimport.sh documention

hku had some questions about the meaning of the filenames in
cacerts. ngm responded so I captured the knowledge in the
certimport.sh for future reference so we can continue to follow the
convention.

Change-Id: I79b4ed333e541f09d60143f785e5b10d3f5e60f4

am 900a3afd: Jit: Fix register usage bug - Issue 2518825 native crash running ARMv5te JIT

Merge commit '900a3afd0e8e0d88426b21447d601ee67e17b642' into dalvik-dev

* commit '900a3afd0e8e0d88426b21447d601ee67e17b642':
  Jit: Fix register usage bug - Issue 2518825 native crash running ARMv5te JIT

Jit: Fix register usage bug - Issue 2518825 native crash running ARMv5te JIT

Change I8ca61804 added a call to dvmCanPutArrayElement for APUT_OBJECT,
but did so in a way that violated register usage restrictions.  This change
tells the register allocation system what registers we expect to remain
live across the call to dvmCanPutArrayElement.

Change-Id: Icd83b888ba60768a196070d62d07d12c7a3c73c6

am fcd1cbd4: Merge "Include strings.h directly for its ffs prototype."

Merge commit 'fcd1cbd456809d2a2c58644efb71e1f42810d3ab' into dalvik-dev

* commit 'fcd1cbd456809d2a2c58644efb71e1f42810d3ab':
  Include strings.h directly for its ffs prototype.

Merge "Include strings.h directly for its ffs prototype."

Merge "Jit: Handle new VOLATILE Dalvik ops by not handling them." into dalvik-dev

am 38b778b3: Fixing an @link issue that is upsetting DroidDoc.

Merge commit '38b778b364061dc87899e7fe0e3efd2502a538fd' into dalvik-dev

* commit '38b778b364061dc87899e7fe0e3efd2502a538fd':
  Fixing an @link issue that is upsetting DroidDoc.

Fixing an @link issue that is upsetting DroidDoc.

Change-Id: I9837cfd46684ac6d5b9ec4ac6809da3c3f61d57d

Merge "make DalvikRunner ignore .#Foo.java files" into dalvik-dev

make DalvikRunner ignore .#Foo.java files

Emacs makes hidden bogus symlinks for open modified files such as
".#Foo.java -> user@host.pid:rand" which I'm guessing are to identify
the owning user/host/process. Unfortunately, DalvikRunner was picking
these up as source and trying to compile them.

This change makes NamingPatternCodeFinder ignore these. In order to
avoid duplicating this in all the subclasses, the subclasses now call
super.matches as a first pass. Now subclasses only apply any
additional filtering they want. MainFinder.matches was deleted since
it wasn't adding any new restrictions.

We arguably could add a better starting filter perhaps, but skipping
dot files seems a reasonable first stem.

Change-Id: Ic9f5ec0bb629d5e1bc25a5b80a6619a64285d2b0

am 6abe2582: Merge "First half of JSON Javadocs."

Merge commit '6abe2582f9dea9887b4cf6cdee200246b6e86117' into dalvik-dev

* commit '6abe2582f9dea9887b4cf6cdee200246b6e86117':
  First half of JSON Javadocs.

Merge "First half of JSON Javadocs."

First half of JSON Javadocs.

Change-Id: I277ec3b35a28802dd7b7f82c1f4bbadbd3cc4c65

Merge "Remove dead fields, constants, and redundant pass-through methods." into dalvik-dev

am f8a44e85: Merge "Point debuggerd at the interesting thread."

Merge commit 'f8a44e85b2beed31f0cb1688ac6bdc08bc37ece1' into dalvik-dev

* commit 'f8a44e85b2beed31f0cb1688ac6bdc08bc37ece1':
  Point debuggerd at the interesting thread.

Merge "Point debuggerd at the interesting thread."

Replace free calls to the pthread mutex lock, unlock, and trylock
functions with calls to the error checked wrapper functions.  This
ensures that all mutex operations are checked in debug builds.

Change-Id: I4a5f181e025a2974f3325bcd9efa861eb6a92978

Include strings.h directly for its ffs prototype.

Change-Id: If957427bd0918b3b06e6b6177227f107a4b5a5c5

am 27182541: Implementing Document.renameNode() and DOMImplementation.getFeature().

Merge commit '271825415aa961bdd9f28a551575bcee6f27b4ab' into dalvik-dev

* commit '271825415aa961bdd9f28a551575bcee6f27b4ab':
  Implementing Document.renameNode() and DOMImplementation.getFeature().

Point debuggerd at the interesting thread.

There are a couple of situations (spin-on-suspend, HeapWorker wedged)
where the current thread's native stack is much less interesting than one
of the others.  This change will cause a couple of signals to be thrown
at the "interesting" thread in an attempt to get more useful information
out of debuggerd in these cases.

For bug 2517042.

Change-Id: Ib8c0e0ef93f07b7114e5f4638a907a0f7802bdc0

Jit: Handle new VOLATILE Dalvik ops by not handling them.

See [Issue 1633591] Volatile long/double accesses should be atomic.
Because we believe this to be a rare case, the Jit will just punt
to the interpreter for these.

Change-Id: Idd05b5acae9aa5ffa60941cba8533534a89c0ff8

Make wide-volatile loads and stores atomic.

This implements the four wide-volatile instructions added in a previous
change, and modifies the verifier to substitute the opcodes into the
instruction stream when appropriate.

For mterp, the ARM wide get/put instructions now have conditional code
that replaces ldrd/strd with a call to the quasiatomic functions.  The
C version does essentially the same thing.  ARMv4T lacks ldrd/stdrd, and
uses separate implementations for the wide field accesses, so those were
updated as well.  x86 will just use stubs.

The JIT should punt these to the interpreter.

Change-Id: Ife88559ed1a698c3267d43c454896f6b12081c0f
Also:
 - We don't seem to be using the negative widths in the instruction
   table.  Not sure they're useful anymore.
 - Tabs -> spaces in x86-atom throw-verification-error impl.

Implementing Document.renameNode() and DOMImplementation.getFeature().

The rename code required moving some behaviour from ElementImpl
and AttrImpl up to their common superclass, NodeImpl.

Change-Id: I30910de146f525a5ecc837895ce5808928b858a0

Remove dead fields, constants, and redundant pass-through methods.

(This is just preliminary to what I really want to do, which is vastly
reduce the number of near-duplicate methods we have: all the send/recv
variants, for example.)

Change-Id: Ib03653820a18f55aea6c5b876c397afd281a98dc

Merge "Eliminate unused heap bitmap functions.  This is mostly the "list" code which is no longer needed." into dalvik-dev

am 1639f705: Merge "Fix the JIT blocking mode to unblock itself."

Merge commit '1639f7054fd7056a089fabe57d02639973d7da10' into dalvik-dev

* commit '1639f7054fd7056a089fabe57d02639973d7da10':
  Fix the JIT blocking mode to unblock itself.

Merge "Fix the JIT blocking mode to unblock itself."

Fix the JIT blocking mode to unblock itself.

Change-Id: Iaf1da9ee3ce337f2c5ad4985a6c776bc68472f8c

am a58fc616: Merge "Try to show lock owner in MONITOR thread dump."

Merge commit 'a58fc61670d91cad6eab32b665742a7ce956aebb' into dalvik-dev

* commit 'a58fc61670d91cad6eab32b665742a7ce956aebb':
  Try to show lock owner in MONITOR thread dump.

Merge "Try to show lock owner in MONITOR thread dump."

Try to show lock owner in MONITOR thread dump.

A thread in the MONITOR state is blocked waiting on a monitor.  This
raises two interesting questions: (1) what lock is it waiting on, and
(2) who holds that lock?  The answer to (1) can be determined easily by
looking at the source code, but (2) is a bit harder.

This change extracts the target object from the instruction stream and
prints some information about it, e.g.:

- waiting to lock <0x40028c68> (a java.lang.Object) held by threadid=1 (main)

Also: fiddled with "must [not] be locked" on a recently-added function.

(cherry-picked from dalvik-dev)

Change-Id: Ic16695741760d50be70e70fb7470972cef28bb09

am 62145b47: Merge "Jit: Minor cleanup - enum size fix, remove useless code, control consistency."

Merge commit '62145b47a29777d29ec0f8883f84f2336f5522df' into dalvik-dev

* commit '62145b47a29777d29ec0f8883f84f2336f5522df':
  Jit: Minor cleanup - enum size fix, remove useless code, control consistency.

Merge "Jit: Minor cleanup - enum size fix, remove useless code, control consistency."

am bcdd4b09: Increase the SHORT_DELAY amount in the JSR166 test.

Merge commit 'bcdd4b09d018f24aaa61311f3e543636f7eb4095' into dalvik-dev

* commit 'bcdd4b09d018f24aaa61311f3e543636f7eb4095':
  Increase the SHORT_DELAY amount in the JSR166 test.

Jit: Minor cleanup - enum size fix, remove useless code, control consistency.

Change-Id: Id8c16303efd25683ad4b04a85e0d2a059b5ec3be

Try to show lock owner in MONITOR thread dump.

A thread in the MONITOR state is blocked waiting on a monitor.  This
raises two interesting questions: (1) what lock is it waiting on, and
(2) who holds that lock?  The answer to (1) can be determined easily by
looking at the source code, but (2) is a bit harder.

This change extracts the target object from the instruction stream and
prints some information about it, e.g.:

- waiting to lock <0x40028c68> (a java.lang.Object) held by threadid=1 (main)

Change-Id: Iad18fc6f2df4142368bdf1063b8cc71de2d66156
Also: fiddled with "must [not] be locked" on a recently-added function.

Increase the SHORT_DELAY amount in the JSR166 test.

Change-Id: I890621226c5ea412baee568f3e9b31833185849a

Merge "Factor out lock inflation into a common subroutine." into dalvik-dev

Factor out lock inflation into a common subroutine.

Change-Id: Iba789778dbaf34cf92f3ca2ebb96d57ec488c20f

am fcfb5220: Adding an Apache-licensed implementation of org.json

Merge commit 'fcfb52201103bf732a6233ea45daaa65e253543b' into dalvik-dev

* commit 'fcfb52201103bf732a6233ea45daaa65e253543b':
  Adding an Apache-licensed implementation of org.json

am f662e543: Removing the non-free org.json implementation

Merge commit 'f662e543f4bf86ccc6d4375f664268da4208b61f' into dalvik-dev

* commit 'f662e543f4bf86ccc6d4375f664268da4208b61f':
  Removing the non-free org.json implementation

am 37cfb2f4: Merge "A cleanroom implementation of the org.json API."

Merge commit '37cfb2f4bbd39b494172aad059fbe89a45d18a00' into dalvik-dev

* commit '37cfb2f4bbd39b494172aad059fbe89a45d18a00':
  A cleanroom implementation of the org.json API.

am 95cd9ac6: Stay in the dbg interpreter if the entry reason is to deal with exceptions.

Merge commit '95cd9ac6c308008b6bd48e6e516c118fce0e6a1f' into dalvik-dev

* commit '95cd9ac6c308008b6bd48e6e516c118fce0e6a1f':
  Stay in the dbg interpreter if the entry reason is to deal with exceptions.

am 3ed90a94: Merge "Add a native method dalvik.system.VMDebug.infopoint(int id)."

Merge commit '3ed90a94fa2eea45a423471405df5df62b187b83' into dalvik-dev

* commit '3ed90a94fa2eea45a423471405df5df62b187b83':
  Add a native method dalvik.system.VMDebug.infopoint(int id).

Merge "Remove dead obfuscatory code." into dalvik-dev

Adding an Apache-licensed implementation of org.json

Change-Id: I1b67bac70bd25220a619e6ebe61f7f1c6f316faa

Removing the non-free org.json implementation

Change-Id: Ife53082824f5916665f198d4b2fb863271841ce0

Merge "A cleanroom implementation of the org.json API."

A cleanroom implementation of the org.json API.

This implementation lacks documentation. I intend to write that after checking
it into the master branch. By not waiting we'll have more time to exercise the
code, if only in Google's own applications.

This passes all of my tests. I rewrote some of the tests to make Crockford's
implementation fail. The tests that fail on Crockford's implementation are:

  JSONArrayTest
  testEqualsAndHashCode equals() not consistent with hashCode()
  testTokenerConstructorParseFail StackOverflowError
  testStringConstructorParseFail StackOverflowError

  JSONObjectTest
  testOtherNumbers Object.put() accepted a NaN (via a custom Number class)
  testMapConstructorWithBogusEntries JSONObject constructor doesn't validate its input!

  JSONTokenerTest
  testNextNWithAllRemaining off-by-one error?
  testNext0 Returning an empty string should be valid
  testNextCleanCommentsTrailingSingleSlash nextClean doesn't consume a trailing slash
  assertNotClean The character line tabulation is not whitespace according to the JSON spec.
  testNextToDoesntStopOnNull nextTo() shouldn't stop after \0 characters
  testNextToConsumesNull nextTo shouldn't consume \0.
  testSkipToStopsOnNull skipTo shouldn't stop when it sees '\0'

  ParsingTest
  testParsingLargeHexValues For input "0x80000000" Hex values are parsed as Strings if their signed value is greater than Integer.MAX_VALUE.
  testSyntaxProblemUnterminatedArray Stack overflowed on input "["

Change-Id: I44c4a4a698a66bf043ed339d6bd804951e732cbf

Remove dead obfuscatory code.

Bug: 2509206
Change-Id: I72a34e1f80fd6936d255b94daf5683cc2a0f2327

Stay in the dbg interpreter if the entry reason is to deal with exceptions.

Bug: 2487514
Change-Id: Id1eb91f347ff772714d9560370b9994f46d3c5cb

Merge "Add a native method dalvik.system.VMDebug.infopoint(int id)."

Add a native method dalvik.system.VMDebug.infopoint(int id).

With gdb, the JIT can use it to inspect the VM state when an instrumented line
is reached.

Change-Id: Id39ac4cd564bc1a61208cb7527c30f62b5de3e4e

am be6534f3: Jit: Fix for [Issue 2487514] Dropped exception

Merge commit 'be6534f384529e51dfba5c3f1b7eb90c86b66e77' into dalvik-dev

* commit 'be6534f384529e51dfba5c3f1b7eb90c86b66e77':
  Jit: Fix for [Issue 2487514] Dropped exception

Jit: Fix for [Issue 2487514] Dropped exception

The jit was failing to call dvmCanPutArrayElement for aput-object.

Change-Id: I8ca618048dc4d1be5b1f1ed85078759041883b09

Merge "Java 6 java.io.File changes." into dalvik-dev

Java 6 java.io.File changes.

IOError and IOException are just copied from harmony. The native code
is all our own, and the File code is rewritten to match our earlier
changes. FileTest gains more tests for the new methods' behavior with
File("").

We still can't test most of this stuff directly because we run our tests
as root. Manual inspection looks good, though, and I've raised a bug
for switching our tests over to run as a non-root user.

I've removed all the Java 6 exceptions from expectations.txt because
we plan on fixing them all, and almost all of the ones mentioned there
are already addressed anyway.

Bug: 2497395
Change-Id: I517cd2d871bff64d63b2f4eb80fda4cfd6d19cd8

am 7cd9d693: Merge "Add a JIT unit test for ArrayStoreException in APUT_OBJECT."

Merge commit '7cd9d693146fe1d9a890b7f950c23e6b220cdde4' into dalvik-dev

* commit '7cd9d693146fe1d9a890b7f950c23e6b220cdde4':
  Add a JIT unit test for ArrayStoreException in APUT_OBJECT.

Merge "Add a JIT unit test for ArrayStoreException in APUT_OBJECT."

Add a JIT unit test for ArrayStoreException in APUT_OBJECT.

Bug: 2487514

tests> ./run-test --fast 083-jit-regressions
/home/bccheng/local/master-git/dalvik/tests/083-jit-regressions: running...
/home/bccheng/local/master-git/dalvik/tests/083-jit-regressions: succeeded!

tests> ./run-test --jit 083-jit-regressions
/home/bccheng/local/master-git/dalvik/tests/083-jit-regressions: running...
/home/bccheng/local/master-git/dalvik/tests/083-jit-regressions: FAILED!

--- expected.txt    2010-03-12 13:45:13.000000000 -0800
+++ output.txt  2010-03-12 14:51:38.000000000 -0800
@@ -1,3 +1,3 @@
 b2296099 passes
 b2302318 passes
-b2487514 passes
+b2487514 fails: catchCount is 17 (expecting 1000)

Change-Id: I6651a3e0945f96bbb9e0c0af13f2c322b82d0cb0

Back out my droiddoc-breaking Object.getClass signature change.

I'll try again when I've got time to fix droiddoc (or we get an SET).
This is a non-functional Java 5/Java 6 difference, so it isn't important.

(Amusingly, there's no need to back out the corresponding api.xml
change: it looks like droiddoc normalizes one but not the other.)

Change-Id: I012ebeb199696d5bd16de63e657b15889d546c5e

Merge "Use ioctl(FIONREAD) rather than recv(MSG_PEEK) to implement Socket.available." into dalvik-dev

am 92e01317: Adding support for DOM3 attribute IDs and SchemaTypeInfos.

Merge commit '92e01317d2428856cee52965745d17699a33be5a' into dalvik-dev

* commit '92e01317d2428856cee52965745d17699a33be5a':
  Adding support for DOM3 attribute IDs and SchemaTypeInfos.

Eliminate unused heap bitmap functions.  This is mostly the "list"
code which is no longer needed.

Change-Id: I3636a4a6a36e55d2ad080571cdb6c23a16f98aaf

Use ioctl(FIONREAD) rather than recv(MSG_PEEK) to implement Socket.available.

As well as avoiding unnecessary work, this also means we can reuse the native
code we already have for ioctl(FIONREAD), though we need to change the
interface to take FileDescriptor rather than int, and extract the int on the
native side.

Change-Id: I4c6d9e4e72e514e3e7872c7f882bda5441059915

Adding support for DOM3 attribute IDs and SchemaTypeInfos.

Change-Id: I35e56ed989820df6b8fea36bbf81fe0314c76304

Fix all instances of "new Integer" (et cetera).

(This doesn't include libcore/xml/ because I don't want to get in the way there.)

Change-Id: I46f638105d26e82d09128fca605117322229e146

Merge "Plug a resource leak with "finally" and make Object.getClass' return type match Java 6." into dalvik-dev

Plug a resource leak with "finally" and make Object.getClass' return type match Java 6.

Change-Id: Ia8ae90634bfb3680c8e82e4e4cf7f7459263c3d3

am 6f9b4d2b: Merge "Implementing getBaseUri() for DOM."

Merge commit '6f9b4d2bdccb2ea5e548b908e5cc882d262d0940' into dalvik-dev

* commit '6f9b4d2bdccb2ea5e548b908e5cc882d262d0940':
  Implementing getBaseUri() for DOM.

Merge "Implementing getBaseUri() for DOM."

Implementing getBaseUri() for DOM.

The tests for this uncover problems with the RI's implementation.
It does bizarre things with hashes, trashing the absolute URI and
returning a page-relative one (like "#foo").

Change-Id: Ib8af163a7b359e3f72a9c94eb3dd7e81e3a9a95c

am 4527387d: Jit: Make debugging mode aware of inlineExecute/moveResult optimization

Merge commit '4527387dd3b5c4dce7300c764805ffd0f3d22649' into dalvik-dev

* commit '4527387dd3b5c4dce7300c764805ffd0f3d22649':
  Jit: Make debugging mode aware of inlineExecute/moveResult optimization

Jit: Make debugging mode aware of inlineExecute/moveResult optimization

The Jit has a mode in which selected opcodes can be handled normally
or single-stepped in the interpter.  This was broken for cases in
which the Jit applied an optimization to fold inlineExecute/moveResult
intruction pairs into a single operation and the debug mode was set
to handle the two opcodes differently.

Change-Id: Ifa436d4ba66ba0c13ea366c0956e6cf92ce9cdfd

Throw UnknownFormatConversionException instead of AssertionError.

There's already a perfectly good exception to throw when we see
a format conversion we don't understand, and my assertion prevented
us from getting to the code that would have thrown it.

Change-Id: I38fdd95482a3d48e26b68b045444f45e94ffc26f

Merge "Don't silently ignore arguments just because we don't take any." into dalvik-dev

am dad7a0ef: Merge "Add armv7-a-neon build target"

Merge commit 'dad7a0ef6e5f056f0842f03ff3b8285ef29e6f01' into dalvik-dev

* commit 'dad7a0ef6e5f056f0842f03ff3b8285ef29e6f01':
  Add armv7-a-neon build target

Merge "Add armv7-a-neon build target"

Don't silently ignore arguments just because we don't take any.

Change-Id: Ie1d97b1b8a8a87cc8298a6de417104db324e9b47

Merge "Tidy up java.lang.Math and java.lang.StrictMath." into dalvik-dev