1. It would have called the ISR the first time.
    But it would have never made it back to the main program to display the change.
    As soon as it exits the ISR, it immediately triggers an interrupt again because the flag wasn't cleared.
    .
  2. The 16F1's do automatic context saving.
    You do not need any wsave variables.

    Which is why it didn't tell you to uncomment any of them.