Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-08 Thread Alan Bateman
On Fri, 24 Sep 2021 11:28:09 GMT, Masanori Yano wrote: > Could you please review the 8250678 bug fixes? > > The `parse` method of ModuleDescriptor.Version class behaves incorrectly when > the input string contains consecutive delimiters. > > The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-08 Thread Mandy Chung
On Fri, 24 Sep 2021 11:28:09 GMT, Masanori Yano wrote: > Could you please review the 8250678 bug fixes? > > The `parse` method of ModuleDescriptor.Version class behaves incorrectly when > the input string contains consecutive delimiters. > > The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-08 Thread Alan Bateman
On Fri, 5 Nov 2021 16:51:02 GMT, Mandy Chung wrote: > I would suggest to create a separate issue to follow up the spec > clarification and keep this PR to fix the implementation. > > The version parsing code is tricky. The fix is straight-forward, just moving > the check of the delimiters as

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-05 Thread Mandy Chung
On Fri, 24 Sep 2021 11:28:09 GMT, Masanori Yano wrote: > Could you please review the 8250678 bug fixes? > > The `parse` method of ModuleDescriptor.Version class behaves incorrectly when > the input string contains consecutive delimiters. > > The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-05 Thread Masanori Yano
On Fri, 24 Sep 2021 11:28:09 GMT, Masanori Yano wrote: > Could you please review the 8250678 bug fixes? > > The `parse` method of ModuleDescriptor.Version class behaves incorrectly when > the input string contains consecutive delimiters. > > The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-03 Thread Alan Bateman
On Tue, 2 Nov 2021 20:09:21 GMT, Mandy Chung wrote: > I reviewed the change. It is reasonable to fix the parsing of the pre-release > version and the build version be consistent with parsing of the version > number which currently skips consecutive delimiters. > > The spec is unclear on

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-02 Thread Mandy Chung
On Fri, 24 Sep 2021 11:28:09 GMT, Masanori Yano wrote: > Could you please review the 8250678 bug fixes? > > The `parse` method of ModuleDescriptor.Version class behaves incorrectly when > the input string contains consecutive delimiters. > > The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-11-01 Thread Masanori Yano
On Wed, 6 Oct 2021 18:40:36 GMT, Mark Reinhold wrote: >> @mbreinhold Could you comment on this pull request? > >> @mbreinhold Could you comment on this pull request? > > This is somewhat tricky code. I’ll take a look, but give me a few days. @mbreinhold @AlanBateman Are you aware of my

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-10-27 Thread Masanori Yano
On Wed, 6 Oct 2021 18:40:36 GMT, Mark Reinhold wrote: >> @mbreinhold Could you comment on this pull request? > >> @mbreinhold Could you comment on this pull request? > > This is somewhat tricky code. I’ll take a look, but give me a few days. @mbreinhold @AlanBateman Could you please reply to

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-10-21 Thread Masanori Yano
On Mon, 27 Sep 2021 08:22:02 GMT, Alan Bateman wrote: >> Could you please review the 8250678 bug fixes? >> >> The `parse` method of ModuleDescriptor.Version class behaves incorrectly >> when the input string contains consecutive delimiters. >> >> The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-10-18 Thread Masanori Yano
On Wed, 6 Oct 2021 18:40:36 GMT, Mark Reinhold wrote: >> @mbreinhold Could you comment on this pull request? > >> @mbreinhold Could you comment on this pull request? > > This is somewhat tricky code. I’ll take a look, but give me a few days. @mbreinhold (@AlanBateman ) Could you tell me how

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-10-13 Thread Masanori Yano
On Wed, 6 Oct 2021 18:40:36 GMT, Mark Reinhold wrote: >> @mbreinhold Could you comment on this pull request? > >> @mbreinhold Could you comment on this pull request? > > This is somewhat tricky code. I’ll take a look, but give me a few days. @mbreinhold I waited for about a week, but I haven't

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-10-06 Thread Mark Reinhold
On Thu, 30 Sep 2021 11:26:12 GMT, Masanori Yano wrote: > @mbreinhold Could you comment on this pull request? This is somewhat tricky code. I’ll take a look, but give me a few days. - PR: https://git.openjdk.java.net/jdk/pull/5679

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-10-06 Thread Masanori Yano
On Mon, 27 Sep 2021 08:22:02 GMT, Alan Bateman wrote: >> Could you please review the 8250678 bug fixes? >> >> The `parse` method of ModuleDescriptor.Version class behaves incorrectly >> when the input string contains consecutive delimiters. >> >> The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-09-30 Thread Masanori Yano
On Fri, 24 Sep 2021 11:28:09 GMT, Masanori Yano wrote: > Could you please review the 8250678 bug fixes? > > The `parse` method of ModuleDescriptor.Version class behaves incorrectly when > the input string contains consecutive delimiters. > > The `parse` method treats strings as three

Re: RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-09-27 Thread Alan Bateman
On Fri, 24 Sep 2021 11:28:09 GMT, Masanori Yano wrote: > Could you please review the 8250678 bug fixes? > > The `parse` method of ModuleDescriptor.Version class behaves incorrectly when > the input string contains consecutive delimiters. > > The `parse` method treats strings as three

RFR: 8250678: ModuleDescriptor.Version parsing treats empty segments inconsistently

2021-09-24 Thread Masanori Yano
Could you please review the 8250678 bug fixes? The `parse` method of ModuleDescriptor.Version class behaves incorrectly when the input string contains consecutive delimiters. The `parse` method treats strings as three sections, but the parsing method differs between the method for the version