On Tue, 29 Nov 2022 12:44:12 GMT, Daniel Fuchs <dfu...@openjdk.org> wrote:

>> Johnny Lim has refreshed the contents of this pull request, and previous 
>> commits have been removed. The incremental views will show differences 
>> compared to the previous content of the PR. The pull request contains one 
>> new commit since the last revision:
>> 
>>   Remove duplicate space in ProtocolException message
>
> src/java.base/share/classes/sun/net/www/protocol/http/HttpURLConnection.java 
> line 1999:
> 
>> 1997:             } while (redirects < maxRedirects);
>> 1998: 
>> 1999:             throw new ProtocolException("Server redirected too many 
>> times ("+ redirects + ")");
> 
> There's a missing space between " and + ; In order to avoid long lines which 
> are a pain to review in side diffs I would suggest to reformat and add a 
> newline there, rather than a whitespace.

@dfuch Thanks for the feedback! I updated as you suggested.

-------------

PR: https://git.openjdk.org/jdk/pull/11002

Reply via email to