semicolon at the end should be a comma