)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"change_message_id":"a7b94417c112d3b727b1fe1abb81dca49b54c31c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b6d8f03d_1e1b7e3d","updated":"2025-11-16 18:53:03.000000000","message":"I don\u0027t like this one.  Looking at `buf_read_u16()`, this is really an unsigned type, so parsing this as int and checking for `len \u003c 0` is just wrong.\n\nSince there are not very many callers, I would tend to actually make it return a proper `uint16_t` - which is what `ntohs(ret)` produces anyway - and fix those callers that need to be able to deal with negative values (if any).","commit_id":"cba5a8be60c78f9736830d2a77cd11ff569cfa6b"},{"author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"change_message_id":"da2d54ed40421b03ddb557cc98c0f524ac9c8181","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"08551d2d_cacfa277","updated":"2025-11-16 19:12:23.000000000","message":"I take that back, we actually do need `-1` to signal `buf_read() error`.  Will do a new review...","commit_id":"cba5a8be60c78f9736830d2a77cd11ff569cfa6b"},{"author":{"_account_id":1000002,"name":"cron2","display_name":"Gert Doering","email":"gert@greenie.muc.de","username":"cron2"},"change_message_id":"435510d9cdfd52cc93b06c16d632864100962818","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5a9554a1_c15b6466","in_reply_to":"b6d8f03d_1e1b7e3d","updated":"2025-11-16 19:12:36.000000000","message":"Done","commit_id":"cba5a8be60c78f9736830d2a77cd11ff569cfa6b"}]}