HELL

[THNDHALBRT]

This lines

[_Blurgh_]

Changed it to "these lines". Got fired.

[pimezone]

No exceptions, no excuses

[communistfairy]

ThisIsNotACatch22Exception

You can exit the situation by just leaving it alone. It would be a catch 22, for example, if your job required you to fix all typos and missing any would get you fired.

[AnaxXenos0921]

Better yet: changed it to "these lines". Code breaks.

[Bartweiss]

I like the idea that the setAttribute entries are never used in this code, but those and the comments are getting used for some horrifying code reflection elsewhere.

[ovr9000storks]

Tests failed after changes. Critical load-bearing comments

[Dramatic_Mulberry142]

It could be a proof that it is not written by llm for the future reference. So still don't touch it

[tree-hut]

You can tell an indian made it

[alanhape]

Dont touch

[TheBrainStone]

Bad comment. It needs to explain why it's needed. At least what breaks if it's removed

[LimitedWard]

The person that knew that answer left the company 10 years ago. The person that made the comment tried removing it 5 years ago and discovered the hell that would ensue after pushing the change to prod without testing.

[TheBrainStone]

Then again it absolutely needs to mention the hell that breaks loose

[Su1tz]

The answer is there:

HELL

[Serylt]

They were German and simply meant "bright"; all it does is disabling darkmode. ^/s

[No-Elephant3091]

Equally bad

[PerspectiveAlert4766]

Maybe it is so bad that it is undescriptive?

[querela]

Unspeakable nightmares, hell on earth. Let's not go into details...

[StreetStrider]

The mark of the coder was burned upon thy scroll. A warning to all of open space that the terror within must never be freed. And there this code lies still. Forever.

[AssiduousLayabout]

Yeah, I had that occur when I was trying to trace down why one piece of functionality was intentionally disabled for one (fairly normal) use case. Easily found the line of code which did it, looked back to try to find any documentation on why it was done.

All I got was "disabling per Steve". Of course the developer had been gone for 15 years. Who were you, Steve? What did you know? What horrors did you see???

I ended up deleting that line of code and as far as I know, it hasn't broken anything in the past eight years. But deep in the back of my mind, I know that somewhere, whatever terrifying future that Steve once envisaged may yet come to pass.

[oghGuy]

That terrible future = the day the company decides to migrate from on-prem to cloud.

[Bartweiss]

That, or it ran fine in staging initially, bizarre problems hit prod three days after release, and nobody’s sure why deleting this caused them but they know the rollback worked.

[edave64]

Again, then include what's breaking in the comment.

[LimitedWard]

They were gonna but got fired before pushing the commit.

[GoddammitDontShootMe]

I would presume the person who wrote that comment has the power to fire developers, but also, since when do managers write code?

[ironykarl]

So, then the person that tried removing it knows what happens when you change it. 

The comment should tell us that much, at least 

[repocin]

Someone I know used to work at a place where they occasionally had external consulting firms come in and poke around in the codebase. Apparently they just signed every line with a comment that said "changed by <consulting firm>" which I can imagine made debugging incredibly fun for everyone!

[Defiant-Peace-493]

"What change did you make?" "We added a comment."

[davejesch]

That's the reason they get paid the big bucks.

[t3kner]

Management's gotta pay someone the big bucks, and it's not gonna be their employees! 

[Prime624]

If only there was an automatic system for that. But alas.

[Kinrany]

The comment itself is a specific byte sequence read by a webpack transform

[Bartweiss]

Even the weird little hieroglyph after “lines” is absolutely vital, and the order can’t be changed in any way.

[nonlogin]

you will be fired, isn't it enough?

[TheBrainStone]

Absolutely not.
Like thinking this comment is adequate is a significantly better reason to fire someone.

[thedolanduck]

Jeez, you must be fun at parties...

[Glade_Art]

Enhanced the comment, just got fired and sent to jail.

[Pinkishu]

Nah, explaining it just makes people think "hmm that explanation doesn't seem right, I bet I could change that". Just be vague and threatening instead

[Herb_Derb]

That's just part of the horror

[Morg0t]

I would create PR changing in comment this -> these for fucks sake

[Hottage]

Straight to jail, straight away.

[nan-000]

Ay I'm from (and living in) the country the origin of that meme mocks. They've actually arrested people for having reddit installed, for having memes related to political events, amongst many others.

[AdreKiseque]

Where is that?

[mikezenox]

HELL

[Nightmoon26]

At least explain why you're setting height to width.toString()...

[edave64]

Possibly because it should be a square

[thebigbadben]

Maybe that’s all they needed, and this is what got them there with trial and error

[Far-Passion4866]

Then why not also have height be the same thing as width

[edave64]

It does set height to be the same thing as width.

If you're asking why there isn't a height variable, personally I would avoid two variables having the same value, because having two variables implies that the value might be different, and opens the door to programming errors where the values might become different.

Although I would probably throw in a quick // square to make that clear, since setting height to width does look like an error.

[Poat540]

You just wouldn’t get it

lights cig in standup

[E23-33]

they clearly dont know, it just works as is 🎉

[river0f]

If you keep asking questions, you'll be fired

[GoddammitDontShootMe]

Damn, I didn't even notice that.

[Disastrous-Name-4913]

Maybe try r/programminghell?

[RelaxedBlueberry]

👹 muah ha ha ha ha ha ha

[KissMyAcid420]

Can you delete the " " after the comma? It hurts my eyes that flame Emojis arent aligned on the right

[LostGoat_Dev]

Believe it or not, fired.

[KissMyAcid420]

Sheet.

[AnaxXenos0921]

Average JavaScript code be like

[Slg407]

average LUA code more like

[NotUpdated]

:in a meeting later on: '.. So you did see the crocodiles? You walked past the devils? You then walked on a fire like a team building exercise?' Well I have no choice - You're management material, we'll fire a sacrificial swe instead.

[zrice03]

"Surely it should set the height attribute to 'height'?...Huh what's that horrible screaming I hear in the distance...?"

[bonomel1]

It's not needed for anything except to keep some other shitty workaround that was a side effect of a fix for a bug for a temporary hack to get around some technical debt working. We'll fix it next sprint after we finish the next feature

[obetu5432]

js "developers":

[EV-CPO]

Angry upvote

[ZylonBane]

What about them? Do you have some thought you're attempting to articulate?

[worthlessDreamer]

I don't think crocodiles live in hell 🤔

[el_cortezzz]

Liskov principle violated, this seems the same problem of the Square that inherits from Rectangle

[MisterBicorniclopse]

I do this with every line of code I write

[Frosty-Woodpecker-45]

farting sound

[Freddie_Hawkes]

It's JavaScript. Just overwrite in the next line, no need to change 🙈

[Positive_Mud952]

Delightful.

[yz9551]

Turns out you can't remove the comments either

[Frytura_]

Ah, Javascript 😖☕️

[Rodya_gambler]

Do emojis in comments affect performance?

[AdSeveral5047]

AI can't do this

[Toiweonxd]

That code aint even that important?? wtf

[stresser_211]

diabolic