summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* CharsetDecoderICU/CharsetEncoderICU should take arrayOffset into account.Elliott Hughes2009-11-035-10/+136
| | | | | | | | | | | | | CharsetDecoderICU and CharsetEncoderICU special-case array-backed ByteBuffers and CharBuffers for performance reasons, but they shouldn't assume that the backing array always has offset 0. An external user hit this while using the jAudioTagger library. Test cases from user submission: http://code.google.com/p/android/issues/detail?id=4237 See also: 2234697
* Remove more @KnownFailures now I've run tests.AllTests.Elliott Hughes2009-11-036-17/+0
| | | | A lunchtime run of the full suite turned up more.
* Hide JIT-specific constants when JIT is not configured.Ben Cheng2009-11-030-0/+0
|
* Merge change Ia61d31e2Android (Google) Code Review2009-11-036-31/+0
|\ | | | | | | | | * changes: Remove @KnownFailure from a bunch of tests that now pass.
| * Remove @KnownFailure from a bunch of tests that now pass.Elliott Hughes2009-11-036-31/+0
| | | | | | | | Looks like we got some of our Locales back.
* | Implement chaining up to the first 64 cases in a switch statement.Ben Cheng2009-11-030-0/+0
|/
* Back out inline string optimizations until cause of instability tracked down.Bill Buzbee2009-11-030-0/+0
|
* Ensure dalvikvm sets "user.name" to "root" for our tests.Elliott Hughes2009-11-021-0/+1
| | | | | | | | dalvikvm sets "user.name" to $USER, but that environment variable isn't set when we run the tests. I was looking at our DatagramSocketTest failure, and it turns out it's another instance of "root can do stuff other users can't". The typical way harmony tests check whether they're running as root is to compare "user.name" against "root", but from run-core-tests they see "".
* Merge change I22212019Android (Google) Code Review2009-11-022-266/+5
|\ | | | | | | | | * changes: Kill OSNetworkSystem.inheritedChannelImpl.
| * Kill OSNetworkSystem.inheritedChannelImpl.Elliott Hughes2009-11-022-266/+5
| | | | | | | | | | | | | | | | This code is broken (and more broken than upstream), and not obviously relevant to our platform. Let's make it clear that all this currently does is return null. The changes in the Java side bring us back in sync with upstream.
* | Merge change I57dd1032Android (Google) Code Review2009-11-0214-139/+516
|\ \ | | | | | | | | | | | | * changes: Supporting expected results for the jtreg test runner.
| * | Supporting expected results for the jtreg test runner.Jesse Wilson2009-11-0114-139/+516
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is necessary to suppress false positives when dalvik intentionally differs from the RI. This came up in the fix for issue 2224903. Also adding additional features to the jtreg runner to make it useful as an iterative development tool: - debugger support, to debug through running tests - verbose output, to aid in diagnosing hangs Also adding a few ".expected" files for known test failures. Hopefully this is scalable. Renaming Run to TestRun and extracting Result as a top-level enum.
* | | Merge change Ie098e2faAndroid (Google) Code Review2009-11-024-317/+319
|\ \ \ | | | | | | | | | | | | | | | | * changes: Fixing PipedWriters/PipedOutputStreams silent failures during close().
| * | | Fixing PipedWriters/PipedOutputStreams silent failures during close().Jesse Wilson2009-11-024-317/+319
| |/ / | | | | | | | | | See bug 2224903 and jtreg test java.io.BufferedWriter.Cleanup.
* | | Merge change I04f3b02bAndroid (Google) Code Review2009-11-020-0/+0
|\ \ \ | |_|/ |/| | | | | | | | * changes: Jit - optimized inline string compareto, indexof; fill_array_data bug fix
| * | Jit - optimized inline string compareto, indexof; fill_array_data bug fixBill Buzbee2009-11-020-0/+0
| | | | | | | | | | | | | | | Added flushAllRegs() prior to C handlers in preparation for upcoming support for holding live/dirty values in physical registers.
* | | Some more adjustments to deal with building for a BSD(-like) target.Dan Bornstein2009-11-023-18/+30
| | | | | | | | | | | | Change-Id: I5e33c3db160ae3af6e324492e0b6a7a0dfb9719b
* | | Merge change I318b1449Android (Google) Code Review2009-11-026-141/+215
|\ \ \ | | | | | | | | | | | | | | | | * changes: More OSNetworkSystem cleanup.
| * | | More OSNetworkSystem cleanup.Elliott Hughes2009-11-026-141/+215
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | valgrind didn't like what we were doing here, specifically the way we were setting ss_family to AF_INET if getsockopt IP_MULTICAST_IF didn't return an AF_INET address, and ignoring the fact that the rest of the sockaddr_storage doesn't necessarily correspond to an AF_INET address. Linux seems to return an AF_UNSPEC address (rather than INADDR_ANY) if we haven't set a multicast interface, so we have to do the conversion ourselves. More disturbingly (and unfixed by this patch) we sometimes see ss_family come back as 127 (when AF_MAX is 32 on our system). My guess is that the kernel is just handing back bad data we gave it earlier, but I'll come back to see about fixing multicast properly later. I've inlined socketAddressToString into osNetworkSystem_byteArrayToIpString, its only caller, because it's clearer that way now we've simplified the division of error handling between the two. I've brought getSocketLocalAddressImpl and getSocketLocalPortImpl in line with each other and changed them to at least log errors. I've also added a new test that fails, to demonstrate that this implementation is broken as designed. I've changed the name of JAVASOCKOPT_IP_MULTICAST_IF to match the Java constant. I've fixed the exception we throw if we're asked to convert a bad address family's address from code that can't throw IOException --- AssertionError doesn't have a String constructor. Really, we need to kill this code, so I've added a TODO. I've also made a real push to get rid of most of the SOCKERR_*-based error reporting. (I had a transient failure with a meaningless error message that sent me to the source to work out which errno it was actually trying to report.) I've changed setTTL to call setTimeToLive, and change the implementation of the latter to pass Integer rather than Byte to the native implementation, since that's what the native code needs anyway and native Byte support was only for TTL. Bug: 2225748
* | | Merge change I88e8376eAndroid (Google) Code Review2009-11-014-3/+23
|\ \ \ | | | | | | | | | | | | | | | | * changes: Update logging to r820767
| * | | Update logging to r820767Jesse Wilson2009-10-274-3/+23
| | | |
* | | | Merge change Id46d9a48Android (Google) Code Review2009-10-313-107/+242
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | * changes: Fixing 1 of the 4 BufferedReader test failures in bug 2224903.
| * | | Fixing 1 of the 4 BufferedReader test failures in bug 2224903.Jesse Wilson2009-10-303-107/+242
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL includes the following functional changes: - changing read to not clear the mark upon reading EOF - changing read(char[], int, int) to use the 'read directly from the source stream' shortcut when the mark has exceeded its limit. Previously we took the shortcut only when the mark was unset. And these nonfunctional changes - rewrote read(char[], int, int) dramatically. The new revision contains only one call to 'System.arrayCopy' and the related bookkeeping. Previously there was one call before the loop, and another call in the loop. - renamed markpos to mark - renamed marklimit to markLimit - renamed count to end (it isn't a count, it's a position) - simplifying conditions that used >= when > was impossible - reducing the number of field reads where convenient
* | | Merge change I474dc860Android (Google) Code Review2009-10-303-84/+3
|\ \ \ | | | | | | | | | | | | | | | | * changes: Rewrite NativeBN_twosCompFitsIntoBytes.
| * | | Rewrite NativeBN_twosCompFitsIntoBytes.Elliott Hughes2009-10-303-84/+3
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | valgrind complains about invalid 4-byte reads, caused by "case 8" in the BNInterface.c function I'm removing here, which assumed that if we're checking whether a BIGNUM fits in 8 bytes, it must require more than 4 bytes (and so accessing d[1] is acceptable). We can implement this in Java using the existing BigInteger.bitLength method (which may call down to native code, but that native code looks okay). Also remove a related commented-out method. Bugs: 2223213, 2225642
* | | Fixed typo introduced during comment reformatting.Bill Buzbee2009-10-300-0/+0
| | |
* | | Merge change Ib000d44fAndroid (Google) Code Review2009-10-300-0/+0
|\ \ \ | |/ / |/| | | | | | | | * changes: Major registor allocation rework - stage 1.
| * | Major registor allocation rework - stage 1.Bill Buzbee2009-10-300-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Direct usage of registers abstracted out. Live values tracked locally. Redundant loads and stores suppressed. Address of registers and register pairs unified w/ single "location" mechanism Register types inferred using existing dataflow analysis pass. Interim (i.e. Hack) mechanism for storing register liveness info. Rewrite TBD. Stubbed-out code for linear scan allocation (for loop and long traces) Moved optimistic lock check for monitor-enter/exit inline for Thumb2 Minor restructuring, renaming and general cleanup of codegen Renaming of enums to follow coding convention Formatting fixes introduced by the enum renaming Rewrite of RallocUtil.c and addition of linear scan to come in stage 2.
* | | Merge change I006c279bAndroid (Google) Code Review2009-10-293-54/+3
|\ \ \ | | | | | | | | | | | | | | | | * changes: Remove broken .so and fix bogus test.
| * | | Remove broken .so and fix bogus test.Elliott Hughes2009-10-293-54/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | valgrind doesn't like this because the .so *only* exists for x86 (contrary to the claim in the @AndroidOnly annotation) but it's broken there, returning a char* instead of a jstring. I'm not really sure what they were trying to test here, but let's assume it was that Runtime.getRuntime().load behaves like System.load, and fix the tests that way. Bug: 2223234
* | | | Merge change I4a356627Android (Google) Code Review2009-10-2912-177/+364
|\ \ \ \ | |/ / / |/| | | | | | | | | | | * changes: Change the jtreg tool to perform both building and running steps.
| * | | Change the jtreg tool to perform both building and running steps.Jesse Wilson2009-10-2812-177/+364
| | | |
* | | | Merge change I8fa1a78aAndroid (Google) Code Review2009-10-281-1/+1
|\ \ \ \ | |/ / / |/| | | | | | | | | | | * changes: Make run-core-tests with no arguments equivalent to tests.AllTests.
| * | | Make run-core-tests with no arguments equivalent to tests.AllTests.Elliott Hughes2009-10-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Previous behavior would run the same set of tests, but without going through the regular pre- and post-processing, and without initializing the command-line flags to their default values.
* | | | Merge change Id7694644Android (Google) Code Review2009-10-2811-0/+736
|\ \ \ \ | |/ / / |/| | | | | | | | | | | * changes: Script for compiling and dexing jtreg tests for Dalvik.
| * | | Script for compiling and dexing jtreg tests for Dalvik.Jesse Wilson2009-10-2811-0/+736
| | |/ | |/|
* | | Merge change I8c671e53Android (Google) Code Review2009-10-287-67/+131
|\ \ \ | |_|/ |/| | | | | | | | * changes: Implement ProcessBuilder.redirectErrorStream.
| * | Implement ProcessBuilder.redirectErrorStream.Elliott Hughes2009-10-287-67/+131
| | | | | | | | | | | | | | | | | | | | | Also simplify and correct the security to ensure that the user can't modify the command to be executed after the SecurityManager has approved it. Bug: 2180063
* | | Fix a few FindBugs warnings in code that isn't upstream.Elliott Hughes2009-10-274-9/+7
| |/ |/| | | | | Bugs: 2099642, 2099637
* | Remove OSNetworkSystem.sendStream and all use of ErrorCodeException.Elliott Hughes2009-10-277-189/+29
|/ | | | | | | | | | | | Removing OSNetworkSystem.sendStream brings us in line with harmony. I've also rewritten writeSocketImpl to not make a temporary copy of the data it's going to send. The weird code in writeSocketDirectImpl that threw ErrorCodeException turns out to be a very awkward way of returning zero to the only two callers that are paying attention. (Harmony actually calls the equivalent of this code every time it throws SocketException from native code, but these two callers were the only places that checked for the special case.)
* The officially sanctioned way to get memcpy() is to #include <string.h>.Dan Bornstein2009-10-261-0/+1
| | | | Change-Id: Ie157d9305df5f9b86a09ea2224666fd2c054cda4
* Merge change I4c4d050dAndroid (Google) Code Review2009-10-260-0/+0
|\ | | | | | | | | * changes: Clone TEMP_FAILURE_RETRY in JNIHelp.h, to make up for (take your pick) laggard libc implementations not keeping up with the times or (perhaps) agressively modern libc implementations too eagerly expanding upon the standard.
| * Clone TEMP_FAILURE_RETRY in JNIHelp.h, to make up for (take your pick)Dan Bornstein2009-10-260-0/+0
| | | | | | | | | | | | | | | | laggard libc implementations not keeping up with the times or (perhaps) agressively modern libc implementations too eagerly expanding upon the standard. Change-Id: I4c4d050d15a00f4e82f8beb2b9c386b7c652f495
* | Merge change Ibc04ef43Android (Google) Code Review2009-10-263-119/+5
|\ \ | |/ |/| | | | | * changes: Remove dead native org.apache.harmony.nio.AddressUtils code.
| * Remove dead native org.apache.harmony.nio.AddressUtils code.Elliott Hughes2009-10-263-119/+5
| | | | | | | | | | | | This code is dead at the moment, and were it ever to become live, I'm pretty sure we'd want to start again from scratch rather than do whatever the author of this had in mind.
* | Merge change I8a7a4c56Android (Google) Code Review2009-10-263-155/+326
|\ \ | |/ |/| | | | | * changes: Split libcore/Android.mk into two files, one for Java code and one for native code, and make each have a set of rules for building on the host.
| * Split libcore/Android.mk into two files, one for Java code and one forDan Bornstein2009-10-263-155/+326
| | | | | | | | | | | | | | | | | | native code, and make each have a set of rules for building on the host. I also tightened up how sub.mk processing works and documented it a little better. Change-Id: I8a7a4c5697b2f22c4d69941dba381d6452200911
* | Merge change Ibebb78b6Android (Google) Code Review2009-10-263-71/+50
|\ \ | | | | | | | | | | | | * changes: Switch ProcessManager and System over to C++.
| * | Switch ProcessManager and System over to C++.Elliott Hughes2009-10-233-71/+50
| | | | | | | | | | | | | | | Bug 2180063 will require changes to ProcessManager, so now's a good time to make the switch in this directory.
* | | DeleteLocalRef the result of GetObjectClass more consistently.Elliott Hughes2009-10-262-1/+14
| |/ |/| | | | | | | | | | | | | | | | | Should fix VM crashes in tests.java.sql.StressTest, which can now be re-enabled. I've also added a "throwoom" to the one place where this code calls malloc but doesn't check the result. Bug: 2213053
generated by cgit v1.1 at 2025-01-25 17:11:05 +0000