MilchRatchet / PAL

Polytime Algorithm Libraries
zlib License
0 stars 0 forks source link

Redundacy in UTILS files #8

Open Mathemalsky opened 3 years ago

Mathemalsky commented 3 years ago

I think there is some redundancy between the files TEST_UTILS.cpp and TEST_UTILS.c. Was that intended? If yes what were your reasons to write the functions again. If not should we merge this into one file?

MilchRatchet commented 3 years ago

The functions in TEST_UTILS.cpp use std::string as an input. Thus I am not able to use them if I want to write the tests in C.

Mathemalsky commented 3 years ago

Is there any benefit of writing the test in c?

MilchRatchet commented 3 years ago

Not really no. Using only TEST_UTILS.cpp would force all tests to be written in C++. It does not really matter, I just did it this way so that the tests can be written in whatever language. In fact, we could even go ahead and write the tests in GO or Rust if we feel like it.