Misc Rust Fixes & Improvements #7752
Merged
+2,590
−1,885
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
644791c to
04c8df1
Compare
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
37b97cf to
bf012e1
Compare
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
20e177b to
23b4b21
Compare
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
23b4b21 to
b52660f
Compare
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
f1eb4d4 to
5eb4d02
Compare
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
5eb4d02 to
0a8be8c
Compare
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
0a8be8c to
4cffc75
Compare
Member
Author
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
4cffc75 to
edc9120
Compare
Forgot to call `BNFreeArchitectureAndAddressList`, also use `Location` instead of `ArchAndAddr`.
Use `Location` instead, `arch` can be a nullptr
To keep backwards compatibility for commonly referenced code we re-export them within the architecture module. Also does some light refactoring of some newly added APIs to keep them more consistent with other parts of the codebase.
There was a TODO there to document that it is optional, we should just wrap it in `Option` instead.
Still need to add a custom architecture for unit tests like in python
- `analysis_info` was causing a double free with function refs, also did not need to be wrapped in a Result
These have been deprecated since last stable, so they will be removed now. I do believe that there might be some external users of this api so we should alert them.
Prevents type confusions considering there are at times, three different id types being referred to in the type archive API
emesare
force-pushed
the
test_rust_misc_fixes_0
branch
from
edc9120 to
29a1fb2
Compare
Member
Author
|
Due to the changes made in the 
|
|
HUGE |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This branch addresses some miscellaneous bugs, as well as code smells such as duplicate APIs and large unstructured modules. The intention with this is to keep breaking changes of well used APIs to a minimum, although APIs which are likely unused by external users (e.g. no known consumer other than internal use like tests or bundled plugins).
APIs inside the
architecturemodule were re-exported after they were moved so this is more or less just a DX change, for those reading the crates documentation or maintaining thearchitecturemodule.