Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
pchilano created a review comment on a pull request on openjdk/jdk
I see your point. I removed this method and had the only caller just call set_owner_from_anonymous() and set_stack_locker(nullptr). There was one other caller in ObjectMonitor::complete_exit() but ...
pchilano created a review comment on a pull request on openjdk/jdk
We don't unmount the virtual thread here, we just temporarily change the thread identity. You could think of this method as switchIdentityToCarrierThread if that helps.
openjdk-notifier[bot] pushed 4 commits to pr/21565 openjdk/jdk
honkar-jdk created a review comment on a pull request on openjdk/jdk
Right, it wasn't intentionally but missed when removing `-Djava.security.manager=allow`. > Anyway, removing /othervm isn't strictly required because java/beans are run in othervm mode. Thank ...
openjdk-notifier[bot] pushed 1 commit to pr/20677 openjdk/jdk
- Enable riscv in CompressedClassPointersEncodingScheme test 434c681
seanjmullan created a review comment on a pull request on openjdk/jdk
This does not need to be handled in this PR. In the majority of changes, we have really tried hard to avoid making unrelated changes, but sometimes a few snuck in, like moving package imports or pe...
openjdk-notifier[bot] created a branch on openjdk/jdk
pr/21695 - JDK main-line development https://openjdk.org/projects/jdk
mlbridge[bot] created a comment on a pull request on openjdk/jdk
<!-- mlbridge webrev comment --> <!-- mlbridge webrev header --> ### Webrevs <!-- mlbridge webrev list --> * 00: [Full](https://webrevs.openjdk.org/?repo=jdk&pr=21695&range=00) ([08eda748](https:/...
openjdk[bot] created a comment on a pull request on openjdk/jdk
@coleenp The following label will be automatically applied to this pull request: - `hotspot` When this pull request is ready to be reviewed, an "RFR" email will be sent to the corresponding maili...
openjdk[bot] created a comment on a pull request on openjdk/jdk
❗ This change is not yet ready to be integrated. See the **Progress** checklist in the description for automated requirements. <!-- PullRequestBot merge is ready comment --> <!-- PullRequestBot pla...
bridgekeeper[bot] created a comment on a pull request on openjdk/jdk
<!-- Welcome message --> :wave: Welcome back coleenp! A progress list of the required criteria for merging this PR into `master` will be added to the body of your pull request. There are additiona...
coleenp opened a pull request on openjdk/jdk
8340212: -Xshare:off -XX:CompressedClassSpaceBaseAddress=0x40001000000 crashes on macos-aarch64
Added checks to verify that the given compressed class base or shared base address and shift given will be decodable for aarch64. This code is moved in PR https://github.com/openjdk/jdk/pull/20677...
magicus created a review on a pull request on openjdk/jdk
This PR disabled partial linking. Partial linking is not a bug. It is currently only enabled on clang/linux, but the long term goal is to enable it on all platforms. You need to explain in the J...
coleenp created a review on a pull request on openjdk/jdk
Even though the order isn't strictly important, having the convention to read one before the other avoids the situation where we wonder whether someone's already looked at this to work out whether ...
coleenp created a review on a pull request on openjdk/jdk
Even though the order isn't strictly important, having the convention to read one before the other avoids the situation where we wonder whether someone's already looked at this to work out whether ...
dean-long created a comment on a pull request on openjdk/jdk
If reading cxq first avoids the benign "race", then why not make that the preferred ordering convention? It sounds like we are not too concerned about avoiding this particular protocol state (if...
aivanov-jdk created a review comment on a pull request on openjdk/jdk
@honkar-jdk I'm inclined to leave it as because it's not the only method which doesn't have a blank line between `@param` and `@throw` in this file. If it's worth taking care of, we may submit a...
openjdk-notifier[bot] pushed 1 commit to pr/21043 openjdk/jdk
- Use more appropriate exception with the alert description 91fc0fd
valeriepeng created a comment on a pull request on openjdk/jdk
> If the upstream file hasn’t changed, that’s fine. I was just curious why the year included (for Ludovic) hasn’t been updated as well. Yup, same here. Since the upstream file is the same, I onl...
