[Rust] Refactor `binary_view` module

- Remove the "viral" `BinaryViewExt` trait and its blanket impl
- Split up the binary view type from the custom trait impl
- Simplify and fix bugs regarding custom binary view initialization
- Rewrite Minidump binary view example, parses the PE headers to create proper sections now
- Add some extra documentation
- Add unit test for custom binary view

3 files changed, 2 insertions, 3 deletions

diff --git a/plugins/svd/src/lib.rs b/plugins/svd/src/lib.rs
index 1eb84a5f..ea37b783 100644
--- a/plugins/svd/src/lib.rs
+++ b/plugins/svd/src/lib.rs
@@ -3,7 +3,7 @@ pub mod settings;
 
 use crate::mapper::DeviceMapper;
 use crate::settings::LoadSettings;
-use binaryninja::binary_view::{BinaryView, BinaryViewBase, BinaryViewExt};
+use binaryninja::binary_view::{BinaryView, BinaryViewBase};
 use binaryninja::command::Command;
 use binaryninja::interaction::{Form, FormInputField};
 use binaryninja::workflow::{activity, Activity, AnalysisContext, Workflow};
diff --git a/plugins/svd/src/mapper.rs b/plugins/svd/src/mapper.rs
index 1ba7f61c..4ec02b9e 100644
--- a/plugins/svd/src/mapper.rs
+++ b/plugins/svd/src/mapper.rs
@@ -1,5 +1,5 @@
 use crate::settings::LoadSettings;
-use binaryninja::binary_view::{BinaryView, BinaryViewExt};
+use binaryninja::binary_view::BinaryView;
 use binaryninja::confidence::{Conf, MAX_CONFIDENCE};
 use binaryninja::data_buffer::DataBuffer;
 use binaryninja::rc::Ref;
diff --git a/plugins/svd/tests/mapper.rs b/plugins/svd/tests/mapper.rs
index 8af8c261..845bfd61 100644
--- a/plugins/svd/tests/mapper.rs
+++ b/plugins/svd/tests/mapper.rs
@@ -1,4 +1,3 @@
-use binaryninja::binary_view::BinaryViewExt;
 use binaryninja::binary_view::{BinaryView, BinaryViewBase};
 use binaryninja::file_metadata::FileMetadata;
 use binaryninja::headless::Session;