Why have you put the printf inside the else? It will only get executed if x <= y and y <= z and z <= x (i.e. it will never print unless they all the same number).
Scrap the else (that said, I have no idea why you've used if/else to do... what? Find the largest number?)
- Compiler warnings are like "Bridge Out Ahead" warnings. DON'T just ignore them.
- A compiler error is something SO stupid that the compiler genuinely can't carry on with its job. A compiler warning is the compiler saying "Well, that's bloody stupid but if you WANT to ignore me..." and carrying on.
- The best debugging tool in the world is a bunch of printf()'s for everything important around the bits you think might be wrong.
