[OpenOCD-devel] [PATCH]: 95f5730 target_request: replace command_print() with command_output_text()

[<ge...@op...>]

This is an automated email from Gerrit.

Antonio Borneo (bor...@gm...) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/5051

-- gerrit

commit 95f57309dc8c7e032cd7987f5bf304705b2d4b3d
Author: Antonio Borneo <bor...@gm...>
Date:   Wed Apr 3 10:14:29 2019 +0200

    target_request: replace command_print() with command_output_text()
    
    To prepare for handling TCL return values consistently, all calls
    to command_print/command_print_sameline should be ready to switch
    to CMD as first parameter.
    
    When dumping traces or semihosting text, the command that enabled
    it is already completed, so CMD is not a valid value.
    Plus, we want the dump to be printed in the command context where
    the enable command was executed.
    
    Replace command_print() with command_output_text().
    
    Change-Id: I297a2b799a914233499641062407cf6e8b9fe012
    Signed-off-by: Antonio Borneo <bor...@gm...>

diff --git a/src/target/target_request.c b/src/target/target_request.c
index 6ca204b..2df2958 100644
--- a/src/target/target_request.c
+++ b/src/target/target_request.c
@@ -56,7 +56,7 @@ static int target_asciimsg(struct target *target, uint32_t length)
 	LOG_DEBUG("%s", msg);
 
 	while (c) {
-		command_print(c->cmd_ctx, "%s", msg);
+		command_output_text(c->cmd_ctx, msg);
 		c = c->next;
 	}
 
@@ -100,7 +100,7 @@ static int target_hexmsg(struct target *target, int size, uint32_t length)
 			LOG_DEBUG("%s", line);
 
 			while (c) {
-				command_print(c->cmd_ctx, "%s", line);
+				command_output_text(c->cmd_ctx, line);
 				c = c->next;
 			}
 			c = target->dbgmsg;

--