I would use Corbin's code, because it is shorter. It has everything Eli's code has, but it is well basically shorter. But to be honest with you I don't think that any of these codes is bad or good. They are a little bit different, but it doesn't change anything in the way they work. If you look at these codes you'll see that they are so similar to each other, that for me it wouldn't make any difference if I had to choose one of them.

whether or not it is poor programming style to call an event procedure from other procedures
It depends on how many times you'll call the event. Corbin's case, he uses it only once. It would be OK with me. But if I had to call it more than 3 times I would rather use the Procedure (your way).

Just my $0.02