Skip to content

Commit

Permalink
Merge pull request #449 from bgould/printf-fix
Browse files Browse the repository at this point in the history
Fixed method signatures on format() methods to return PrintStream as in OpenJDK
  • Loading branch information
joshuawarner32 committed Jul 29, 2015
2 parents 38e66d8 + b766193 commit 1c039c5
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions classpath/java/io/PrintStream.java
Expand Up @@ -78,23 +78,25 @@ public void print(char[] s) {
print(String.valueOf(s));
}

public synchronized void printf(java.util.Locale locale, String format, Object... args) {
public synchronized PrintStream printf(java.util.Locale locale, String format, Object... args) {
// should this be cached in an instance variable??
final java.util.Formatter formatter = new java.util.Formatter(this);
formatter.format(locale, format, args);
return this;
}

public synchronized void printf(String format, Object... args) {
public synchronized PrintStream printf(String format, Object... args) {
final java.util.Formatter formatter = new java.util.Formatter(this);
formatter.format(format, args);
return this;
}

public void format(String format, Object... args) {
printf(format, args);
public PrintStream format(String format, Object... args) {
return printf(format, args);
}

public void format(java.util.Locale locale, String format, Object... args) {
printf(locale, format, args);
public PrintStream format(java.util.Locale locale, String format, Object... args) {
return printf(locale, format, args);
}

public synchronized void println(String s) {
Expand Down

0 comments on commit 1c039c5

Please sign in to comment.