More very minor changes.

eranpeer · May 12, 2024 · 2bc4623 · 2bc4623
1 parent 6438b95
commit 2bc4623
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/include/mockutils/gcc/VirtualTable.hpp b/include/mockutils/gcc/VirtualTable.hpp
@@ -131,7 +131,7 @@ namespace fakeit {
 
     private:
         static void **buildVTArray() {
-            int size = VTUtils::getVTSize<C>();
+            unsigned int size = VTUtils::getVTSize<C>();
             auto array = new void *[size + 2 + numOfCookies]{};
             array += numOfCookies; // skip cookies
             array++; // skip top_offset

diff --git a/tests/multiple_translation_units_stub.h b/tests/multiple_translation_units_stub.h
@@ -9,8 +9,6 @@ namespace multiple_tu {
 	struct SomeInterface {
 		virtual int func() = 0;
 		virtual std::string func2() = 0;
-		virtual int funcOverloaded(int) = 0;
-		virtual int funcOverloaded(double) = 0;
 	};
 
 	void stubFunc(fakeit::Mock<SomeInterface>& mock);

diff --git a/tests/multiple_translation_units_stub_test.cpp b/tests/multiple_translation_units_stub_test.cpp
@@ -14,14 +14,17 @@ struct MultipleTranslationUnitsStub : tpunit::TestFixture {
 	{
 	}
 
-	// TODO: more tests to be sure I didn't broke something by changing stuff (alterning mocked func with when and func).
+	// TODO: more tests to be sure I didn't broke something by changing stuff:
+	// - alterning mocked func with when and func.
+	// - testing return value.
+	// - multiple mocks on same method.
+	// - etc ?
 
 	void NoCollidingIds() {
 		Mock<SomeInterface> mock;
 
 		stubFunc(mock);
 		When(Method(mock, func2)).Return("Something");
-		When(OverloadedMethod(mock, funcOverloaded, int(int))).Return(5);
 
 		SomeInterface &i = mock.get();