@Arlo Siemsen that fix was really prompt, thanks a lot!
this just shows we should perf-test more PRs, perhaps after making the perf testing server more capable (or starting to distribute it across 2-3 servers)
@Arlo Siemsen I posted 3 comparisons here which show the fix, the overall effect and the PRs landed in between the original PR and the fix (the latter two both seem negligible) https://github.com/rust-lang/rust/issues/70785#issuecomment-609438109
@Arlo Siemsen if you want to help test the SHA1 performance compared to MD5, you could make a PR that changes the default (which we'd not actually land) and we could test that. the perf stuff is all on linux but if we make linux do SHA1 hashing it should be indicative of what is now happening on windows
/msg @eddyb what was the original topic for "proposal: add source file hashes to debuginfo #69718"? Your statement that "move previous and following... " doesn't move other people's message does not match my own recollection of what I'd observed myself...
/msg is not a thing in zulip; that seems like something they could fix...
@pnkfelix "cryptographic" in lieu of "source file"
Okay well I think I was able to revise other peoples' titles
but that might because I have admin privileges perhaps?
the weird thing is that the option shows up for me too, except... oh right it's that timed thing
so maybe because I can't edit topics on other people's messages after a while, the before+after feature is also affected
whereas the usual "move this message and all after" does affect all messages
but if its a timing thing, then why let me do it as an admin right now?
(because I just did move all the messages over into this very topic...)
everyone should be able to do it
you may need to start on your own message? not sure