)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1002273,"name":"Richard Allen","email":"rsaxvc@gmail.com","username":"rsaxvc","status":"@BryghtLabs, @rsaxvc.net"},"change_message_id":"2280a0dd4e768c2349da13c637e786812490fc76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bcd29712_1be080c2","updated":"2026-05-21 00:18:57.000000000","message":"Looks good to me","commit_id":"dc9ec73349b432652b52f2db70e230aab1366bfe"}],"src/target/target.c":[{"author":{"_account_id":1002273,"name":"Richard Allen","email":"rsaxvc@gmail.com","username":"rsaxvc","status":"@BryghtLabs, @rsaxvc.net"},"change_message_id":"2280a0dd4e768c2349da13c637e786812490fc76","unresolved":false,"context_lines":[{"line_number":4331,"context_line":"\t\treturn retval;"},{"line_number":4332,"context_line":"\t}"},{"line_number":4333,"context_line":""},{"line_number":4334,"context_line":"\tif (with_range) {"},{"line_number":4335,"context_line":"\t\tuint32_t num_filtered_samples \u003d 0;"},{"line_number":4336,"context_line":"\t\tfor (uint32_t in \u003d 0; in \u003c num_of_samples; ++in) {"},{"line_number":4337,"context_line":"\t\t\tuint32_t sample \u003d samples[in];"}],"source_content_type":"text/x-csrc","patch_set":1,"id":"86d2a66f_97178750","side":"PARENT","line":4334,"range":{"start_line":4334,"start_character":1,"end_line":4334,"end_character":18},"updated":"2026-05-21 00:18:57.000000000","message":"This could just be `if(with_range \u0026\u0026 num_of_samples) {` if we wanted to keep only a single check, but the right-hand side looks good too, and I like the indication of why there were no samples post-filter.","commit_id":"103cde964855ab5693e15f3b0cefbdb10027ba0d"}]}