diff options
| author | Paul Duffin <paulduffin@google.com> | 2018-07-11 13:46:57 +0100 |
|---|---|---|
| committer | Paul Duffin <paulduffin@google.com> | 2018-07-12 13:01:57 +0100 |
| commit | 8ce3b14a85056cdaf8da5980c2a569bd3d28a704 (patch) | |
| tree | e0b45d2980c691c6561fc06e27012011cb85f49d /include/ScopedJavaUnicodeString.h | |
| parent | f2096fd9a7085946be28faae6d36f24b712eb998 (diff) | |
Inflater: Revert ensureOpen() changes to upstream 8u121-b13
The luni implementation of Inflater threw an IllegalStateException if
it was used after its end() method had been called but upstream throws
NullPointerException in that situation. The ojluni version was patched
in change b3c5785d in order to fix some failing tests. This change
reverts that patch and fixes the affected tests.
Throwing an NPE instead of an ISE could cause app compatibility issues
but it seems unlikely and in that case the application could simply
catch both.
An ISE is a more meaningful exception to throw but the benefit is minor
especially given that the message clearly indicates why it was thrown
and does not seem sufficient to justify carrying an additional patch.
Bug: 111061052
Test: flash, CtsLibcoreTestCases
Change-Id: I91ca999a7d3e81c571f960e01b5a2557a1676592
Diffstat (limited to 'include/ScopedJavaUnicodeString.h')
0 files changed, 0 insertions, 0 deletions