)]}'
{"nova/service.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"aa6279bf93b6dc1f8a47da0ac04b88f0e0ee3986","unresolved":false,"context_lines":[{"line_number":241,"context_line":"        manager \u003d manager or SERVICE_MANAGERS.get(binary)"},{"line_number":242,"context_line":"        report_interval \u003d report_interval or CONF.report_interval"},{"line_number":243,"context_line":"        periodic_enable \u003d periodic_enable or CONF.periodic_enable"},{"line_number":244,"context_line":"        periodic_fuzzy_delay \u003d periodic_fuzzy_delay or CONF.periodic_fuzzy_delay"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"        debugger.init()"},{"line_number":247,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_023aa7e5","line":244,"range":{"start_line":244,"start_character":79,"end_line":244,"end_character":80},"updated":"2018-04-24 13:33:02.000000000","message":"Here\u0027s where pep8 is hitting you.","commit_id":"9696c20636c13b123a2cf1175f06c04517a2ec19"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"aa6279bf93b6dc1f8a47da0ac04b88f0e0ee3986","unresolved":false,"context_lines":[{"line_number":239,"context_line":"        binary \u003d binary or os.path.basename(sys.argv[0])"},{"line_number":240,"context_line":"        topic \u003d topic or binary.rpartition(\u0027nova-\u0027)[2]"},{"line_number":241,"context_line":"        manager \u003d manager or SERVICE_MANAGERS.get(binary)"},{"line_number":242,"context_line":"        report_interval \u003d report_interval or CONF.report_interval"},{"line_number":243,"context_line":"        periodic_enable \u003d periodic_enable or CONF.periodic_enable"},{"line_number":244,"context_line":"        periodic_fuzzy_delay \u003d periodic_fuzzy_delay or CONF.periodic_fuzzy_delay"},{"line_number":245,"context_line":""},{"line_number":246,"context_line":"        debugger.init()"},{"line_number":247,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9f6a8fd7_2216cb56","line":244,"range":{"start_line":242,"start_character":0,"end_line":244,"end_character":80},"updated":"2018-04-24 13:33:02.000000000","message":"These changes aren\u0027t quite right, because e.g. \"\" (empty string) or 0 will evaluate to False.  They would have to be more like:\n\n x \u003d x if x is not None else CONF.x","commit_id":"9696c20636c13b123a2cf1175f06c04517a2ec19"}]}