)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9555,"name":"Matthew Booth","email":"mbooth@redhat.com","username":"MatthewBooth"},"change_message_id":"4fb8ed1231e683c03fe6bbb9f7e97ebede494116","unresolved":false,"context_lines":[{"line_number":9,"context_line":"There are quite a number of use cases where we construct additional"},{"line_number":10,"context_line":"datastore paths out of one representing some ancestor directory,"},{"line_number":11,"context_line":"so implemented a join() method that allows for just that."},{"line_number":12,"context_line":"It is useful to define equality operator when comparing two DatstorePath"},{"line_number":13,"context_line":"objects, so added that as well."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"partial blueprint vmware-spawn-refactor"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"baada198_794cd59e","line":12,"updated":"2014-07-04 13:04:58.000000000","message":"nit: DatastorePath","commit_id":"ea8270e32509d1b6c7b084dfdca2be46c995e3ff"}],"nova/virt/vmwareapi/ds_util.py":[{"author":{"_account_id":9555,"name":"Matthew Booth","email":"mbooth@redhat.com","username":"MatthewBooth"},"change_message_id":"4fb8ed1231e683c03fe6bbb9f7e97ebede494116","unresolved":false,"context_lines":[{"line_number":141,"context_line":"        return self._rel_path"},{"line_number":142,"context_line":""},{"line_number":143,"context_line":"    def join(self, *paths):"},{"line_number":144,"context_line":"        if paths:"},{"line_number":145,"context_line":"            if None in paths:"},{"line_number":146,"context_line":"                raise ValueError(_(\"path component cannot be None\"))"},{"line_number":147,"context_line":"            return DatastorePath(self.datastore,"}],"source_content_type":"text/x-python","patch_set":3,"id":"baada198_d97081de","line":144,"updated":"2014-07-04 13:04:58.000000000","message":"nit: You can safely omit this if statement, as the code below is correct without it. Omitting it would also have the advantage that the function *always* returns a copy, rather than sometimes a copy and sometimes not.","commit_id":"ea8270e32509d1b6c7b084dfdca2be46c995e3ff"}]}