Re: RFR: 8273059: Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize [v3]

2021-08-30 Thread Daniel Fuchs
On Fri, 27 Aug 2021 16:11:49 GMT, Michael McMahon wrote: >> Hi, >> >> Could I get the following trivial change reviewed please? >> It removes a redundant call to Math.min(Integer.MAX_VALUE, ) >> >> Thanks, >> Michael. > > Michael McMahon has updated the pull request incrementally with one

Re: RFR: 8273059: Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize [v3]

2021-08-27 Thread Michael McMahon
> Hi, > > Could I get the following trivial change reviewed please? > It removes a redundant call to Math.min(Integer.MAX_VALUE, ) > > Thanks, > Michael. Michael McMahon has updated the pull request incrementally with one additional commit since the last revision: mistake in last commit

Re: RFR: 8273059: Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize [v2]

2021-08-27 Thread Daniel Fuchs
On Fri, 27 Aug 2021 13:28:51 GMT, Michael McMahon wrote: >> Hi, >> >> Could I get the following trivial change reviewed please? >> It removes a redundant call to Math.min(Integer.MAX_VALUE, ) >> >> Thanks, >> Michael. > > Michael McMahon has updated the pull request incrementally with one

Re: RFR: 8273059: Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize [v2]

2021-08-27 Thread Michael McMahon
> Hi, > > Could I get the following trivial change reviewed please? > It removes a redundant call to Math.min(Integer.MAX_VALUE, ) > > Thanks, > Michael. Michael McMahon has updated the pull request incrementally with one additional commit since the last revision: Removed second chunk of

Re: RFR: 8273059: Redundant Math.min call in Http2ClientImpl#getConnectionWindowSize

2021-08-27 Thread Daniel Fuchs
On Fri, 27 Aug 2021 12:01:25 GMT, Michael McMahon wrote: > Hi, > > Could I get the following trivial change reviewed please? > It removes a redundant call to Math.min(Integer.MAX_VALUE, ) > > Thanks, > Michael. Marked as reviewed by dfuchs (Reviewer). - PR: https://git.openjd