)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"d493ab34f65b9c98fa084d994a84b72658c5ef1b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove workaround for bug #1219530"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Bug #1219530 has been fixed before Havana. We can remove this workaround."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ib0b1abf7d11627045be922f79aff6b80448ccbf4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAWn%2F%2F73E%3D","line":10,"updated":"2014-04-08 12:03:55.000000000","message":"This change needs a proper bug filed and then linked in this commit message.","commit_id":"c0fe785df461e86be8ccf7f839ebb197466c4522"}],"neutron/agent/linux/ovsdb_monitor.py":[{"author":{"_account_id":7183,"name":"Xu Han Peng","email":"xuhanp@linux.vnet.ibm.com","username":"xuhanp"},"change_message_id":"9c929221251ad775c3300059081c9a5149091779","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    def _read_stdout(self):"},{"line_number":42,"context_line":"        data \u003d self._process.stdout.readline()"},{"line_number":43,"context_line":"        if not data:"},{"line_number":44,"context_line":"            return"},{"line_number":45,"context_line":"        self._stdout_lines.put(data)"},{"line_number":46,"context_line":"        LOG.debug(_(\u0027Output received from ovsdb monitor: %s\u0027) % data)"},{"line_number":47,"context_line":"        return data"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2Fim8%3D","line":44,"updated":"2014-04-13 09:35:11.000000000","message":"nit: these two lines can be changed to:\nif data:\nand include line 45 to 47 in the if.","commit_id":"c0fe785df461e86be8ccf7f839ebb197466c4522"},{"author":{"_account_id":7183,"name":"Xu Han Peng","email":"xuhanp@linux.vnet.ibm.com","username":"xuhanp"},"change_message_id":"998dd9d863ec39d9bcdbe86d5dae0a2f55d8c6fa","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    def _read_stdout(self):"},{"line_number":42,"context_line":"        data \u003d self._process.stdout.readline()"},{"line_number":43,"context_line":"        if not data:"},{"line_number":44,"context_line":"            return"},{"line_number":45,"context_line":"        self._stdout_lines.put(data)"},{"line_number":46,"context_line":"        LOG.debug(_(\u0027Output received from ovsdb monitor: %s\u0027) % data)"},{"line_number":47,"context_line":"        return data"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FX4k%3D","line":44,"in_reply_to":"AAAAW3%2F%2FeK0%3D","updated":"2014-04-15 03:24:44.000000000","message":"OK. thanks for the reply.","commit_id":"c0fe785df461e86be8ccf7f839ebb197466c4522"},{"author":{"_account_id":708,"name":"Yuriy Taraday","email":"yuriy@taraday.nl","username":"yorik-sar"},"change_message_id":"39a5e1d8b684d9460089558fa2f0d0fa3cf7e9ea","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    def _read_stdout(self):"},{"line_number":42,"context_line":"        data \u003d self._process.stdout.readline()"},{"line_number":43,"context_line":"        if not data:"},{"line_number":44,"context_line":"            return"},{"line_number":45,"context_line":"        self._stdout_lines.put(data)"},{"line_number":46,"context_line":"        LOG.debug(_(\u0027Output received from ovsdb monitor: %s\u0027) % data)"},{"line_number":47,"context_line":"        return data"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAW3%2F%2FeK0%3D","line":44,"in_reply_to":"AAAAW3%2F%2Fim8%3D","updated":"2014-04-14 12:03:24.000000000","message":"I prefer to keep things this way (as they were). It clearly separates error handling (in if: branches) from primary control flow (without extra indent).","commit_id":"c0fe785df461e86be8ccf7f839ebb197466c4522"}]}