[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

6 files changed, 6 insertions, 9 deletions

diff --git a/plugins/workflow_objc/src/activities/alloc_init.rs b/plugins/workflow_objc/src/activities/alloc_init.rs
index 5c8bf064..953e0738 100644
--- a/plugins/workflow_objc/src/activities/alloc_init.rs
+++ b/plugins/workflow_objc/src/activities/alloc_init.rs
@@ -1,8 +1,5 @@
 use binaryninja::{
-    binary_view::{BinaryView, BinaryViewExt as _},
-    medium_level_il::MediumLevelILLiftedInstruction,
-    rc::Ref,
-    types::Type,
+    binary_view::BinaryView, medium_level_il::MediumLevelILLiftedInstruction, rc::Ref, types::Type,
     workflow::AnalysisContext,
 };
 use bstr::ByteSlice;
diff --git a/plugins/workflow_objc/src/activities/objc_msg_send_calls.rs b/plugins/workflow_objc/src/activities/objc_msg_send_calls.rs
index 157bb19c..41fc38c0 100644
--- a/plugins/workflow_objc/src/activities/objc_msg_send_calls.rs
+++ b/plugins/workflow_objc/src/activities/objc_msg_send_calls.rs
@@ -1,5 +1,5 @@
 use binaryninja::{
-    binary_view::{BinaryView, BinaryViewExt as _},
+    binary_view::BinaryView,
     function::Function,
     low_level_il::{
         expression::{ExpressionHandler as _, LowLevelILExpressionKind},
diff --git a/plugins/workflow_objc/src/activities/objc_msg_send_calls/adjust_call_type.rs b/plugins/workflow_objc/src/activities/objc_msg_send_calls/adjust_call_type.rs
index 07e83ba7..4d22476a 100644
--- a/plugins/workflow_objc/src/activities/objc_msg_send_calls/adjust_call_type.rs
+++ b/plugins/workflow_objc/src/activities/objc_msg_send_calls/adjust_call_type.rs
@@ -1,6 +1,6 @@
 use binaryninja::{
     architecture::CoreRegister,
-    binary_view::{BinaryView, BinaryViewBase as _, BinaryViewExt},
+    binary_view::{BinaryView, BinaryViewBase as _},
     confidence::Conf,
     function::Function,
     low_level_il::{
diff --git a/plugins/workflow_objc/src/activities/remove_memory_management.rs b/plugins/workflow_objc/src/activities/remove_memory_management.rs
index cd613ee2..a6389d22 100644
--- a/plugins/workflow_objc/src/activities/remove_memory_management.rs
+++ b/plugins/workflow_objc/src/activities/remove_memory_management.rs
@@ -1,6 +1,6 @@
 use binaryninja::{
     architecture::{Architecture as _, CoreRegister, Register as _, RegisterInfo as _},
-    binary_view::{BinaryView, BinaryViewExt as _},
+    binary_view::BinaryView,
     low_level_il::{
         expression::{ExpressionHandler, LowLevelILExpressionKind},
         function::{LowLevelILFunction, Mutable, NonSSA},
diff --git a/plugins/workflow_objc/src/activities/super_init.rs b/plugins/workflow_objc/src/activities/super_init.rs
index 7b6bb3f7..533fe9e6 100644
--- a/plugins/workflow_objc/src/activities/super_init.rs
+++ b/plugins/workflow_objc/src/activities/super_init.rs
@@ -1,5 +1,5 @@
 use binaryninja::{
-    binary_view::{BinaryView, BinaryViewBase as _, BinaryViewExt as _},
+    binary_view::{BinaryView, BinaryViewBase as _},
     medium_level_il::{
         operation::{Constant, LiftedSetVarSsa, LiftedSetVarSsaField, Var, VarSsa},
         MediumLevelILLiftedInstruction, MediumLevelILLiftedInstructionKind,
diff --git a/plugins/workflow_objc/src/activities/util.rs b/plugins/workflow_objc/src/activities/util.rs
index 3ac7df7c..d9acf29f 100644
--- a/plugins/workflow_objc/src/activities/util.rs
+++ b/plugins/workflow_objc/src/activities/util.rs
@@ -1,5 +1,5 @@
 use binaryninja::{
-    binary_view::{BinaryView, BinaryViewExt as _},
+    binary_view::BinaryView,
     confidence::Conf,
     function::Function,
     medium_level_il::{