[GitHub] [any23] HansBrende edited a comment on issue #147: ANY23-281 Build Policeman's Forbidden API Checker into Maven config

[GitBox <gi...@apache.org>]

HansBrende edited a comment on issue #147: ANY23-281 Build Policeman's Forbidden API Checker into Maven config
URL: https://github.com/apache/any23/pull/147#issuecomment-535333636
 
 
   I'll have to take a more detailed look at the individual locale and `String.format` errors to be sure, but at first glance, there are 3 options I can think of that might help Locale errors:
   
   Option 1. Use string concatenation instead of String.format where applicable
   Option 2. Use Locale.ROOT or Locale.ENGLISH where applicable
   Option 3. Prefer Character.toLowerCase() over String.toLowerCase() where applicable
   
   Charset errors:
   1. Prefer to construct PrintWriters from Writers rather than OutputStreams, where applicable
   2. Otherwise, use StandardCharsets.UTF_8 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services