minor bug, same mistake again. compare, not assign