You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/11/12 17:13:04 UTC

[GitHub] [incubator-kvrocks] PragmaTwice commented on pull request #1106: Fix performance-unnecessary-value-param warning reported by clang-tidy

PragmaTwice commented on PR #1106:
URL: https://github.com/apache/incubator-kvrocks/pull/1106#issuecomment-1312530385

   > By the way, I'm curious if there are any automated tools to fix relatively simple check like this one. @PragmaTwice
   
   Sure, some clang-tidy checks provide auto-fix, but it cannot guarantee 100% right for fixed code, so I recommand at least manual review these reports one by one.
   
   BTW, if you configure your editor/IDE correctly (vim/vscode/emacs with clangd or clion), you should see a `fix` button in the pop-up window in every code segment where a report exists here.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org