)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1001964,"name":"Erhan Kurubas","display_name":"Erhan Kurubas","email":"erhan.kurubas@espressif.com","username":"erhankur"},"change_message_id":"71c6990769a37445759f87d5eae60a47bd66fab4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c78a7edd_9be8956b","updated":"2024-08-04 10:48:01.000000000","message":"And now we have a new warning.\nWARNING:LONG_LINE_STRING: line length of 136 exceeds 120 columns\n#163: FILE: src/flash/startup.tcl:104:\n+add_help_text program \"write an image to flash, address is only required for binary images. preverify, verify, reset, exit are optional\"","commit_id":"13f9f29fa8ba9ad5a73a3bcf56708c47988e2c96"},{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"803cb17254e1d4b9cc8593023620612297e00ab9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"96766fe3_1848cd4f","in_reply_to":"c78a7edd_9be8956b","updated":"2024-08-04 22:49:57.000000000","message":"Sure, it exceed the max line length.\n\nTCL allows to split the line, e.g.\n```\nadd_help_text program \"write an image to flash, address is only required for\\\nbinary images. preverify, verify, reset, exit are optional\"\n```\nbut it\u0027s not nice splitting the strings in this way as we cannot properly indent the second line.\n\nI think this is one of the cases where it would be useful adding in the commit message\n`Checkpatch-ignore: LONG_LINE_STRING`\nas explained in HACKING","commit_id":"13f9f29fa8ba9ad5a73a3bcf56708c47988e2c96"}]}