On Wed, 20 Sep 2023 00:18:59 GMT, Alex Menkov wrote:
> I took some time to understand what this code for. I think the condition is
> not correct. it should be
>
> ```
> if (!allowOnlyIPv4) {
> ```
>
> allowOnlyIPv4 (reflects "java.net.preferIPv4Stack" sys.prop) means that only
> IPv4 connecti
On Tue, 19 Sep 2023 23:36:42 GMT, Chris Plummer wrote:
> The only time listenAddr is not set is when using AF_UNSPEC ("system").
@plummercj no it can also be unset if there are no listed addresses for the
preferred family e.g. you prefer v6 but they are all v4, or vice-versa. The new
code mak
On Tue, 19 Sep 2023 23:54:31 GMT, Liam Miller-Cushon wrote:
> > I think Alex has commented that this can be made more obvious with a check
> > for AF_UNSPEC.
>
> Thanks, yes, I have applied that suggestion.
>
> > > For 748, it's handling a situation where IPv4 isn't specifically
> > > preferr
On Tue, 19 Sep 2023 23:36:42 GMT, Chris Plummer wrote:
> I think Alex has commented that this can be made more obvious with a check
> for AF_UNSPEC.
Thanks, yes, I have applied that suggestion.
> > For 748, it's handling a situation where IPv4 isn't specifically preferred.
> > The implementat
On Tue, 19 Sep 2023 05:30:39 GMT, David Holmes wrote:
> Okay I see now that `ai` is not important. We won't have set `listenAddr` so
> we will take the first `addrInfo`. But this seems somewhat random behaviour
> as the first `addrInfo` could be either V4 or V6 (what controls the order?).
> Bu
On Tue, 19 Sep 2023 11:03:36 GMT, Alan Bateman wrote:
>> Liam Miller-Cushon has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Review feedback
>
> test/jdk/com/sun/jdi/JdwpNetProps.java line 75:
>
>> 73: .run(TestResult
On Tue, 19 Sep 2023 00:03:27 GMT, Liam Miller-Cushon wrote:
>> Please consider this fix for
>> [JDK-8313804](https://bugs.openjdk.org/browse/JDK-8313804), which adds
>> support to JDWP for `-Djava.net.preferIPv6Addresses=system`. Previously it
>> only handled `-Djava.net.preferIPv6Addresses=tr
On Tue, 19 Sep 2023 00:03:27 GMT, Liam Miller-Cushon wrote:
>> Please consider this fix for
>> [JDK-8313804](https://bugs.openjdk.org/browse/JDK-8313804), which adds
>> support to JDWP for `-Djava.net.preferIPv6Addresses=system`. Previously it
>> only handled `-Djava.net.preferIPv6Addresses=tr
On Tue, 19 Sep 2023 00:03:27 GMT, Liam Miller-Cushon wrote:
>> Please consider this fix for
>> [JDK-8313804](https://bugs.openjdk.org/browse/JDK-8313804), which adds
>> support to JDWP for `-Djava.net.preferIPv6Addresses=system`. Previously it
>> only handled `-Djava.net.preferIPv6Addresses=tr
> Please consider this fix for
> [JDK-8313804](https://bugs.openjdk.org/browse/JDK-8313804), which adds
> support to JDWP for `-Djava.net.preferIPv6Addresses=system`. Previously it
> only handled `-Djava.net.preferIPv6Addresses=true` and
> `-Djava.net.preferIPv6Addresses=false`.
Liam Miller-Cu
10 matches
Mail list logo
