Почему cppcheck говорит, что «параметр функции должен передаваться по ссылке»?

Question

Это код cppcheck show waring "[event.cpp: 20]: (performance) Параметр функции 'path' должен передаваться по ссылке."

void 
event::set_path(const std::string path) 
{ 
    this->_path = path; 
} 

но другой код, включая строку paramer не показывают это предупреждение, как:

int 
watcher::init_watch(const struct stat *sb, std::string path, bool linked) 
{ 
    int wd; 
     .... 
} 

почему?

Answer 1

Потому что это должно быть! Нет причин передавать const-копию, вы никак не можете ее изменить, поэтому зачем ее копировать. В худшем случае придется выделять память для новой строки, а затем копировать строку по одному байту за раз. В лучшем случае это может сделать некоторую внутреннюю магию подсчета ссылок, но если вы просто передали ее по ссылке, вы, скорее всего, скопируете один указатель на новую точку в стеке. pass by const std::string& path - Это будет намного быстрее.

Параметр пути в init_watch также должен быть передан посредством ссылки на константу, потому что это тоже сделает копию.