shoudln't you return the result ?