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