vm_object: drop pointless assignment

An assignment in collapse_scan() has become useless because, on every
path, another assignment to that variable overrides it before that
variable is read. Another assignment can be avoided sometimes, so
move it down in the loop to where it's really necessary.

Reviewed by: alc, markj
Differential Revision: https://reviews.freebsd.org/D49017