On Wed, 17 Apr 2019 15:37:49 +0900 Masahiro Yamada <yamada.masah...@socionext.com> wrote:
> > It's pretty benign: Doug will still only get a single email. > > > > > > checkpatch checks for duplicates cc's but I think it assumes that > > things like > > > > Reported-by: fred > > Tested-by: fred > > > > were intentional. > > > > It would perhaps be better for checkpatch to special-case the "Cc: > > fred" tag and report a duplicated Cc: if fred was also mentioned in any > > other tag. > > Sorry, I do not understand what you mean. > > > Douglas did not add any Cc: tag at all > in his original patch: > https://lore.kernel.org/patchwork/patch/1053953/ > > > It's you who added the Cc: tags > (and I am guessing you did it by scripting.) > > Why is this topic related to checkpatch? Because I regularly check all patches with checkpatch but checkpatch failed to detect that I had Signed-off-by: Douglas Anderson <diand...@chromium.org> ... Cc: Douglas Anderson <diand...@chromium.org> and I believe that checkpatch could and should have warned about this situation. _______________________________________________ Kgdb-bugreport mailing list Kgdb-bugreport@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kgdb-bugreport
