)]}'
{"swift/obj/updater.py":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"a54e63e868a3b9f6ab1af366d7a31e02d5ba5630","unresolved":true,"context_lines":[{"line_number":407,"context_line":"                self.stats.unlinks +\u003d 1"},{"line_number":408,"context_line":"                self.logger.increment(\u0027unlinks\u0027)"},{"line_number":409,"context_line":"                os.unlink(update_path)"},{"line_number":410,"context_line":"                try:"},{"line_number":411,"context_line":"                    # If this was the last async_pending in the directory,"},{"line_number":412,"context_line":"                    # then this will succeed. Otherwise, it\u0027ll fail, and"},{"line_number":413,"context_line":"                    # that\u0027s okay."},{"line_number":414,"context_line":"                    os.rmdir(os.path.dirname(update_path))"},{"line_number":415,"context_line":"                except OSError:"},{"line_number":416,"context_line":"                    pass"},{"line_number":417,"context_line":"            elif redirects:"},{"line_number":418,"context_line":"                # erase any previous successes"},{"line_number":419,"context_line":"                update.pop(\u0027successes\u0027, None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"20a8bd8e_41057faf","line":416,"range":{"start_line":410,"start_character":16,"end_line":416,"end_character":24},"updated":"2021-06-23 06:16:51.000000000","message":"NIT: if we\u0027re going to repeat this code, maybe it could be refactored to a method (even if it\u0027s in the method like do_update())","commit_id":"66ab1da99aa034a5a31d06fabf1adf0c4ee30f18"}]}
