The Linux codebase has over 3k TODO comments, many from over a decade ago
In my opinion, any comment prefixed by TODO, XXX, or the like should not survive past the review stage. Either fix the problem immediately or accept that it's going to be wonky forever.<p>Edit:<p>Amending here to avoid replying to ten different threads individually.<p>1. Long-term improvements should be managed by a ticket tracking system. The code is not the correct place to manage that.<p>2. Most of the disagreements below seem to come down to nomenclature differences.<p>In my own work, the code is liberally sprinkled with "Note:"s. These are, as others have pointed out, useful for providing context of what is going on, why, how it is non-optimal, etc.<p>TODOs that I come across tend to be of the "You ain't gonna need it" variety. If you can get a pull through review as-is, then the thing wasn't actually a "TODO", and if it was, then it should be added as a ticket to the backlog.<p>XXXs tend to show up in the reviews I do as code that is explicitly meant to be fixed before the pull is merged.<p>So to summarize:<p>1. "Note:"s in code are great.<p>2. "XXX" shouldn't survive the review<p>3. "TODO" should be handled by the ticketing backlog to better separate out "actually needed" from "theoretically nice, but not necessary in practice".<p>Further, things that are "TODO" today often make zero sense as "TODO" in a year when the code has grown and evolved more, but since someone put it in as a TODO, I find that they rarely ever get removed since "surely someone knows what that means" (but that person is usually either gone or no longer remembers).