Shouldn't this return instead of print?