-
Notifications
You must be signed in to change notification settings - Fork 17
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
Cut the 0.6.0 release #86
Conversation
Update CHANGELOG. Update README. Cut the 0.6.0 release.
One question before a release: could we get a specific function? get_or_create? |
I could whip up a PR if you want :) |
Isn't |
|
Ah yes, I'd forgotten about that. If you would be kind enough to submit a PR I'd certainly be willing to take a look :) |
Hmmm not sure how to implement So we have the following options:
|
@fitzgen @philipc @indiv0 which option should we go with? Note that right now, std uses one global backtrace context that all threads share... |
That lock is needed to prevent mixed output, so does that mean it won't be going away even if addr2line supports multi-threaded? If so, then I think making addr2line single threaded is ok. For the semi-racy option, we need to make sure nothing needs to store a reference to the value. Currently The busy wait and semi-racy options sound like hacks to workaround artificial issues due to libstd integration, so my preference would be to avoid them. Either the platform supports multi-threading and mutexes, and we use them, or it doesn't and we make addr2line single threaded. Can we make the mutex support indirect? eg define a |
Sounds like a viable approach. @fitzgen what do you think? Is it OK to drop multi threading? It seems that libstd would only need single threaded access any way due to its global lock (that lock would probably also be the place where we store the symbol context, if we can switch from sys::mutex::Mutex to sync::Mutex). |
Both the trait approach as well as the single threaded approach could use a release of |
Alrighty :) |
Version 1.0.0 has been released. |
Update CHANGELOG.
Update README.
Cut the 0.6.0 release.