return value is not required since you always return true, this also means you'll never get to the failure path above (need to catch exception).