If you follow the thread you will see that I was specifically responding to this comment:

But is it reasonable to add extra lines of "useless" code to such small subroutine, just to make it more readable?

What does that question have to do with Catalyst? It's a pretty generic best practices type question, so I stand by my original comment.

Regardless, what makes Catalyst so special anyway? Basic principles in writing maintainable code still apply.