Replace C++ with C - Githubissues

AFLplusplus / Grammar-Mutator

A grammar-based custom mutator for AFL++

Apache License 2.0

229 stars 17 forks source link

Replace C++ with C #5

Closed h1994st closed 4 years ago

h1994st commented 4 years ago

Current C++ parts:

[x] Chunk store relies on
- [x] std::map --> rxi/map
- [x] std::set --> barrust/set
- [x] std::vector
[x] Some parts rely on std::string, which can be replaced by C char[] and related functions

TBD:

[ ] googletesting is implemented in C++. Other candidates for the testing framework: CUnit, CMocka

vanhauser-thc commented 4 years ago

I like C and dislike C++. C++ however can make coding things much easier. as the module is self-contained for me its fine if it stays c++. the llvm modules are also c++. getting rid of the cmake however ... :)

h1994st commented 4 years ago

Yeah, C is also more efficient than C++. I have added this task to TODO, so, probably in the following weeks, I will change the project to pure C.

BTW, Makefile support has been added, and it works correctly. README.md is also updated accordingly, so you can have a try in case I missed anything. : )

CMake still lives in the project just for my personal preference. I will maintain both building scripts, so we can keep both of them.

andreafioraldi commented 4 years ago

imo googletest can stay here in C++

h1994st commented 4 years ago

imo googletest can stay here in C++

If so, it will become easier for me : ). I will finish replacing other parts at first

h1994st commented 4 years ago

Will reopen this issue, if we decide to replace googletesting