Ehh it doesn't seem to be a problem. Understanding the function doesn't require thinking about those 900 lines at once since the code for handling each message type doesn't interact with any other code in the function.