28
Is it a bad practice to replace compiler warnings with a bunch of TODO notes? (lemmy.zip)
submitted 5 months ago by to c/programming@programming.dev
11 comments fedilink hide all child comments
 

I am using rust, but this applies to many other languages, I get warnings like, dead code, unused variables, and etc, and while I remove most of them, there are some im not sure of, I like running my program and there being 0 warnings, or 0 warnings as i scroll down my code, so for things im unsure of, i mark them so the compiler doesn't put warnings there. I also add comments, starting with TODO:, it has some information about what to think about when i revisit it, also the todo's gets highlighed in my IDE with my extension, but is this bad practice?

you are viewing a single comment's thread
view the rest of the comments
[–] 20 points 5 months ago*

It's not ideal.

Depending on the kind of warning, you may be able to re-write the code in question including a 'todo!()' to capture the "i need to think about this" case.

This way you get a crash at runtime. This is preferable to continuing execution with incorrect computation.

It's best just to fix the warnings ASAP. They are usually trivial anyway.

  • permalink
  • fedilink
  • source
    •