)]}'
{"novaclient/utils.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"1f8fa3c34012b5fdcc8b1737e090a88adecb4162","unresolved":false,"context_lines":[{"line_number":99,"context_line":"                    data \u003d \u0027-\u0027"},{"line_number":100,"context_line":"                if (data and isinstance(data, six.string_types)"},{"line_number":101,"context_line":"                        and \"\\r\" in data):"},{"line_number":102,"context_line":"                    data \u003d data.replace(\"\\r\", \"\\t\")"},{"line_number":103,"context_line":"                row.append(data)"},{"line_number":104,"context_line":"        pt.add_row(row)"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_4cdab64d","line":102,"updated":"2015-07-22 19:15:11.000000000","message":"Why \\r?  AFAIK, only macs use that to terminate a line; for the rest of the world, it stands in for a carriage return.  I would have expected \\n, optionally preceded by \\r.  Further, why replace it with \\t?  That wouldn\u0027t necessarily force a line break at that location, particularly if \\r was unsuccessful at that too…","commit_id":"1cf1b1eb87de9df62af70eda5bc6b36e97125cf8"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"4d9f90a20b90ed65284d3279aa4ac16fccb4c115","unresolved":false,"context_lines":[{"line_number":99,"context_line":"                    data \u003d \u0027-\u0027"},{"line_number":100,"context_line":"                if (data and isinstance(data, six.string_types)"},{"line_number":101,"context_line":"                        and \"\\r\" in data):"},{"line_number":102,"context_line":"                    data \u003d data.replace(\"\\r\", \"\\t\")"},{"line_number":103,"context_line":"                row.append(data)"},{"line_number":104,"context_line":"        pt.add_row(row)"},{"line_number":105,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_39251b08","line":102,"in_reply_to":"3a50d1a3_4cdab64d","updated":"2015-07-23 01:13:33.000000000","message":"\u0027\\r\u0027 will break the table, an example: http://paste.openstack.org/show/401954/, so i think it should be replaced,  hm, you are right we shouldn\u0027t replace it with \u0027\\t\u0027, remove \u0027\\r\u0027 would work here. I\u0027ll upload a new patch soon :) thanks","commit_id":"1cf1b1eb87de9df62af70eda5bc6b36e97125cf8"}]}