)]}'
{"swift/obj/mem_diskfile.py":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"2b8ea7b914fc66c7d9d511b5bebe535f685622e5","unresolved":false,"context_lines":[{"line_number":395,"context_line":"        :param timestamp: timestamp to compare with each file"},{"line_number":396,"context_line":"        \"\"\""},{"line_number":397,"context_line":"        fp, md \u003d self._filesystem.get_object(self._name)"},{"line_number":398,"context_line":"        if md and md[\u0027X-Timestamp\u0027] \u003c timestamp:"},{"line_number":399,"context_line":"            self._filesystem.del_object(self._name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_05644ae3","line":398,"updated":"2014-07-25 17:47:36.000000000","message":"I\u0027m not sure, looking at obj/server.py line 629, the timestamp passed in to delete() is a Timestamp object, so shouldn\u0027t this beTimestamp(md[\u0027X-Timestamp\u0027]) \u003c timestamp?\n\nIts confusing when comparing to the equivalent delete() in diskfile.py because that seems to treat the timestamp arg as if it were a string.","commit_id":"fced1324ec1953e145fe1e8bcf52acef73ef9db6"},{"author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"change_message_id":"108802114c69b12dc4fe89c9d9b4c65fffd31a69","unresolved":false,"context_lines":[{"line_number":395,"context_line":"        :param timestamp: timestamp to compare with each file"},{"line_number":396,"context_line":"        \"\"\""},{"line_number":397,"context_line":"        fp, md \u003d self._filesystem.get_object(self._name)"},{"line_number":398,"context_line":"        if md and md[\u0027X-Timestamp\u0027] \u003c timestamp:"},{"line_number":399,"context_line":"            self._filesystem.del_object(self._name)"}],"source_content_type":"text/x-python","patch_set":4,"id":"baada198_0591aa51","line":398,"in_reply_to":"baada198_05644ae3","updated":"2014-07-25 17:57:16.000000000","message":"removing the Timestamp was a merge mistake, thanks for catching. New patch coming soon...","commit_id":"fced1324ec1953e145fe1e8bcf52acef73ef9db6"}]}
