GitLab

This way cppcheck should recognize it as such.

Instead of assignment in the copy constructor.

Put it so cppcheck finds it.

It contains pointers, doesn't make sense to copy and writing copy
constructors would be extra work.

The variable is in larger scope because it is used as a constant, so
it's at the top of the function.

They are used, but cppcheck is probably confused about use from friend
class and templates. I'd be confused myself and I'm not written in C++,
so it is excused.

It seems to check more and run longer. It also produces bunch of new
warnings.

This reverts commit 6ce96d1b.

This should have gone to a branch, not master :-|.

It seems to check more and run longer. It also produces bunch of new
warnings.

edited src/bin/resolver/bench/Makefile.am
to remove unneeded linking.

remove GTEST_LDADD
as discussed on jabber
the benchmark tool doesn't use googletest

getenv requires include of stdlib.h in src/bin/d2/d2_controller.cc