How about renaming `err` to `success` or something? It's confusing to see `err == YES` as the success case :)