On Mon, 4 Dec 2023 10:39:05 GMT, Darragh Clarke wrote:
> Updated tests to use vm.flagless as they already ignore Vm flags
This pull request has now been integrated.
Changeset: 62b7c5ea
Author: Darragh Clarke
URL:
https://git.openjdk.org/jdk/com
Updated tests to use vm.flagless as they already ignore Vm flags
-
Commit messages:
- ignore vm flags
Changes: https://git.openjdk.org/jdk/pull/16946/files
Webrev: https://webrevs.openjdk.org/?repo=jdk=16946=00
Issue: https://bugs.openjdk.org/browse/JDK-8319647
Stats: 14 lines
On Tue, 14 Nov 2023 16:41:31 GMT, Darragh Clarke wrote:
> Currently the `IsAlive` test occasionally times out.
>
> This PR changes the timeout from `10` to `25` which I think is an adequate
> increase based on the failures I've seen. Though I'd be happy to change it to
>
> Currently the `IsAlive` test occasionally times out.
>
> This PR changes the timeout from `10` to `25` which I think is an adequate
> increase based on the failures I've seen. Though I'd be happy to change it to
> another value if someone thinks `25` isn't ideal.
Darragh Cla
Currently the `IsAlive` test occasionally times out.
This PR changes the timeout from `10` to `25` which I think is an adequate
increase based on the failures I've seen. Though I'd be happy to change it to
another value if someone thinks `25` isn't ideal.
-
Commit messages:
-
On Tue, 16 May 2023 10:38:52 GMT, Darragh Clarke wrote:
> Updated instances of `toLowerCase` and `toUpperCase` in several net and io
> files to specify `Locale.ROOT` to ensure that case conversion issues don't
> occur,
>
> I didn't add any new tests but ran tier 1-3 with no is
On Fri, 19 May 2023 11:24:30 GMT, Michael McMahon wrote:
> Seems like a useful change and I can see how issues could arise if strings
> were stored somewhere after being upper/lower cased and then reused in a
> different locale.
>
> Is it correct to say that the assumption is these strings
On Wed, 17 May 2023 10:41:57 GMT, Daniel JeliĆski wrote:
>> Darragh Clarke has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Update
>> src/java.base/share/classes/sun/net/www/protocol/http/DigestAuthe
> Updated instances of `toLowerCase` and `toUpperCase` in several net and io
> files to specify `Locale.ROOT` to ensure that case conversion issues don't
> occur,
>
> I didn't add any new tests but ran tier 1-3 with no issues
Darragh Clarke has updated the pull request increme
Updated instances of `toLowerCase` and `toUpperCase` in several net and io
files to specify `Locale.ROOT` to ensure that case conversion issues don't
occur,
I didn't add any new tests but ran tier 1-3 with no issues
-
Commit messages:
- added some case conversions missed
On Fri, 7 Oct 2022 14:14:13 GMT, Darragh Clarke wrote:
> Added an int cast for the `timeoutLeft` assignment, looking through the code
> and other uses of the variable it seems to show no issues with lossy
> conversion. As part of this I removed the file to disable the conversion
Added an int cast for the `timeoutLeft` assignment, looking through the code
and other uses of the variable it seems to show no issues with lossy
conversion. As part of this I removed the file to disable the conversion warning
I tested this build and it doesn't give any warnings on this file
12 matches
Mail list logo