Permalink
Browse files

Merge pull request #281 from tkellogg/master

FizzStringReturner may not have copied all characters
  • Loading branch information...
emiln committed Feb 4, 2017
2 parents 4590711 + 16d9d00 commit cdfac751e51e18607b67d6ca3794a8d703863aa9
@@ -11,7 +11,8 @@ public String getReturnString() {
final StringBuilder myStringBuilder = new StringBuilder(
com.seriouscompany.business.java.fizzbuzz.packagenamingpackage.impl.Constants.FIZZ);
final String myString = myStringBuilder.toString();
return new String(myString);
final char[] myCharacters = myString.toCharArray();
return new String(myCharacters, 0, myCharacters.length);
}
}

0 comments on commit cdfac75

Please sign in to comment.