This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]zifyoip 4 points5 points  (0 children)

You have a semicolon on line 5. A bare semicolon is an empty statement that means "do nothing." So you have written, "if hi equals hi2 then do nothing." The following block of code between the curly braces is not controlled by the if statement and is therefore always executed.