)]}'
{"cinderclient/utils.py":[{"author":{"_account_id":17275,"name":"Nate Potter","email":"nathaniel.potter@intel.com","username":"npotter"},"change_message_id":"a4c7a675d1a64ac27b2b4bc79f69cde3eb8cb7d4","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        if r[0] in formatters:"},{"line_number":199,"context_line":"            r[1] \u003d unicode_key_value_to_string(r[1])"},{"line_number":200,"context_line":"        if r[0] in bigdicts:"},{"line_number":201,"context_line":"            r[1] \u003d \u0027\\n    \u0027.join(textwrap.wrap(str(r[1])))"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"        if isinstance(r[1], six.string_types) and \"\\r\" in r[1]:"},{"line_number":204,"context_line":"            r[1] \u003d r[1].replace(\"\\r\", \" \")"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_37a61dc1","line":201,"range":{"start_line":201,"start_character":22,"end_line":201,"end_character":26},"updated":"2016-06-30 22:33:55.000000000","message":"What\u0027s this extra space here for, is it necessary?","commit_id":"ecd097d704421cd9f3ce45d02a2509064ba44964"},{"author":{"_account_id":22314,"name":"Ryan Selden","email":"seldenr@icloud.com","username":"seldenr"},"change_message_id":"5547715472706b070b6baf214ead0a4c5bbe4585","unresolved":false,"context_lines":[{"line_number":198,"context_line":"        if r[0] in formatters:"},{"line_number":199,"context_line":"            r[1] \u003d unicode_key_value_to_string(r[1])"},{"line_number":200,"context_line":"        if r[0] in bigdicts:"},{"line_number":201,"context_line":"            r[1] \u003d \u0027\\n    \u0027.join(textwrap.wrap(str(r[1])))"},{"line_number":202,"context_line":""},{"line_number":203,"context_line":"        if isinstance(r[1], six.string_types) and \"\\r\" in r[1]:"},{"line_number":204,"context_line":"            r[1] \u003d r[1].replace(\"\\r\", \" \")"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_f244a347","line":201,"range":{"start_line":201,"start_character":22,"end_line":201,"end_character":26},"in_reply_to":"3aaa91ec_37a61dc1","updated":"2016-06-30 22:58:05.000000000","message":"I added it to give an indent to all but the first line of a multi-line parameter. Just another eye-hint that it\u0027s not a separate value. I think it\u0027s helpful but not 100% necessary.","commit_id":"ecd097d704421cd9f3ce45d02a2509064ba44964"},{"author":{"_account_id":23083,"name":"TommyLike","email":"tommylikehu@gmail.com","username":"TommyLike"},"change_message_id":"ec3534f0867e4c7ef8417a4c0092b18c04b8f010","unresolved":false,"context_lines":[{"line_number":199,"context_line":"        if r[0] in formatters:"},{"line_number":200,"context_line":"            r[1] \u003d unicode_key_value_to_string(r[1])"},{"line_number":201,"context_line":"        if r[0] in bigdicts:"},{"line_number":202,"context_line":"            r[1] \u003d \u0027\\n    \u0027.join(textwrap.wrap(str(r[1])))"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"        if isinstance(r[1], six.string_types) and \"\\r\" in r[1]:"},{"line_number":205,"context_line":"            r[1] \u003d r[1].replace(\"\\r\", \" \")"}],"source_content_type":"text/x-python","patch_set":6,"id":"3ac371cc_23512437","line":202,"range":{"start_line":202,"start_character":11,"end_line":202,"end_character":57},"updated":"2016-08-17 02:21:24.000000000","message":"accroding to the document [textwarp](https://docs.python.org/2/library/textwrap.html)\n\nwhy not use **textwrap.fill()**","commit_id":"d312e474a615a3ea88aef4048ba874ec60d373c4"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"5de200283899ae1f544e6f98be9c03a738f1d12e","unresolved":false,"context_lines":[{"line_number":199,"context_line":"        if r[0] in formatters:"},{"line_number":200,"context_line":"            r[1] \u003d unicode_key_value_to_string(r[1])"},{"line_number":201,"context_line":"        if r[0] in bigdicts:"},{"line_number":202,"context_line":"            r[1] \u003d \u0027\\n    \u0027.join(textwrap.wrap(str(r[1])))"},{"line_number":203,"context_line":""},{"line_number":204,"context_line":"        if isinstance(r[1], six.string_types) and \"\\r\" in r[1]:"},{"line_number":205,"context_line":"            r[1] \u003d r[1].replace(\"\\r\", \" \")"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_a526ebcb","line":202,"range":{"start_line":202,"start_character":11,"end_line":202,"end_character":57},"in_reply_to":"3ac371cc_23512437","updated":"2016-09-30 09:58:15.000000000","message":"-1: str should be replaced with six.text_type\n\nnit: fill method seems like a cleaner option","commit_id":"d312e474a615a3ea88aef4048ba874ec60d373c4"}],"cinderclient/v3/shell.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"5de200283899ae1f544e6f98be9c03a738f1d12e","unresolved":false,"context_lines":[{"line_number":295,"context_line":""},{"line_number":296,"context_line":"    if \u0027attachments\u0027 in info:"},{"line_number":297,"context_line":"        for attachment in info[\u0027attachments\u0027]:"},{"line_number":298,"context_line":"            for key in [\u0027id\u0027, \u0027volume_id\u0027]:"},{"line_number":299,"context_line":"                attachment.pop(key, None)"},{"line_number":300,"context_line":"    info.pop(\u0027links\u0027, None)"},{"line_number":301,"context_line":"    utils.print_dict(info,"}],"source_content_type":"text/x-python","patch_set":6,"id":"3a98d1f4_6536a353","line":298,"range":{"start_line":298,"start_character":23,"end_line":298,"end_character":43},"updated":"2016-09-30 09:58:15.000000000","message":"nit: It\u0027s better to use a tuple","commit_id":"d312e474a615a3ea88aef4048ba874ec60d373c4"}]}