if (!success)
would be more conventional