Issue #35 - printToString doesn't encode to bytes

[tonicsoft]

For any AbstractCharBasedFormatter, printToString can print directly to a String and not have to convert to bytes and back again. This avoids call to ByteArrayOutputStream.toString() at ProtobufFormatter:91 which uses JVM default encoding instead of ProtobufFormatter.defaultCharset. (I discovered this issue when trying to create JSON with a broad range of unicode characters on a machine where the default Charset did not cover the full Unicode range.)

[scr]

Please update https://github.com/bivas/protobuf-java-format/blob/master/RELEASE-NOTES.md for this fix in 1.5.

[scr]

Would you mind fixing the issue you pointed out in Issue #35 ?

https://github.com/bivas/protobuf-java-format/blob/master/src/main/java/com/googlecode/protobuf/format/ProtobufFormatter.java#L91

Basically,

Convert this to

return new String(out.getBytes(), defaultCharset);

[tonicsoft]

Sure - shall I create a new issue/PR etc or just tag onto this one?

[tonicsoft]

Done - just added change to this issue, found another bug too.

[scr]

So sorry, but recent RELEASE-NOTES change for #37 collide with yours… please resolve the conflicts and upload.

[tonicsoft]

Conflicts resolved. Should be ready for merging now.

[onesuper]

Is this PR still in progress now? Without this PR, We have to invoke print() method and convert a OutputStream to String manually

[luigi-riefolo]

any update on this?

[tonicsoft]

I am no longer working on this, apologies.

Please feel free to take it up.