I actually like it, since the uC is supplying current when the port is low. With the exception of the modern derivatives that can be configured as push-pull, supplying current when low should always be the preference.

Erik