Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Named Mutators and MultiMutator API change #1387

Merged
merged 8 commits into from
Aug 1, 2023
Merged

Named Mutators and MultiMutator API change #1387

merged 8 commits into from
Aug 1, 2023

Conversation

domenukk
Copy link
Member

This adds the Named attribute to all mutators, which may be more useful down the road (vs. taking names away as #1379 suggests)

cc @addisoncrump

@domenukk domenukk requested a review from addisoncrump July 31, 2023 13:10
@domenukk
Copy link
Member Author

@tokatoka I went down a rabbit hole and changed the MultipleMutator Api (and renamed it to MutliMutator as well), please take a look :)

@domenukk domenukk requested a review from tokatoka July 31, 2023 13:48
@domenukk domenukk changed the title Mutators need names (alternative to #1379) Named Mutators and MultiMutator API change Jul 31, 2023
@domenukk
Copy link
Member Author

domenukk commented Jul 31, 2023

Also @tokatoka right now all Afl names are Afl, Redqueen uses AFLpp, which is odd. How about just renaming it to RedQueen (no Afl) and referencing the original implementation in the docs?

Like, we are AFLpp/LibAFL anyway, so AFLpp is kinda redundant :P

@tokatoka
Copy link
Member

How about just renaming it to RedQueen (no Afl) and referencing the original implementation in the docs?

ok

@domenukk
Copy link
Member Author

domenukk commented Aug 1, 2023

No space left on device :/

@domenukk domenukk merged commit 006dcac into main Aug 1, 2023
@domenukk domenukk deleted the named_mutators branch August 1, 2023 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants