)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"14a9e58dc6da38c6c85f73a401c1a5451a2195a1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove an unused variable from tests.py"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The varialbe \u0027size\u0027 in TestFile.testMetadataNumberLimit is not used."},{"line_number":10,"context_line":"This patch remove the variable from the test."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I255a1dcee12bb6b8dec6ff26ed7edf93ab2acf64"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1aa78d24_2b3a3599","line":9,"updated":"2016-07-07 13:29:46.000000000","message":"nit: s/varialbe/variable/","commit_id":"2be1d6a77e9fcd4c92d487ba7116407ec5389723"}],"test/functional/tests.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"14a9e58dc6da38c6c85f73a401c1a5451a2195a1","unresolved":false,"context_lines":[{"line_number":1587,"context_line":"                key \u003d Utils.create_ascii_name()"},{"line_number":1588,"context_line":"                val \u003d Utils.create_name()"},{"line_number":1589,"context_line":""},{"line_number":1590,"context_line":"                if len(key) \u003e j:"},{"line_number":1591,"context_line":"                    key \u003d key[:j]"},{"line_number":1592,"context_line":"                    val \u003d val[:j]"},{"line_number":1593,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_8b66c9ea","line":1590,"updated":"2016-07-07 13:29:46.000000000","message":"hmmm, this single condition made sense when kay and val were both generated using create_name, but now they could have different lengths. However, val will be shorter than key, so looks ok, just strange.","commit_id":"2be1d6a77e9fcd4c92d487ba7116407ec5389723"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"14a9e58dc6da38c6c85f73a401c1a5451a2195a1","unresolved":false,"context_lines":[{"line_number":1592,"context_line":"                    val \u003d val[:j]"},{"line_number":1593,"context_line":""},{"line_number":1594,"context_line":"                metadata[key] \u003d val"},{"line_number":1595,"context_line":""},{"line_number":1596,"context_line":"            file_item \u003d self.env.container.file(Utils.create_name())"},{"line_number":1597,"context_line":"            file_item.metadata \u003d metadata"},{"line_number":1598,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"1aa78d24_805ce84b","line":1595,"updated":"2016-07-07 13:29:46.000000000","message":"alternatively, we could use the size variable for a sanity check:\n\n  self.assertLessEqual(size, size_limit)  # sanity check\n\nbut there\u0027s not really any reason to suspect that would ever fail, since the size is limited at lines 1590-1592","commit_id":"2be1d6a77e9fcd4c92d487ba7116407ec5389723"}]}