You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think some of the crates in the project could have better names, especially since #816 will be adding another crate. Changing crate names causes confusion for current consumers, however keeping poor names can cause confusion for future consumers: either decision has a downside.
This is my proposal (as if #816 occurred, anything omitted stays the same):
crate
new name
reasoning
minidump-processor (non-stackwalk logic)
minidump-processor
encompasses all the extra things this does
minidump-processor (stackwalk logic)
minidump-stackwalk
it's the stackwalk function
minidump-stackwalk
minidump-analyzer
it does much more than just stack walking
The mozilla minidump-analyzer crate (not in this project) could have another more specific name, like minidump-stack-info.
The text was updated successfully, but these errors were encountered:
Now that I've cut a new release I think we can proceed with this. We might aim for a 1.0.0 release which will break some things WRT the current setup but will also guarantee future stability. One thing I'd like to do first is figure out who are our non-Mozilla downstream users and how this would affect them (if at all).
I think some of the crates in the project could have better names, especially since #816 will be adding another crate. Changing crate names causes confusion for current consumers, however keeping poor names can cause confusion for future consumers: either decision has a downside.
This is my proposal (as if #816 occurred, anything omitted stays the same):
minidump-processor
(non-stackwalk logic)minidump-processor
minidump-processor
(stackwalk logic)minidump-stackwalk
minidump-stackwalk
minidump-analyzer
The mozilla
minidump-analyzer
crate (not in this project) could have another more specific name, likeminidump-stack-info
.The text was updated successfully, but these errors were encountered: