8353479: jcmd with streaming output breaks intendation

Alex Menkov · Alex Menkov · commit d4353626d55a · 2025-04-02T21:58:33.000Z
Reviewed-by: sspitsyn, jsjolen
diff --git a/src/hotspot/share/services/attachListener.cpp b/src/hotspot/share/services/attachListener.cpp
@@ -138,6 +138,7 @@ class attachStream : public bufferedStream {
     if (is_streaming()) {
       if (!_error) {
         _error = !_reply_writer->write_fully(str, (int)len);
+        update_position(str, len);
       }
     } else {
       bufferedStream::write(str, len);

Original file line number	Diff line number	Diff line change
`@@ -138,6 +138,7 @@ class attachStream : public bufferedStream {`
`138`	`138`	`if (is_streaming()) {`
`139`	`139`	`if (!_error) {`
`140`	`140`	`_error = !_reply_writer->write_fully(str, (int)len);`
	`141`	`+ update_position(str, len);`
`141`	`142`	`}`
`142`	`143`	`} else {`
`143`	`144`	`bufferedStream::write(str, len);`