Just a nit, but doesn't our style guide say to use parens for method arguments? Especially when there are multiple args (and in this case a no-op assignment), it's kinda hard to see what's happening at a glance.