Static analysis: include/deal.II/base/table_handler.h

[bangerth]

/home/davydden/libs/deal.ii/include/deal.II/base/thread_management.h	2957	warn	V678 An object is used as an argument to its own method. Consider checking the first actual argument of the 'destroy' function.

We should address these warnings and errors from the static analysis tool PVS. In response to #3342.

[bangerth]

We should address these warnings and errors from the static analysis tool PVS:

include/deal.II/base/table_handler.h    682 err V730 Not all members of a class are initialized inside the constructor. Consider inspecting: cached_value.

In response to #3342.

[bangerth]

Same as #3532 .

[bangerth]

Oh, we ended up with different messages here. This issue should be for the issue in table_handler.h.

But the point is that cached_value is a string, so it has a default constructor and need not be initialized explicitly.