Yeah, too simple is sometimes bad too. I had to refactor for better readability and it added 3k lines. But it was easier for everyone to understand the code. :)
Brought in a feature from a fork of our engine at work the other day. 8000 additions and a bunch of deletions and fix commits.And apparently that's a pretty small merge...Took me days just to review it.
Simplifying doesn't have to mean fewer lines. LGTM.
Yeah, too simple is sometimes bad too. I had to refactor for better readability and it added 3k lines. But it was easier for everyone to understand the code. :)