Apply suggestions from code review

Co-authored-by: Trent Nelson <trent.a.b.nelson@gmail.com>
This commit is contained in:
Eric Warehime
2022-01-11 11:13:15 -08:00
committed by Trent Nelson
parent b635073829
commit 66a97bdde0

View File

@ -211,12 +211,12 @@ impl RemoteWalletManager {
#[allow(unused_variables)] #[allow(unused_variables)]
pub trait RemoteWallet<T> { pub trait RemoteWallet<T> {
fn name(&self) -> &str { fn name(&self) -> &str {
"remote wallet" "unimplemented"
} }
/// Parse device info and get device base pubkey /// Parse device info and get device base pubkey
fn read_device(&mut self, dev_info: &T) -> Result<RemoteWalletInfo, RemoteWalletError> { fn read_device(&mut self, dev_info: &T) -> Result<RemoteWalletInfo, RemoteWalletError> {
unreachable!(); unimplemented!();
} }
/// Get solana pubkey from a RemoteWallet /// Get solana pubkey from a RemoteWallet
@ -225,7 +225,7 @@ pub trait RemoteWallet<T> {
derivation_path: &DerivationPath, derivation_path: &DerivationPath,
confirm_key: bool, confirm_key: bool,
) -> Result<Pubkey, RemoteWalletError> { ) -> Result<Pubkey, RemoteWalletError> {
unreachable!(); unimplemented!();
} }
/// Sign transaction data with wallet managing pubkey at derivation path m/44'/501'/<account>'/<change>'. /// Sign transaction data with wallet managing pubkey at derivation path m/44'/501'/<account>'/<change>'.
@ -234,7 +234,7 @@ pub trait RemoteWallet<T> {
derivation_path: &DerivationPath, derivation_path: &DerivationPath,
data: &[u8], data: &[u8],
) -> Result<Signature, RemoteWalletError> { ) -> Result<Signature, RemoteWalletError> {
unreachable!(); unimplemented!();
} }
} }