Useless code in privatemsg breaks settings with other modules

In privatemsg's implementation of user hook, there is a rather inane piece of code that works on ops 'insert' and 'update'

It is totally redundant and actually breaks the code when other modules create other categories because when those settings are saved the privatemsg settings get turned to 0.

~line 94

    case 'insert':
    case 'update':
      if (!isset($edit['privatemsg_mailalert'])) {
        $edit['privatemsg_mailalert'] = 0;
      }
      if (!isset($edit['privatemsg_allow'])) {
        $edit['privatemsg_allow'] = 0;
      }
      break;

This entire block of code is totally redundant. Removing it leaves everything functional for me.

The bug:

Running when the hook is called with "update" from another user settings category makes it default to zero, because the checkboxes are not on the category pages and therefore cannot be set. They will fail the isset test and the settings will revert to 0. I imagine this would also break when used with custom profile categories.

Removing this code fixes this problem. There should be no negative side effects but try anyway.