i think return 1 is better