braces around if else block? since you are modifying it?