Remove trailing spaces from source files, test files and cmakelists.txt files

Author: mcbarton

CMakeLists.txt

@@ -246,9 +246,9 @@ endif()
       message(fatal "LLVM/CppInterOp requires c++14 or later")
     endif()
   endif()
-  
+
   ## Find supported Cling
-  
+
   if (CPPINTEROP_USE_CLING)
     if (NOT Cling_FOUND AND DEFINED Cling_DIR)
       find_package(Cling REQUIRED CONFIG ${cling_extra_hints} NO_DEFAULT_PATH)

include/clang/Interpreter/CppInterOp.h

@@ -315,8 +315,8 @@ namespace Cpp {
   /// underlying decl is not a class it returns the input unchanged.
   CPPINTEROP_API TCppScope_t GetUnderlyingScope(TCppScope_t scope);
 
-  /// Gets the namespace or class (by stripping typedefs) for the name 
-  /// passed as a parameter, and if the parent is not passed, 
+  /// Gets the namespace or class (by stripping typedefs) for the name
+  /// passed as a parameter, and if the parent is not passed,
   /// then global scope will be assumed.
   CPPINTEROP_API TCppScope_t GetScope(const std::string& name,
                                       TCppScope_t parent = nullptr);

lib/Interpreter/CMakeLists.txt

@@ -114,12 +114,12 @@ endif()
 
 
 if(EMSCRIPTEN)
-  # FIXME: When dynamically linking the Emscripten shared library to the 
+  # FIXME: When dynamically linking the Emscripten shared library to the
   # unit tests main_module you get errors due to undefined symbols. The reading of the file
   # below into a SYMBOLS_LIST variable is a temporary workaround that exports the undefined
   # symbols from the shared library, until it can be determined why they are not being exported already.
   file(READ "${CMAKE_SOURCE_DIR}/lib/Interpreter/exports.ld" SYMBOLS_LIST)
-  
+
   # Replace newlines with spaces
   string(REPLACE "\n" " " SYMBOLS_LIST "${SYMBOLS_LIST}")
 

lib/Interpreter/DynamicLibraryManagerSymbol.cpp

@@ -908,7 +908,7 @@ namespace Cpp {
       // of the Object file. They are present in the ExportDirectory section.
       for (auto I=CoffObj->export_directory_begin(),
                 E=CoffObj->export_directory_end(); I != E; I = ++I) {
-        // All the symbols are already flagged as exported. 
+        // All the symbols are already flagged as exported.
         // We cannot really ignore symbols based on flags as we do on unix.
         StringRef Name;
         auto Err = I->getSymbolName(Name);

unittests/CMakeLists.txt

@@ -43,7 +43,7 @@ endif()
   # FIXME: Just call gtest_add_tests this function is available.
   #gtest_add_tests(${name} "${Arguments}" AUTO)
 endfunction()
- 
+
 add_subdirectory(CppInterOp)
 
 add_custom_target(check-cppinterop COMMAND ${CMAKE_CTEST_COMMAND} -V --build-config $<CONFIG>

unittests/CppInterOp/EnumReflectionTest.cpp

@@ -190,7 +190,7 @@ TEST(EnumReflectionTest, GetIntegerTypeFromEnumType) {
 #else
   EXPECT_EQ(get_int_type_from_enum_var(Decls[10]), "unsigned int");
 #endif
-  EXPECT_EQ(get_int_type_from_enum_var(Decls[11]), "NULL TYPE"); // When a non Enum Type variable is used 
+  EXPECT_EQ(get_int_type_from_enum_var(Decls[11]), "NULL TYPE"); // When a non Enum Type variable is used
 }
 
 TEST(EnumReflectionTest, GetEnumConstants) {

unittests/CppInterOp/FunctionReflectionTest.cpp

@@ -170,7 +170,7 @@ TEST(FunctionReflectionTest, HasDefaultConstructor) {
     int sum(int a, int b){
       return a+b;
     }
-    
+
     )";
 
   GetAllTopLevelDecls(code, Decls);
@@ -277,7 +277,7 @@ TEST(FunctionReflectionTest, GetClassDecls) {
 
       template<class A = int, class B = char>
       long get_float_size(int i, A a = A(), B b = B()) {}
-           
+
       template<class A>
       void get_char_size(long k, A, char ch = 'a', double l = 0.0) {}
 
@@ -294,7 +294,7 @@ TEST(FunctionReflectionTest, GetClassDecls) {
 
   GetAllTopLevelDecls(code, Decls);
   GetAllSubDecls(Decls[0], SubDecls);
-  
+
   std::vector<Cpp::TCppFunction_t> methods;
   std::vector<Cpp::TCppFunction_t> template_methods;
 
@@ -337,7 +337,7 @@ TEST(FunctionReflectionTest, GetFunctionReturnType) {
       char get_string(A) {
           return 'A';
       }
-              
+
       template<class A>
       void get_size() {}
 
@@ -412,7 +412,7 @@ TEST(FunctionReflectionTest, GetFunctionNumArgs) {
       char get_string(A, int i) {
           return 'A';
       }
-              
+
       template<class A>
       void get_size() {}
 
@@ -452,7 +452,7 @@ TEST(FunctionReflectionTest, GetFunctionRequiredArgs) {
 
       template<class A = int, class B = char>
       long get_size(int i, A a = A(), B b = B()) {}
-           
+
       template<class A>
       void get_size(long k, A, char ch = 'a', double l = 0.0) {}
     };
@@ -693,7 +693,7 @@ TEST(FunctionReflectionTest, BestOverloadFunctionMatch1) {
       long MyTemplatedMethodClass::get_size(A a, B b) {
           return sizeof(A) + sizeof(B);
       }
-      
+
       template <int N, class T>
       long MyTemplatedMethodClass::get_size(T a) {
         return N + sizeof(T) + a;
@@ -818,7 +818,7 @@ TEST(FunctionReflectionTest, BestOverloadFunctionMatch3) {
     template<typename T>
     struct A {
       T value;
-      
+
       template<typename TT>
       A<TT> operator-(A<TT> rhs) {
         return A<TT>{value - rhs.value};
@@ -1293,12 +1293,12 @@ TEST(FunctionReflectionTest, GetFunctionCallWrapper) {
   FCI_Dtor.Invoke(object);
   output = testing::internal::GetCapturedStdout();
   EXPECT_EQ(output, "Dtor Called\n");
-  
+
   std::vector<Decl*> Decls1;
   std::string code1 = R"(
-        template<typename T> 
+        template<typename T>
         struct S {
-        
+
         static T Add(T a, T b) { return a + b; }
 
         };
@@ -1430,7 +1430,7 @@ TEST(FunctionReflectionTest, GetFunctionArgName) {
 
     template<class A = int, class B = char>
     long get_size(int i, A a = A(), B b = B()) {}
-          
+
     template<class A>
     void get_size(long k, A, char ch = 'a', double l = 0.0) {}
     )";
@@ -1470,7 +1470,7 @@ TEST(FunctionReflectionTest, GetFunctionArgDefault) {
 
     template<class A = int, class B = char>
     long get_size(int i, A a = A(), B b = B()) {}
-          
+
     template<class A>
     void get_size(long k, A, char ch = 'a', double l = 0.0) {}
 

unittests/CppInterOp/ScopeReflectionTest.cpp

@@ -101,14 +101,14 @@ TEST(ScopeReflectionTest, IsClassPolymorphic) {
   std::vector<Decl*> Decls;
   GetAllTopLevelDecls(R"(
     namespace N {}
-    
+
     class C{};
-    
+
     class C2 {
     public:
       virtual ~C2() {}
     };
-    
+
     int I;
   )",
                       Decls);
@@ -577,7 +577,7 @@ TEST(ScopeReflectionTest, GetScopeFromType) {
     N::S s;
 
     int i;
-    
+
     N::T t;
 
     N::E e;

unittests/CppInterOp/TypeReflectionTest.cpp

@@ -384,7 +384,7 @@ TEST(TypeReflectionTest, GetTypeFromScope) {
     struct S {};
     int a = 10;
     )";
-  
+
   GetAllTopLevelDecls(code, Decls);
 
   EXPECT_EQ(Cpp::GetTypeAsString(Cpp::GetTypeFromScope(Decls[0])), "C");
@@ -476,7 +476,7 @@ TEST(TypeReflectionTest, GetDimensions) {
   {
     EXPECT_EQ(dims[i], truth_dims[i]);
   }
-  
+
   // Variable c
   dims = Cpp::GetDimensions(Cpp::GetVariableType(Decls[2]));
   truth_dims = std::vector<long int>({1, 2});

unittests/CppInterOp/VariableReflectionTest.cpp

@@ -166,11 +166,11 @@ TEST(VariableReflectionTest, GetTypeAsString) {
 
   std::string code = R"(
   namespace my_namespace {
-  
+
   struct Container {
     int value;
   };
-  
+
   struct Wrapper {
     Container item;
   };
@@ -490,7 +490,7 @@ TEST(VariableReflectionTest, DISABLED_GetArrayDimensions) {
   //FIXME: is_vec_eq is an unused variable until test does something
   //auto is_vec_eq = [](const std::vector<size_t> &arr_dims,
   //                    const std::vector<size_t> &truth_vals) {
-  //  if (arr_dims.size() != truth_vals.size()) 
+  //  if (arr_dims.size() != truth_vals.size())
   //    return false;
   //
   //  return std::equal(arr_dims.begin(), arr_dims.end(), truth_vals.begin());