algorithmica: 'merge_sort::merge()' crashes with double-free for T: Drop

[JOE1994]

Original issue report: AbrarNitk/algorithmica#1

Thank you for reviewing this PR :)

[Shnatsel]

Thanks!

[Shnatsel]

@tarcieri looks like the ID assignment is not running on main, and the force push is not triggered either. How is the automation is set up with master vs main?

[tarcieri]

master is set up as a (hopefully temporary) mirror of main:

https://github.com/RustSec/advisory-db/blob/master/.github/workflows/mirror-main-to-master.yml

PRs shouldn't be merged to master. They'll get clobbered.

I used the rename branch feature of GitHub which should've reparented all open PRs to main. I guess keep an eye out for PRs opened against master.

If there are commits merged to master that aren't on main, they need to be cherry-picked.

[Shnatsel]

Oh, I was hoping to use master as a staging branch and only mirror it to main once the IDs are assigned, so that RUSTSEC-0000-0000 IDs are not visible to clients at all. And that way we'd mirror from master to main, conditionally.

[tarcieri]

That'd be a pretty big change from how things work now. I'd prefer to get rid of the master branch entirely (see #312)

Also the client library already ignores RUSTSEC-0000-0000.

If you'd like to do something like that, I'd suggest opening a discussion issue and we can plan it out.