)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"1d4a5d3d073083fe9c99e2044278d1db6884ae86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2142db83_9bd2b85f","updated":"2021-10-02 05:07:35.000000000","message":"Hi,\nI would appreciate a review about this. Going to +1 as previous. This is just a bunch of cleanups\nRegards,\nNishanth Menon","commit_id":"20865f4f2a14c9c2f8949bbb26ae840f416439e5"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"9b3c6a19f81004991cf7c9cf09f1e2001c7043b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9f8d85e2_e4dd1069","updated":"2022-03-01 15:15:50.000000000","message":"Update to fixup foreach","commit_id":"7223eb725d6e7f8023c1e11e14756599b1743753"}],"tcl/target/ti_k3.cfg":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"9316f461ef2c57436a93646d614265e1d9a64489","unresolved":true,"context_lines":[{"line_number":198,"context_line":""},{"line_number":199,"context_line":"if { $_v8_smp_debug \u003d\u003d 0 } {"},{"line_number":200,"context_line":"\tproc $_armv8_up_cmd { args } {"},{"line_number":201,"context_line":"\t\tforeach { _core } [set args] {"},{"line_number":202,"context_line":"\t\t\ttargets $_core"},{"line_number":203,"context_line":"\t\t\t_cpu_no_smp_up"},{"line_number":204,"context_line":"\t\t}"}],"source_content_type":"text/x-ttcn-cfg","patch_set":5,"id":"a4205579_14580844","line":201,"updated":"2022-03-01 14:31:52.000000000","message":"Just for understanding, it\u0027s not a bug, no need to change the code...\nI vaguely remember that using [set args] was required by some old TCL version that does not support \u0027$args\u0027, but jimtcl should not need it.\nIs there a real need here for [set args]?","commit_id":"7c5acdec2c882b8f05c32b64489f462eef45cef7"},{"author":{"_account_id":1001678,"name":"Nishanth Menon","email":"nm@ti.com","username":"nmenon"},"change_message_id":"ed4836084684e677234a2bdfa8cabde9dade27f3","unresolved":false,"context_lines":[{"line_number":198,"context_line":""},{"line_number":199,"context_line":"if { $_v8_smp_debug \u003d\u003d 0 } {"},{"line_number":200,"context_line":"\tproc $_armv8_up_cmd { args } {"},{"line_number":201,"context_line":"\t\tforeach { _core } [set args] {"},{"line_number":202,"context_line":"\t\t\ttargets $_core"},{"line_number":203,"context_line":"\t\t\t_cpu_no_smp_up"},{"line_number":204,"context_line":"\t\t}"}],"source_content_type":"text/x-ttcn-cfg","patch_set":5,"id":"e0ed934a_de9681e3","line":201,"in_reply_to":"a4205579_14580844","updated":"2022-03-01 15:08:03.000000000","message":"I think I picked that from target/xilinx_zynqmp.cfg ;)\nforeach _core $args { Should do fine.\n\nI will update in a min","commit_id":"7c5acdec2c882b8f05c32b64489f462eef45cef7"}]}
