Comment 2 for bug 179738

It seems that the second copy of the code block is covering up the error in the first code block.

If you are replying to a message with a "reply-to" header, then the first code block adds it to 'to' and the second code block becomes a no-op.

If you are reply to a message without a "reply-to" header, then the first code block adds a nil to 'to' and the second code block adds the "from" header. (If the first code block had done the job, then this would become a no-op as well.)

The spurious nil is then eliminated by vm-parse-addresses later on.

What confusion!