Did I miss something or is it replacing a string with the same string?

Le samedi 2 juin 2018, GitBox <g...@apache.org> a écrit :

> rmannibucau commented on a change in pull request #2: Upgrade to Eclipse
> Microprofile 1.1 and minor things
> URL: https://github.com/apache/geronimo-jwt-auth/pull/2#
> discussion_r192568348
>
>
>
>  ##########
>  File path: geronimo-jwt-auth-impl/src/main/java/org/apache/geronimo/
> microprofile/impl/jwtauth/cdi/GeronimoJwtAuthExtension.java
>  ##########
>  @@ -397,7 +397,7 @@ public Claims standard() {
>
>          @Override
>          public String toString() {
> -            return super.toString().replace(", ", ", ");
>
>  Review comment:
>    Why? If so then drop tostring ;)
>
> ----------------------------------------------------------------
> This is an automated message from the Apache Git Service.
> To respond to the message, please log on GitHub and use the
> URL above to go to the specific comment.
>
> For queries about this service, please contact Infrastructure at:
> us...@infra.apache.org
>
>
> With regards,
> Apache Git Services
>


-- 
Jean-Louis

Reply via email to