Update to heapless 0.9.1 and iso7816 0.2.0

dispatch/Cargo.toml

@@ -11,9 +11,9 @@ repository.workspace = true
 [dependencies]
 apdu-app = "0.1.0"
 delog = "0.1.4"
-heapless = "0.7"
 interchange = "0.3.0"
 iso7816.workspace = true
+heapless.workspace = true
 
 [dev-dependencies]
 # Testing

dispatch/src/dispatch.rs

@@ -8,7 +8,6 @@
 //!
 //! Apps need to implement the App trait to be managed.
 //!
-use core::mem;
 
 use crate::response::SIZE as ResponseSize;
 use crate::App;
@@ -123,8 +122,8 @@ impl<'pipe> ApduDispatch<'pipe> {
     // but that won't work due to ownership rules
     fn find_app<'a, 'b>(
         aid: Option<&Aid>,
-        apps: &'a mut [&'b mut dyn App<ResponseSize>],
-    ) -> Option<&'a mut &'b mut dyn App<ResponseSize>> {
+        apps: &'a mut [&'b mut dyn App],
+    ) -> Option<&'a mut &'b mut dyn App> {
         // match aid {
         //     Some(aid) => apps.iter_mut().find(|app| aid.starts_with(app.rid())),
         //     None => None,
@@ -366,12 +365,7 @@ impl<'pipe> ApduDispatch<'pipe> {
     }
 
     #[inline(never)]
-    fn handle_app_select(
-        &mut self,
-        apps: &mut [&mut dyn App<ResponseSize>],
-        aid: Aid,
-        interface: Interface,
-    ) {
+    fn handle_app_select(&mut self, apps: &mut [&mut dyn App], aid: Aid, interface: Interface) {
         // three cases:
         // - currently selected app has different AID -> deselect it, to give it
         //   the chance to clear sensitive state
@@ -412,11 +406,7 @@ impl<'pipe> ApduDispatch<'pipe> {
     }
 
     #[inline(never)]
-    fn handle_app_command(
-        &mut self,
-        apps: &mut [&mut dyn App<ResponseSize>],
-        interface: Interface,
-    ) {
+    fn handle_app_command(&mut self, apps: &mut [&mut dyn App], interface: Interface) {
         // if there is a selected app, send it the command
         let mut response = response::Data::new();
         if let Some(app) = Self::find_app(self.current_aid.as_ref(), apps) {
@@ -431,7 +421,7 @@ impl<'pipe> ApduDispatch<'pipe> {
         };
     }
 
-    pub fn poll(&mut self, apps: &mut [&mut dyn App<ResponseSize>]) -> Option<Interface> {
+    pub fn poll(&mut self, apps: &mut [&mut dyn App]) -> Option<Interface> {
         // Only take on one transaction at a time.
         let request_type = self.check_for_request();
 

dispatch/tests/dispatch.rs

@@ -1,6 +1,7 @@
 use apdu_dispatch::app::{App, CommandView, Result as AppResult};
 use apdu_dispatch::dispatch;
 use apdu_dispatch::{interchanges, response};
+use heapless::VecView;
 use hex_literal::hex;
 use interchange::Channel;
 use iso7816::Status;
@@ -47,12 +48,12 @@ impl iso7816::App for TestApp1 {
 }
 
 // This app echos to Ins code 0x10
-impl App<{ apdu_dispatch::response::SIZE }> for TestApp1 {
+impl App for TestApp1 {
     fn select(
         &mut self,
         _interface: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         Ok(())
     }
@@ -63,7 +64,7 @@ impl App<{ apdu_dispatch::response::SIZE }> for TestApp1 {
         &mut self,
         _: dispatch::Interface,
         apdu: CommandView<'_>,
-        reply: &mut response::Data,
+        reply: &mut VecView<u8>,
     ) -> AppResult {
         println!("TestApp1::call");
         match apdu.instruction().into() {
@@ -79,8 +80,8 @@ impl App<{ apdu_dispatch::response::SIZE }> for TestApp1 {
             }
             // For measuring the stack burden of dispatch
             0x15 => {
-                let buf = heapless::Vec::new();
-                let addr = (&buf as *const response::Data) as u32;
+                let buf = heapless::Vec::<u8, { response::SIZE }>::new();
+                let addr = (&buf as *const VecView<u8>).addr() as u32;
                 reply.extend_from_slice(&addr.to_be_bytes()).unwrap();
                 Ok(())
             }
@@ -122,12 +123,12 @@ impl iso7816::App for TestApp2 {
 }
 
 // This app echos to Ins code 0x20
-impl App<{ apdu_dispatch::response::SIZE }> for TestApp2 {
+impl App for TestApp2 {
     fn select(
         &mut self,
         _interface: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         Ok(())
     }
@@ -138,7 +139,7 @@ impl App<{ apdu_dispatch::response::SIZE }> for TestApp2 {
         &mut self,
         _: dispatch::Interface,
         apdu: CommandView<'_>,
-        reply: &mut response::Data,
+        reply: &mut VecView<u8>,
     ) -> AppResult {
         println!("TestApp2::call");
         match apdu.instruction().into() {
@@ -174,12 +175,12 @@ impl iso7816::App for PanicApp {
 }
 
 // This app echos to Ins code 0x20
-impl App<{ apdu_dispatch::response::SIZE }> for PanicApp {
+impl App for PanicApp {
     fn select(
         &mut self,
         _interface: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         panic!("Dont call the panic app");
     }
@@ -192,7 +193,7 @@ impl App<{ apdu_dispatch::response::SIZE }> for PanicApp {
         &mut self,
         _: dispatch::Interface,
         _apdu: CommandView<'_>,
-        _reply: &mut response::Data,
+        _reply: &mut VecView<u8>,
     ) -> AppResult {
         panic!("Dont call the panic app");
     }