Skip to content

Conversation

murchandamus
Copy link
Contributor

@murchandamus murchandamus commented Jul 2, 2025

Merge created using these instructions: https://github.com/murchandamus/fuzzing-helpers

@maflcko
Copy link
Contributor

maflcko commented Jul 3, 2025

Looks like the line and function coverage did not change at all. This seems a bit odd.

Happy to merge this, but there may be something wrong with your setup?

@maflcko
Copy link
Contributor

maflcko commented Jul 3, 2025

For reference, in your last pull there was a nice .3% increase overall: #225

@murchandamus
Copy link
Contributor Author

I wasn’t fuzzing for a while, due to setting up a new office workstation, and then after recommencing fuzzing missed that the service I had set up didn’t automatically start after restarting my computer, so I have less recent fuzzing before opening this, so I might just not have any additional coverage with your inputs getting merged around the same time.
Now everything seems to be up and running again, so I could circle back in a few weeks instead.

@maflcko
Copy link
Contributor

maflcko commented Jul 3, 2025

For reference, I compared the coverage on main from before my inputs were merged, so that it can be evaluated independently.

Sounds good about waiting a few weeks and then circling back.

As heads up, looking at bitcoin/bitcoin#32275, it could be good to submit new inputs before mid of August and then use end of August to reduce/delete.

@murchandamus
Copy link
Contributor Author

For reference, I compared the coverage on main from before my inputs were merged, so that it can be evaluated independently.

I see. Maybe I did do something wrong then. Even with having less overall fuzzing time, I would have expected some additional coverage. 🤔
Either way, I think I’ll redo it in a couple weeks then.

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.

2 participants