Use switch expressions in jdk.internal.net.http and java.net.http

[Kartik Ohri]

Hi Chris and Patrick,
It is the first time I am running the tier2 tests so I am not sure if I am
doing it correctly. I'll share what I did and my observations.

I executed make run-test-tier2 on my Ubuntu 20.10 machine and the tests
failed to build due to some warnings in a hostpot jtreg test. The output
was as follows:

* For target
support_test_hotspot_jtreg_native_support_exesigtest_exesigtest.o:
/home/lucifer/IdeaProjects/jdk/test/hotspot/jtreg/runtime/signal/exesigtest.c:
In function 'setSignalHandler':
/home/lucifer/IdeaProjects/jdk/test/hotspot/jtreg/runtime/signal/exesigtest.c:245:9:
error: 'sigset' is deprecated: Use the signal and sigprocmask functions
instead [-Werror=deprecated-declarations]
  245 |         sigset(signal_num, handler);
      |         ^~~~~~
In file included from
/home/lucifer/IdeaProjects/jdk/test/hotspot/jtreg/runtime/signal/exesigtest.c:25:
/usr/include/signal.h:353:23: note: declared here
  353 | extern __sighandler_t sigset (int __sig, __sighandler_t __disp)
__THROW
      |                       ^~~~~~
cc1: all warnings being treated as errors


However, when I ran the same command on my Ubuntu 20.04 machine, I got the
following output.
==============================
Test summary
==============================
   TEST                                              TOTAL  PASS  FAIL
ERROR
   jtreg:test/jdk:tier2                               3664  3664     0
0
   jtreg:test/langtools:tier2                           11    11     0
0
   jtreg:test/jaxp:tier2                               448   448     0
0
==============================
TEST SUCCESS

So, I think the tests passed on Ubuntu 20.04 but failed to execute on
Ubuntu 20.10.
I tried to find a reason for this and found
https://sourceware.org/pipermail/libc-alpha/2020-May/113971.html. I think
sigset got deprecated after Ubuntu 20.04 but so that is probably a
different issue and not related to this patch.

I do not have access to any other machine so I could not run the tests on
Windows, macOS and so on.

Thanks.

Regards,
Kartik.

On Mon, Nov 23, 2020 at 11:37 PM Patrick Concannon <
patrick.concannon at oracle.com> wrote:

> Hi Kartik,
>
> Thanks for submitting the patch. Once you’ve run the tier2 tests, I’d be
> happy to sponsor it for you.
>
> -Patrick
>
> > On 23 Nov 2020, at 09:09, Chris Hegarty <chris.hegarty at oracle.com>
> wrote:
> >
> > Hi Kartik,
> >
> >> On 21 Nov 2020, at 12:01, Kartik Ohri <kartikohri13 at gmail.com> wrote:
> >>
> >> Hi!
> >> I would like to submit this patch
> https://github.com/openjdk/jdk/pull/1364 with the rationale to improve
> the readability of the code. Can someone please take a look at it and
> create a public issue if the patch is OK to be included ?
> >
> > This certainly seems like a reasonable thing to do.
> >
> > Can you please run tier2 testing, since it contains the tests for the
> HTTP Client.
> >
> > Thanks,
> > -Chris.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/net-dev/attachments/20201125/0fbd96f3/attachment-0001.htm>