Code cleanup

LedgerHQ · Nov 20, 2023 · 98f2019 · 98f2019
1 parent 9f4c4b3
commit 98f2019
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/src/app_ui/sign.rs b/src/app_ui/sign.rs
@@ -16,11 +16,10 @@
  *****************************************************************************/
 
 use crate::handlers::sign_tx::Tx;
-use crate::AppSW;
 use ledger_device_ui_sdk::bitmaps::{CROSSMARK, EYE, VALIDATE_14};
 use ledger_device_ui_sdk::ui::{Field, MultiFieldReview};
 
-pub fn ui_display_tx(tx: &Tx) -> Result<bool, AppSW> {
+pub fn ui_display_tx(tx: &Tx) -> bool {
  // Define transaction review fields
  let my_fields = [
  Field {
@@ -48,5 +47,5 @@ pub fn ui_display_tx(tx: &Tx) -> Result<bool, AppSW> {
  Some(&CROSSMARK),
  );
 
- Ok(my_review.show())
+ my_review.show()
 }
diff --git a/src/handlers/sign_tx.rs b/src/handlers/sign_tx.rs
@@ -100,7 +100,7 @@ pub fn handler_sign_tx(
  // Display transaction. If user approves
  // the transaction, sign it. Otherwise,
  // return a "deny" status word.
- if ui_display_tx(&tx)? {
+ if ui_display_tx(&tx) {
  return compute_signature_and_append(comm, ctx);
  } else {
  return Err(AppSW::Deny);

diff --git a/src/main.rs b/src/main.rs
@@ -44,7 +44,6 @@ use handlers::{
 
 ledger_device_sdk::set_panic!(ledger_device_sdk::exiting_panic);
 
-use ledger_device_sdk::testing::debug_print;
 // CLA (APDU class byte) for all APDUs.
 const CLA: u8 = 0xe0;
 // P2 for last APDU to receive.