non-void function should return a value