Use correct flavor of abs()
Use correct return type
Remove unnecessary conditionals. They're always true.