A couple of IR client fixes.

Closed Michael Whittaker requested to merge inconsistent_confirm into master
  1. PendingRequest was used polymorphically but had a non-virtual destructor. Now, the destructor is virtual.
  2. HandleConfirm had what I thought was a bug (discussed in #13). Turns out it wasn't. I added a comment explaining why the code is not buggy.

Merge request reports