Mprogrammerhumor·Programmer HumorbyMatriks404 "So let's simplify code a bit today..."View original on lemmy.world237Comments16
Rooki replylemmy.worldYeah, 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. :)28
mmb_ replylemm.eeRemoved unreadable // Added /* */ on their own lines with fancy design above and below comments12
Rooki replylemmy.world1Hide 1 replyThen the code is noisy. And that isnt simplyfied it is made noisy/clustered up.-8
EEufalconimorph discuss.tchncs.deIf GitHub's UI isn't saying "infinity files changed" you're not trying hard enough.12
bboletus sh.itjust.worksBrought 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.5
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. :)
@Matriks404
I pray that all those extra lines are comments.
Removed unreadable // Added /* */ on their own lines with fancy design above and below comments
Some of them, yes.
ctrl+P > format.
Git explodes.
Then the code is noisy. And that isnt simplyfied it is made noisy/clustered up.
Your username is very fitting
If GitHub's UI isn't saying "infinity files changed" you're not trying hard enough.
By simply I thought you meant delete the project.
Ahh, yes, tracked eight hours of refactoring today…
I've done worse
Pov: you find an image of a coconut in your code and try to remove it
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.
It's just code formatting, isn't it?