What's wrong with this simple function?

Printable View