I think this function is too big. I generally prefer to have static functions to call for each case unless it's a one-liner to keep the switch state from getting too big.
The code review has to be automatic. It has to be added to the code review by Crucible not the developer as of our process. Please create code reviews that are automatic and not modified by the developer otherwise it would be out of context and useless.
