)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"d8321402009b693ea6a4e5389a9975eed0878e1d","unresolved":true,"context_lines":[{"line_number":7,"context_line":"flash/nor/efm32: Fix coding style issues"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We going to do major changes in src/flash/nor/efm32.c. Let\u0027s fix all the"},{"line_number":10,"context_line":"warnigns reported by checkpatch.pl."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch should not bring any functional changes."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"49019724_f643ff6d","line":10,"updated":"2026-02-08 15:39:39.000000000","message":"Actually this patch does more than just fixing the issues reported by checkpatch. Anyway, I think it can be taken without need to update the commit message","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"e1f54b4e497aac4fb713d93772b43839c10e3806","unresolved":true,"context_lines":[{"line_number":7,"context_line":"flash/nor/efm32: Fix coding style issues"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We going to do major changes in src/flash/nor/efm32.c. Let\u0027s fix all the"},{"line_number":10,"context_line":"warnigns reported by checkpatch.pl."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch should not bring any functional changes."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"93ae2555_5d589717","line":10,"in_reply_to":"49019724_f643ff6d","updated":"2026-02-08 17:07:57.000000000","message":"Antonio, I added you to the reviewer list to make you aware that checkpatch.pl obviously fails in CODE_INDENT test. If we fix it, it would help us to catch wrong tab size errors. I\u0027ll send a patch. Test it on this one.","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"f2b3521c51cfb7eb564e7f7387680d61c0a08b95","unresolved":false,"context_lines":[{"line_number":7,"context_line":"flash/nor/efm32: Fix coding style issues"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We going to do major changes in src/flash/nor/efm32.c. Let\u0027s fix all the"},{"line_number":10,"context_line":"warnigns reported by checkpatch.pl."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"This patch should not bring any functional changes."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"eb5a6aca_436b0637","line":10,"in_reply_to":"93ae2555_5d589717","updated":"2026-04-26 05:06:03.000000000","message":"Done","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"5886931765e8f41b999d11f7cb5826035eaa8d26","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"57ea4e28_46522cd4","updated":"2026-02-05 09:10:03.000000000","message":"Thanks Jérôme!\n\nAll changes looks good but the indentation/spacing.\nPlease ensure your editor has properly set TAB size to 4.\nYou may need also to set it in the gerrit diff preferences.\nIt\u0027s strange that checkpatch does not complain on 4 and more\nspaces after TAB.","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"},{"author":{"_account_id":1002493,"name":"Jérôme Pouiller","email":"jerome.pouiller@silabs.com","username":"jerome-pouiller"},"change_message_id":"2e430264b87543af83984ed09e30d28811d198cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"54b8d385_e00d397f","in_reply_to":"3ceed620_0aa28dfa","updated":"2026-02-05 16:46:16.000000000","message":"Done","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"},{"author":{"_account_id":1002493,"name":"Jérôme Pouiller","email":"jerome.pouiller@silabs.com","username":"jerome-pouiller"},"change_message_id":"7e8c5edb4f27f1d90d30e5e6269fa588b1d72abe","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c6e6d7fe_33198bd3","in_reply_to":"57ea4e28_46522cd4","updated":"2026-02-05 09:55:20.000000000","message":"Would you mind if I fix this issue in a separated commit at the end of the PR?\nOtherwise, it will generate a lot of conflicts and I am afraid to introduce bugs.","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"b1c28636d70a4f52dcf85ee3db067c20804c191e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3ceed620_0aa28dfa","in_reply_to":"c6e6d7fe_33198bd3","updated":"2026-02-05 10:02:27.000000000","message":"Okay","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"}],"tools/scripts/checkpatch.pl":[{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"e1f54b4e497aac4fb713d93772b43839c10e3806","unresolved":true,"context_lines":[{"line_number":3878,"context_line":"# at the beginning of a line any tabs must come first and anything"},{"line_number":3879,"context_line":"# more than $tabsize must use tabs."},{"line_number":3880,"context_line":"\t\tif ($rawline \u003d~ /^\\+\\s* \\t\\s*\\S/ ||"},{"line_number":3881,"context_line":"\t\t    $rawline \u003d~ /^\\+\\s*        \\s*/) {"},{"line_number":3882,"context_line":"\t\t\tmy $herevet \u003d \"$here\\n\" . cat_vet($rawline) . \"\\n\";"},{"line_number":3883,"context_line":"\t\t\t$rpt_cleaners \u003d 1;"},{"line_number":3884,"context_line":"\t\t\tif (ERROR(\"CODE_INDENT\","}],"source_content_type":"text/x-perl","patch_set":3,"id":"cc6bf2ec_e29b9c35","line":3881,"range":{"start_line":3881,"start_character":25,"end_line":3881,"end_character":33},"updated":"2026-02-08 17:07:57.000000000","message":"Note the fixed 8 spaces regardless $tabsize.\nI reduced to `/^\\+\\s* {4}\\s*/` and many CODE_INDENT errors was detected in this patch. The code generating regexp from $tabsize submitted as\nhttps://review.openocd.org/c/openocd/+/9457","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"f2b3521c51cfb7eb564e7f7387680d61c0a08b95","unresolved":false,"context_lines":[{"line_number":3878,"context_line":"# at the beginning of a line any tabs must come first and anything"},{"line_number":3879,"context_line":"# more than $tabsize must use tabs."},{"line_number":3880,"context_line":"\t\tif ($rawline \u003d~ /^\\+\\s* \\t\\s*\\S/ ||"},{"line_number":3881,"context_line":"\t\t    $rawline \u003d~ /^\\+\\s*        \\s*/) {"},{"line_number":3882,"context_line":"\t\t\tmy $herevet \u003d \"$here\\n\" . cat_vet($rawline) . \"\\n\";"},{"line_number":3883,"context_line":"\t\t\t$rpt_cleaners \u003d 1;"},{"line_number":3884,"context_line":"\t\t\tif (ERROR(\"CODE_INDENT\","}],"source_content_type":"text/x-perl","patch_set":3,"id":"eccb3afc_c8ebb1ca","line":3881,"range":{"start_line":3881,"start_character":25,"end_line":3881,"end_character":33},"in_reply_to":"cc6bf2ec_e29b9c35","updated":"2026-04-26 05:06:03.000000000","message":"Done","commit_id":"167b1e01fa690265e4a1b27ebbf300dd484639c4"}]}