)]}'
{"tools/scripts/camelcase.txt":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"2b637ca4399de8835a9ade41f850bd23025f0b6a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# format types from inttypes.h"},{"line_number":2,"context_line":"(PRI|SCN)[diouxX](MAX|PTR|((FAST|LEAST|)(8|16|32|64)))"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"# from libusb.h"}],"source_content_type":"text/plain","patch_set":1,"id":"aedf27f1_1595d435","line":1,"updated":"2021-04-22 08:53:09.000000000","message":"Ooops, forgot the SPDX header, and checkpatch did not complained! To be checked.","commit_id":"27fc25c9a731196ad66df07b4ecaccd864d56c42"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"11f03e9b705a81b50ac798961c2bcfdb21391052","unresolved":false,"context_lines":[{"line_number":83,"context_line":"lpSecurityDescriptor"},{"line_number":84,"context_line":"nLength"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"# OpenOCD camelcase that should be removed but require wide rework"},{"line_number":87,"context_line":"ARMV7M_xPSR"},{"line_number":88,"context_line":"ARMV8_MRS_xPSR_T1"},{"line_number":89,"context_line":"ARMV8_MSR_GP_xPSR_T1"}],"source_content_type":"text/plain","patch_set":1,"id":"aedf27f1_f57cd01d","line":86,"range":{"start_line":86,"start_character":10,"end_line":86,"end_character":19},"updated":"2021-04-22 05:14:59.000000000","message":"I think we should not call these \u0027camelcase\u0027. https://en.wikipedia.org/wiki/Camel_case defines the camelcase as phrases without spaces or punctuation, indicating the separation of words with a single capitalized letter. Let\u0027s say that some commonly used mixed-case identifiers can justify the exception. And yes, there is no point having FPv4_SP if we have ARMV7M_...","commit_id":"27fc25c9a731196ad66df07b4ecaccd864d56c42"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"73fb4f1626a08f959761fee84f270c7485e69b5f","unresolved":false,"context_lines":[{"line_number":83,"context_line":"lpSecurityDescriptor"},{"line_number":84,"context_line":"nLength"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"# OpenOCD camelcase that should be removed but require wide rework"},{"line_number":87,"context_line":"ARMV7M_xPSR"},{"line_number":88,"context_line":"ARMV8_MRS_xPSR_T1"},{"line_number":89,"context_line":"ARMV8_MSR_GP_xPSR_T1"}],"source_content_type":"text/plain","patch_set":1,"id":"aedf27f1_b52f289c","line":86,"range":{"start_line":86,"start_character":10,"end_line":86,"end_character":19},"in_reply_to":"aedf27f1_7590e046","updated":"2021-04-24 20:03:18.000000000","message":"Done","commit_id":"27fc25c9a731196ad66df07b4ecaccd864d56c42"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"2b637ca4399de8835a9ade41f850bd23025f0b6a","unresolved":false,"context_lines":[{"line_number":83,"context_line":"lpSecurityDescriptor"},{"line_number":84,"context_line":"nLength"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"# OpenOCD camelcase that should be removed but require wide rework"},{"line_number":87,"context_line":"ARMV7M_xPSR"},{"line_number":88,"context_line":"ARMV8_MRS_xPSR_T1"},{"line_number":89,"context_line":"ARMV8_MSR_GP_xPSR_T1"}],"source_content_type":"text/plain","patch_set":1,"id":"aedf27f1_7590e046","line":86,"range":{"start_line":86,"start_character":10,"end_line":86,"end_character":19},"in_reply_to":"aedf27f1_f57cd01d","updated":"2021-04-22 08:53:09.000000000","message":"agree, I will rephrase it and maybe add a longer explanation in the header of this file.","commit_id":"27fc25c9a731196ad66df07b4ecaccd864d56c42"}],"tools/scripts/checkpatch.pl":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"bdc27c3861b51ebb41f8fd993d8bb54dfd3c8464","unresolved":false,"context_lines":[{"line_number":5582,"context_line":"#CamelCase"},{"line_number":5583,"context_line":"\t\t\t# OpenOCD specific: use list of CamelCase names"},{"line_number":5584,"context_line":"\t\t\tif (defined($camelcase_names) \u0026\u0026"},{"line_number":5585,"context_line":"\t\t\t    $var \u003d~ /\\b($camelcase_names)\\b/) {"},{"line_number":5586,"context_line":"\t\t\t\tnext;"},{"line_number":5587,"context_line":"\t\t\t}"},{"line_number":5588,"context_line":""}],"source_content_type":"text/x-perl","patch_set":1,"id":"aedf27f1_d59e6c59","line":5585,"updated":"2021-04-22 08:57:56.000000000","message":"Actually this should be /^($camelcase_names)$/\nbut I found that using the \u0027correct\u0027 version on Tarek\u0027s patch for autocomplete, it fails at this line\n499: struct command *cmd \u003d jim_cmd-\u003eu.native.privData;\nAfter debug, I found that here $var is NOT \"privData\" but it\u0027s \"jim_cmd-\u003eu.native.privData\" !\nI will keep the patch as is, but I will add an additional comment here","commit_id":"27fc25c9a731196ad66df07b4ecaccd864d56c42"}]}