removed unnecessary friend declarations from checks

Author: firewave

lib/check64bit.h

@@ -41,8 +41,6 @@ class Tokenizer;
  */
 
 class CPPCHECKLIB Check64BitPortability : public Check {
-    friend class Test64BitPortability;
-
 public:
     /** This constructor is used when registering the Check64BitPortability */
     Check64BitPortability() : Check("64-bit portability") {}

lib/checkclass.h

@@ -49,10 +49,6 @@ enum class FunctionType : std::uint8_t;
 
 /** @brief %Check classes. Uninitialized member variables, non-conforming operators, missing virtual destructor, etc */
 class CPPCHECKLIB CheckClass : public Check {
-    friend class TestClass;
-    friend class TestConstructors;
-    friend class TestUnusedPrivateFunction;
-
 public:
     /** @brief This constructor is used when registering the CheckClass */
     CheckClass() : Check("Class") {}

lib/checkio.h

@@ -42,8 +42,6 @@ enum class Severity : std::uint8_t;
 
 /** @brief %Check input output operations. */
 class CPPCHECKLIB CheckIO : public Check {
-    friend class TestIO;
-
 public:
     /** @brief This constructor is used when registering CheckIO */
     CheckIO() : Check("IO using format string") {}

lib/checkmemoryleak.h

@@ -65,8 +65,6 @@ enum class Severity : std::uint8_t;
  */
 
 class CPPCHECKLIB CheckMemoryLeakInFunction : public Check {
-    friend class TestMemleakInFunction;
-
 public:
     /** @brief This constructor is used when registering this class */
     CheckMemoryLeakInFunction() : Check("Memory leaks (function variables)") {}
@@ -92,8 +90,6 @@ class CPPCHECKLIB CheckMemoryLeakInFunction : public Check {
  */
 
 class CPPCHECKLIB CheckMemoryLeakInClass : public Check {
-    friend class TestMemleakInClass;
-
 public:
     CheckMemoryLeakInClass() : Check("Memory leaks (class variables)") {}
 
@@ -112,8 +108,6 @@ class CPPCHECKLIB CheckMemoryLeakInClass : public Check {
 /** @brief detect simple memory leaks for struct members */
 
 class CPPCHECKLIB CheckMemoryLeakStructMember : public Check {
-    friend class TestMemleakStructMember;
-
 public:
     CheckMemoryLeakStructMember() : Check("Memory leaks (struct members)") {}
 
@@ -132,8 +126,6 @@ class CPPCHECKLIB CheckMemoryLeakStructMember : public Check {
 /** @brief detect simple memory leaks (address not taken) */
 
 class CPPCHECKLIB CheckMemoryLeakNoVar : public Check {
-    friend class TestMemleakNoVar;
-
 public:
     CheckMemoryLeakNoVar() : Check("Memory leaks (address not taken)") {}
 

lib/checknullpointer.h

@@ -48,8 +48,6 @@ namespace ValueFlow
 /** @brief check for null pointer dereferencing */
 
 class CPPCHECKLIB CheckNullPointer : public Check {
-    friend class TestNullPointer;
-
 public:
     /** @brief This constructor is used when registering the CheckNullPointer */
     CheckNullPointer() : Check("Null pointer") {}

lib/checkother.h

@@ -50,10 +50,6 @@ struct UnionMember;
 /** @brief Various small checks */
 
 class CPPCHECKLIB CheckOther : public Check {
-    friend class TestCharVar;
-    friend class TestIncompleteStatement;
-    friend class TestOther;
-
 public:
     /** @brief This constructor is used when registering the CheckClass */
     CheckOther() : Check("Other") {}

lib/checkpostfixoperator.h

@@ -41,8 +41,6 @@ class Tokenizer;
  */
 
 class CPPCHECKLIB CheckPostfixOperator : public Check {
-    friend class TestPostfixOperator;
-
 public:
     /** This constructor is used when registering the CheckPostfixOperator */
     CheckPostfixOperator() : Check("Using postfix operators") {}

lib/checkuninitvar.h

@@ -58,8 +58,6 @@ struct VariableValue {
 /** @brief Checking for uninitialized variables */
 
 class CPPCHECKLIB CheckUninitVar : public Check {
-    friend class TestUninitVar;
-
 public:
     /** @brief This constructor is used when registering the CheckUninitVar */
     CheckUninitVar() : Check("Uninitialized variables") {}

lib/checkunusedvar.h

@@ -43,8 +43,6 @@ class Tokenizer;
 /** @brief Various small checks */
 
 class CPPCHECKLIB CheckUnusedVar : public Check {
-    friend class TestUnusedVar;
-
 public:
     /** @brief This constructor is used when registering the CheckClass */
     CheckUnusedVar() : Check("UnusedVar") {}