Page 1 of 1

Coding Bounty - Botany trait bug fix

Posted: Tue May 18, 2021 10:57 am
by bean_sprout
Edit: Seems no one is interested, and I don't want to keep checking back in the event someone randomly decides to take it, so I'm closing the bounty.

Re: Coding Bounty - Botany trait bug fix

Posted: Tue May 18, 2021 12:51 pm
by XDTM
From a cursory look the bug is caused by the liquid contents proc qdeleting the object before it has the chance to process the prickles signal on impact.
A quick patchwork solution would be to simply add a tiny delay to the qdel() call to allow for all signal reactions to go through, while a more comprehensive fix would probably involve a refactor that allows signal registrations to set their priority for cases like these (unless something like this already exists). It'd be up to maintainers to decide if the first solution is acceptable or not.

I'd make the fix myself, but i'm not familiar with bitcoin so i wouldn't be able to claim the bounty anyway

Re: Coding Bounty - Botany trait bug fix

Posted: Tue May 18, 2021 1:24 pm
by bean_sprout
Yeah, what you describe seems to be the more fundamental problem. I had talked to a couple people, who seemed to think that the core problem would be more of a pain to fix. MrMelbert in particular pretty much came up with what is outline in my initial post as an easier alternative.

I would of course be fine with the core problem being fixed, but I'm not sure how big of a job it would be and how that would effect the expected payout. If anyone has an interest in it, though, let me know and we might be able to work something out.