)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"5166f6fa837be5bf76848c0bd0d543c911956ca5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"079d0ebc_bbde9cda","updated":"2022-06-02 22:22:56.000000000","message":"looks correct, now. Thanks","commit_id":"f14ddd0b0f155d52e9a8192c8af0bd15a3f52fe1"}],"src/server/telnet_server.c":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"58829b671f1747916c6bf087726d0b9c0bab776b","unresolved":true,"context_lines":[{"line_number":626,"context_line":""},{"line_number":627,"context_line":"\t/* user command length */"},{"line_number":628,"context_line":"\tassert(usr_cmd_pos \u003c\u003d t_con-\u003eline_cursor);"},{"line_number":629,"context_line":"\tsize_t usr_cmd_len \u003d t_con-\u003eline_cursor - usr_cmd_pos;"},{"line_number":630,"context_line":""},{"line_number":631,"context_line":"\t/* optimize multiple spaces in the user command,"},{"line_number":632,"context_line":"\t * because info commands does not tolerate multiple spaces */"}],"source_content_type":"text/x-csrc","patch_set":1,"id":"77bfbfa4_618897cd","line":629,"range":{"start_line":629,"start_character":9,"end_line":629,"end_character":10},"updated":"2021-09-20 21:05:29.000000000","message":"scan-build is right, and with this patch I can get:\nopenocd: src/server/telnet_server.c:628: telnet_auto_complete: Assertion `usr_cmd_pos \u003c\u003d t_con-\u003eline_cursor\u0027 failed.\nAborted (core dumped)\n\nI have followed scan-build suggestion:\nopen telnet console\ntype \u0027;\u0027\nmove cursor back one position (go on the just typed \u0027;\u0027)\npress TAB to run autocomplete\nAbort!","commit_id":"b5b9073e7672317dc86c917dfe189323dc0b5154"},{"author":{"_account_id":1000863,"name":"Tarek BOCHKATI","email":"tarek.bouchkati@gmail.com","username":"BouchkatiTarek"},"change_message_id":"1748a0894a01008fd50a15f502bd8ec92b71f430","unresolved":true,"context_lines":[{"line_number":626,"context_line":""},{"line_number":627,"context_line":"\t/* user command length */"},{"line_number":628,"context_line":"\tassert(usr_cmd_pos \u003c\u003d t_con-\u003eline_cursor);"},{"line_number":629,"context_line":"\tsize_t usr_cmd_len \u003d t_con-\u003eline_cursor - usr_cmd_pos;"},{"line_number":630,"context_line":""},{"line_number":631,"context_line":"\t/* optimize multiple spaces in the user command,"},{"line_number":632,"context_line":"\t * because info commands does not tolerate multiple spaces */"}],"source_content_type":"text/x-csrc","patch_set":1,"id":"a28911fa_b1716734","line":629,"range":{"start_line":629,"start_character":9,"end_line":629,"end_character":10},"in_reply_to":"77bfbfa4_618897cd","updated":"2021-09-20 22:27:04.000000000","message":"correct, Thanks for the tip. sorry I was lazy to check scan-build report.\nNow I have tried stgh different, but I\u0027m quite surprised that the issue is still there.\nI will share the other fix, just to showcase it (I\u0027m feeiling tired) so I will check further tomorrow.","commit_id":"b5b9073e7672317dc86c917dfe189323dc0b5154"}]}