[maramizo]

Quote:

Originally Posted by JaTochNietDan The error means that the piece of code will never possibly be reached. The last return in that code can never be reached, because there is no logical path to where it can be reached, the code is always stopped before it. So either remove it, or else re-structure the code. Additionally you have two un-needed brackets in the snippet.

There really was no reed to write all that... Lol.
@OP :
If you use "else", you shouldn't really be using return 1; at the end, because ELSE includes EVERYTHING other than the IF earlier stated.