)]}'
{"third_party/go/patches/netlink-permhwaddr.patch":[{"author":{"_account_id":1000002,"name":"Serge Bazanski","display_name":"Serge","email":"serge@monogon.tech","username":"serge","avatars":[{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"6422f356b66aadfa00fc091b9d0dac578d93859d","unresolved":true,"context_lines":[{"line_number":23,"context_line":" \t\t\tbase.Group \u003d native.Uint32(attr.Value[0:4])"},{"line_number":24,"context_line":"+\t\tcase unix.IFLA_PERM_ADDRESS:"},{"line_number":25,"context_line":"+\t\t\tvar nonzero bool"},{"line_number":26,"context_line":"+\t\t\tfor _, b :\u003d range attr.Value {"},{"line_number":27,"context_line":"+\t\t\t\tif b !\u003d 0 {"},{"line_number":28,"context_line":"+\t\t\t\t\tnonzero \u003d true"},{"line_number":29,"context_line":"+\t\t\t\t}"},{"line_number":30,"context_line":"+\t\t\t}"},{"line_number":31,"context_line":"+\t\t\tif nonzero {"},{"line_number":32,"context_line":"+\t\t\t\tbase.PermHardwareAddr \u003d attr.Value[:]"},{"line_number":33,"context_line":"+\t\t\t}"},{"line_number":34,"context_line":" \t\t}"},{"line_number":35,"context_line":" \t}"},{"line_number":36,"context_line":" "}],"source_content_type":"text/x-diff","patch_set":1,"id":"ae4245ba_443afba1","line":33,"range":{"start_line":26,"start_character":4,"end_line":33,"end_character":5},"updated":"2023-03-06 17:51:14.000000000","message":"Guards! Guards!\n\n```suggestion\n+\t\t\tfor _, b :\u003d range attr.Value {\n+\t\t\t\tif b \u003d\u003d 0 {\n+\t\t\t\t\tcontinue\n+\t\t\t\t}\n+\t\t\t\tbase.PermHardwareAddr \u003d attr.Value[:]\n+\t\t\t\tbreak\n+\t\t\t}\n```","commit_id":"9a76063972bab90ffcf9309952a183b669477b1c"},{"author":{"_account_id":1000002,"name":"Serge Bazanski","display_name":"Serge","email":"serge@monogon.tech","username":"serge","avatars":[{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/52c41428b6369f2c02b9717425216f7d.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"623de36c7ee297bdc9926410a89040e480bf7749","unresolved":false,"context_lines":[{"line_number":23,"context_line":" \t\t\tbase.Group \u003d native.Uint32(attr.Value[0:4])"},{"line_number":24,"context_line":"+\t\tcase unix.IFLA_PERM_ADDRESS:"},{"line_number":25,"context_line":"+\t\t\tvar nonzero bool"},{"line_number":26,"context_line":"+\t\t\tfor _, b :\u003d range attr.Value {"},{"line_number":27,"context_line":"+\t\t\t\tif b !\u003d 0 {"},{"line_number":28,"context_line":"+\t\t\t\t\tnonzero \u003d true"},{"line_number":29,"context_line":"+\t\t\t\t}"},{"line_number":30,"context_line":"+\t\t\t}"},{"line_number":31,"context_line":"+\t\t\tif nonzero {"},{"line_number":32,"context_line":"+\t\t\t\tbase.PermHardwareAddr \u003d attr.Value[:]"},{"line_number":33,"context_line":"+\t\t\t}"},{"line_number":34,"context_line":" \t\t}"},{"line_number":35,"context_line":" \t}"},{"line_number":36,"context_line":" "}],"source_content_type":"text/x-diff","patch_set":1,"id":"6f315187_0745f28b","line":33,"range":{"start_line":26,"start_character":4,"end_line":33,"end_character":5},"in_reply_to":"4add8565_404a5776","updated":"2023-03-07 10:39:10.000000000","message":"Ack","commit_id":"9a76063972bab90ffcf9309952a183b669477b1c"},{"author":{"_account_id":1000001,"name":"Lorenz Brun","display_name":"Lorenz","email":"lorenz@monogon.tech","username":"lorenz","avatars":[{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/75c04f6e9881c24ee621fba80667eed8.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"change_message_id":"7d6f5b00849bdf33d113f4de8e66e368d0534a11","unresolved":true,"context_lines":[{"line_number":23,"context_line":" \t\t\tbase.Group \u003d native.Uint32(attr.Value[0:4])"},{"line_number":24,"context_line":"+\t\tcase unix.IFLA_PERM_ADDRESS:"},{"line_number":25,"context_line":"+\t\t\tvar nonzero bool"},{"line_number":26,"context_line":"+\t\t\tfor _, b :\u003d range attr.Value {"},{"line_number":27,"context_line":"+\t\t\t\tif b !\u003d 0 {"},{"line_number":28,"context_line":"+\t\t\t\t\tnonzero \u003d true"},{"line_number":29,"context_line":"+\t\t\t\t}"},{"line_number":30,"context_line":"+\t\t\t}"},{"line_number":31,"context_line":"+\t\t\tif nonzero {"},{"line_number":32,"context_line":"+\t\t\t\tbase.PermHardwareAddr \u003d attr.Value[:]"},{"line_number":33,"context_line":"+\t\t\t}"},{"line_number":34,"context_line":" \t\t}"},{"line_number":35,"context_line":" \t}"},{"line_number":36,"context_line":" "}],"source_content_type":"text/x-diff","patch_set":1,"id":"4add8565_404a5776","line":33,"range":{"start_line":26,"start_character":4,"end_line":33,"end_character":5},"in_reply_to":"ae4245ba_443afba1","updated":"2023-03-06 17:59:28.000000000","message":"As cool as this is, I don\u0027t think the tradeoff of compactness versus readablity is worth it.","commit_id":"9a76063972bab90ffcf9309952a183b669477b1c"}]}