On Mon, Jun 29, 2015 at 1:08 PM, Stefan Beller <[email protected]> wrote: > (optional nit, bikeshedding) > In conjunction with the previous patch I just wonder when we put a > TODO and when we want to put a NEEDSWORK, or if we're being > inconsistent here as both issues will be resolved in a later patch > in the series.
That's a code style thing that I don't personally have a strong opinion about. Not sure if the following means anything, but on master, git grep '\bTODO\b' | wc -l 102 git grep '\bNEEDSWORK\b' | wc -l 45 git log -G'\bTODO\b --oneline | wc -l 185 git log -G'\bNEEDSWORK\b' --oneline | wc -l 120 So it does seem that temporary stuff is usually tagged with "NEEDSWORK", rather than "TODO". Anyway, it's probably better to be consistent, so I will switch to "NEEDSWORK". Thanks, Paul -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
