To prevent errors like these, the various pointers in the Action struct should be marked const.
Should I add this changes on the patch?
The proper way to fix this is to check in movement(...) whether the action.operator is set to &ops[OP_CHANGE] and if so adjust action.movement accordingly. Would be nice if you sent a follow up patch.
Sure I will. Matias Linares,
