)]}'
{"nova/network/neutronv2/api.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"4d39b13ecd9d0d939f6b53ef7f7f6c516dabfbb9","unresolved":false,"context_lines":[{"line_number":2885,"context_line":"            return []"},{"line_number":2886,"context_line":"        if not client:"},{"line_number":2887,"context_line":"            client \u003d get_client(context)"},{"line_number":2888,"context_line":"        search_opts \u003d {\u0027id\u0027: list(set([ip[\u0027subnet_id\u0027] for ip in fixed_ips]))}"},{"line_number":2889,"context_line":"        data \u003d client.list_subnets(**search_opts)"},{"line_number":2890,"context_line":"        ipam_subnets \u003d data.get(\u0027subnets\u0027, [])"},{"line_number":2891,"context_line":"        subnets \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_2893d350","line":2888,"range":{"start_line":2888,"start_character":38,"end_line":2888,"end_character":39},"updated":"2018-10-04 18:02:33.000000000","message":"nit: set() accepts a comp statement directly, so if you remove this pair of square brackets you can avoid constructing an extra list in memory.","commit_id":"5cf470d28b714a4f6811bb017624f6c74e876e70"},{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"7972181ac7ffaf8215fcc12ca33d29246c87b1bf","unresolved":false,"context_lines":[{"line_number":2885,"context_line":"            return []"},{"line_number":2886,"context_line":"        if not client:"},{"line_number":2887,"context_line":"            client \u003d get_client(context)"},{"line_number":2888,"context_line":"        search_opts \u003d {\u0027id\u0027: list(set([ip[\u0027subnet_id\u0027] for ip in fixed_ips]))}"},{"line_number":2889,"context_line":"        data \u003d client.list_subnets(**search_opts)"},{"line_number":2890,"context_line":"        ipam_subnets \u003d data.get(\u0027subnets\u0027, [])"},{"line_number":2891,"context_line":"        subnets \u003d []"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_e8215bff","line":2888,"range":{"start_line":2888,"start_character":38,"end_line":2888,"end_character":39},"in_reply_to":"3f79a3b5_2893d350","updated":"2018-10-04 18:06:17.000000000","message":"Cool, thank you. I had thought I tried that earlier and it didn\u0027t work. But indeed it works.","commit_id":"5cf470d28b714a4f6811bb017624f6c74e876e70"}]}
