Calling the function just means "I'm a movement command and would like to follow the new select-on-shift behavior".
Then the function can decide on what actually happens when.
"Extend the temporary region highlighting for the next command"is a neater and more self-contained task for a function than"Extend the temporary region highlighting for the next command, but only if this-single-command-shift-translated is non-nil; otherwise, do nothing"
The docstring of the function would be more like "Handle the mark selection for movement commands."