)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000021,"name":"Antonio Borneo","email":"borneo.antonio@gmail.com","username":"borneoa"},"change_message_id":"c8f401e00db3100d19aa419bdefb2c52b85123b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"10d2fac0_4df048e1","updated":"2024-09-14 12:22:23.000000000","message":"`gdb_breakpoint_watchpoint_packet()` is a static function that is called only once, as an helper in the giant switch/case of packet decoding.\n\nI think that adding the `assert()` is a kind of over-engineering that also reduces the readability because it triggers the question \"why we check it here?\", while a comment\n`/* packet[0] \u003d\u003d \u0027z\u0027 */`\nwould have been an improvement as a reminder of what is the `else` case.\n\nAnyway, that looks line my personal fight against `assert()`, already lost!","commit_id":"49b2a71e14c268d2857566531e1578a0a48f05cc"}]}