Is there a good reason to use an assignment inside an expression here?