Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread Iris Clark
On Thu, 4 Nov 2021 11:09:42 GMT, Magnus Ihse Bursie wrote: > I ran bin/blessed-modifier-order.sh on source owned by core-libs. This > scripts verifies that modifiers are in the "blessed" order, and fixes it > otherwise. I have manually checked the changes made by the script to make > sure

Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread Mandy Chung
On Thu, 4 Nov 2021 11:09:42 GMT, Magnus Ihse Bursie wrote: > I ran bin/blessed-modifier-order.sh on source owned by core-libs. This > scripts verifies that modifiers are in the "blessed" order, and fixes it > otherwise. I have manually checked the changes made by the script to make > sure

Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread Daniel Fuchs
On Thu, 4 Nov 2021 11:09:42 GMT, Magnus Ihse Bursie wrote: > I ran bin/blessed-modifier-order.sh on source owned by core-libs. This > scripts verifies that modifiers are in the "blessed" order, and fixes it > otherwise. I have manually checked the changes made by the script to make > sure

Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread Magnus Ihse Bursie
On Thu, 4 Nov 2021 12:30:40 GMT, Aleksei Efimov wrote: >> I can see that this PR changes java.naming. Another bug to change >> java.naming, JDK-8276552, was filed yesterday. Please check with its >> reporter and coordinate this effort if necessary. > >> I can see that this PR changes

Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread Aleksei Efimov
On Thu, 4 Nov 2021 11:31:50 GMT, Pavel Rappo wrote: > I can see that this PR changes java.naming. Another bug to change > java.naming, JDK-8276552, was filed yesterday. Please check with its reporter > and coordinate this effort if necessary. @magicus I'm ok with having `java.naming` changes

Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread Aleksei Efimov
On Thu, 4 Nov 2021 11:09:42 GMT, Magnus Ihse Bursie wrote: > I ran bin/blessed-modifier-order.sh on source owned by core-libs. This > scripts verifies that modifiers are in the "blessed" order, and fixes it > otherwise. I have manually checked the changes made by the script to make > sure

Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread David Holmes
On Thu, 4 Nov 2021 11:09:42 GMT, Magnus Ihse Bursie wrote: > I ran bin/blessed-modifier-order.sh on source owned by core-libs. This > scripts verifies that modifiers are in the "blessed" order, and fixes it > otherwise. I have manually checked the changes made by the script to make > sure

Re: RFR: 8276629: Use blessed modifier order in core-libs code

2021-11-04 Thread Pavel Rappo
On Thu, 4 Nov 2021 11:09:42 GMT, Magnus Ihse Bursie wrote: > I ran bin/blessed-modifier-order.sh on source owned by core-libs. This > scripts verifies that modifiers are in the "blessed" order, and fixes it > otherwise. I have manually checked the changes made by the script to make > sure