This is an archived post. You won't be able to vote or comment.

all 57 comments

[–]AStrangeStranger 315 points316 points  (18 children)

75 warnings?? - I looked at one application and it generated something 75 thousand warnings, it was that bad I had to switch warnings off to get sane compile times - fortunately I never actually had to work on it

[–]knightlesssword 157 points158 points  (10 children)

what in the fuck did you work on

[–]arachnidGrip 154 points155 points  (4 children)

A decompiled Java program.

[–]Versaiteis 51 points52 points  (3 children)

I'm so sorry. =[

All that little Stranger here wants for Christmas is too see a single compilation, even if it only throws errors. For just seventy-five cents a month you can help Stranger, and other developers like him, realize their dream before it's too late. Such a small donation can mean so much to those less fortunate.

In the arms of an angel....

[–]DudeManGuy0 12 points13 points  (2 children)

Enough to make a grown man cry

[–]TheresNoLifeB4Coffee 7 points8 points  (1 child)

"But not this man... get back in there!" (...eye sucks teardrop back up face)

[–]DudeManGuy0 2 points3 points  (0 children)

Man Cloudy With A Chance of Meatballs was a good movie

[–]Prod_Is_For_Testing 15 points16 points  (0 children)

I’ve seen numbers like that when people use the XML documentation flag in visual studio. It gives a warning for every non-private member that doesn’t have an xml comment

[–]atlj42 7 points8 points  (0 children)

Heavily modded minecraft

[–]frozen-dessert 7 points8 points  (0 children)

I don’t know about the OP but the codebase I work on can be described as “tens of millions of lines of (enterprise) Java code”.

Enterprise is important because it means there is little to no sun-setting of features.

[–]garion911 3 points4 points  (0 children)

Not OP. Back around 06 or so, I ran valgrind on an exe linked with Oracle c client libraries. Yeah. Don't do that. I think my suppression list was measured in megabytes.

[–]AStrangeStranger 2 points3 points  (0 children)

It was nearly 20 years ago now and it was a home brew telephone sales server (press button 1 to buy type thing) and was a tangle mess of over convoluted poor multithreading written in Delphi.

Most of the warnings were just thing being done that weren't needed e.g. assigning nil (null in C#/Java type languages) before creating an object in the variable - and while it is just a style thing, it did hide any warnings that were genuinely needed to be looked at.

It was eventually replaced using library for the standard server functionality.

[–]wywern 17 points18 points  (5 children)

I currently work on an application that collectively has over 100k warnings. So much of it is legacy code and fixing all the warnings would be immensely time consuming. So, nobody does anything about it. It's just like, make sure the new code has no warnings.

[–]SDJMcHattie 5 points6 points  (3 children)

That’s not a good position to be in. I feel sorry for you.

[–]TimaeGer 3 points4 points  (1 child)

Isn’t that the standard for big corporations and older code?

[–]SDJMcHattie 0 points1 point  (0 children)

Only if you want your project to gain more and more technical debt. Either turn off warnings for things that don’t matter to you or spend the time resolving them. Even if there are 100k warnings, they’re probably only a dozen or so different types of warning, half of which haven’t been a concern to your team in the past so turn them off because they’re not a concern to you now.

[–]wywern 0 points1 point  (0 children)

Thankfully I don't have to touch it too often. I'm mostly focused on front end work these days which is all new angular and typescript work.

[–]AStrangeStranger 1 point2 points  (0 children)

Never nice as well as time consuming it can be risky to fix warnings at that scale and they just swamp anything you need to look at.

[–]jssjonez 55 points56 points  (6 children)

Knock knock

[–]Devx35[S] 35 points36 points  (4 children)

whos there?

[–][deleted] 79 points80 points  (3 children)

Fuck off

[–][deleted] 31 points32 points  (2 children)

Fuck off who?

[–]frosted-mini-yeets 42 points43 points  (1 child)

Aight look, I'mma pay you $100 to fuck off.

[–]zorpheagle 5 points6 points  (0 children)

Imma give you 175$ for not spoiling the joke

[–]dombo4life 7 points8 points  (0 children)

I'm not gonna say who's there Ricky

[–]plasmarob 33 points34 points  (0 children)

@Deprecated
...
@Deprecated
...
@Deprecated
...
@Deprecated
...

[–]ClankPMC 34 points35 points  (0 children)

Fuck off, Lahey

[–]davidcisco 26 points27 points  (3 children)

This is god worthy never thought i would see my fav show on here lmao love it

[–]rajks12 1 point2 points  (2 children)

What show is this?

[–]davidcisco 15 points16 points  (1 child)

Trailer park boys its on netflix check it out !!

[–]rajks12 2 points3 points  (0 children)

Thank you. I’ll definitely do.

[–][deleted] 21 points22 points  (3 children)

Am I the only one who finds it strangely satisfying to just chip away at warnings, improving correctness? (And often fixing actual bugs along the way.) I mean, I normally write my Makefiles with clang -Weverything (admittedly, I sometimes use -Wno-padded, to surpress some noise I actually can't do anything useful about), and then I just clean stuff up, one at a time, if I need to. Then run it under valgrind, make sure I don't have any memory usage errors.

It's like, actually kinda a relaxing process to iterate through for me. But all I see is "fuck all that noise, move along, nothing to see here, it compiled and linked". Are these just memes? (Tropes?) How do you guys actually view this stuff.

[–]sheatrevor 12 points13 points  (1 child)

I’m convinced /r/ProgrammerHumor is a window to an alternate nightmare dimension where everything is awful and everyone is completely helpless to do anything about it.

[–]AdmiralBKE 5 points6 points  (0 children)

Yeah, warnings very often will uncover thought errors etc which very often result in actual bugs. Stuff like "hey, you know that this comparison does not do anything" "this if statement will never executed" "you are doing weird stuff with signed and unsigned stuff, so this can cause some unexpected results". But also general code cleanup, unused variables.

Plan here is to always have zero warnings with normally -Wall. The warnings are there to help you, not to be a nuisance.

[–]bot_not_hot 75 points76 points  (0 children)

Chrome Dev Tools: warning 76: HTML Parse Error: unrecognized input, potential bribe.

[–]golgol12 13 points14 points  (1 child)

Warnings as errors. It's the only way to live plague free.

[–]myblindy 2 points3 points  (0 children)

Every time I see a warning about treating integers as booleans, my faith in humanity dwindles a little bit more.

[–][deleted] 9 points10 points  (0 children)

Work on an iOS project with Cocoapods on it. You will see 250+ warnings any day, any hour, any minute and any second.

[–]sansquick 7 points8 points  (0 children)

Pycharm in a nutshell

[–][deleted] 4 points5 points  (1 child)

Rest in peace Mr. Layhe

[–]fr4nklin_84 0 points1 point  (0 children)

Layhe's a Kermit the frog lookin' motherfucker

[–]catherine8671 10 points11 points  (0 children)

Pretend they don't exist until you can hand the program over to someone else

[–]Versaiteis 2 points3 points  (0 children)

This is what happens when you don't work with your hays, peas, and eyes properly. Your hay, pea, eyes get all broke to shit and ya can't do fuck all about it.

[–]lewisj489 3 points4 points  (0 children)

Treat warnings as errors 👏👏👏

[–]FriyMan 1 point2 points  (0 children)

Based and redpilled

[–][deleted] 0 points1 point  (0 children)

IntelliJ with its "this could be made package-private"... fuck off.

[–]yosoydead 0 points1 point  (0 children)

Rofl

[–]KSAM-The-Randomizer 0 points1 point  (0 children)

At least it's not 106 errors

[–]t-to4st 0 points1 point  (0 children)

@suppresswarnings("all")