From 85eda917c444d56424622e2814ae096ce32431e1 Mon Sep 17 00:00:00 2001 From: iximeow Date: Tue, 6 Jul 2021 19:02:38 -0700 Subject: bump to yaxpeax-arch 0.2.3, fix API changes --- CHANGELOG | 7 ++++++ Cargo.toml | 6 ++--- src/lib.rs | 85 ++++++++++++++++---------------------------------------------- 3 files changed, 32 insertions(+), 66 deletions(-) create mode 100644 CHANGELOG diff --git a/CHANGELOG b/CHANGELOG new file mode 100644 index 0000000..ee75039 --- /dev/null +++ b/CHANGELOG @@ -0,0 +1,7 @@ +# 0.1.0 + +update yaxpeax-arch to 0.2.3, fix some API differences + +# 0.0.4 + +first version with changelog, decoder is complete but untested diff --git a/Cargo.toml b/Cargo.toml index 061056b..e3289a2 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -1,14 +1,14 @@ [package] name = "yaxpeax-pic18" -version = "0.0.4" +version = "0.1.0" authors = [ "iximeow " ] license = "0BSD" repository = "http://git.iximeow.net/yaxpeax-pic18/" -description = "pic18 decoders for the yaxpeax project" +description = "pic18 decoder for the yaxpeax project" [dependencies] -yaxpeax-arch = { version = "0.0.5", default-features = false, features = [] } +yaxpeax-arch = { version = "0.2.3", default-features = false, features = [] } "serde" = { version = "1.0", optional = true } "serde_derive" = { version = "1.0", optional = true } diff --git a/src/lib.rs b/src/lib.rs index 74937fb..9fe0228 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -5,27 +5,22 @@ extern crate serde; //#[cfg(feature="use-serde")] //use serde::{Serialize, Deserialize}; -use std::fmt; - extern crate yaxpeax_arch; -use yaxpeax_arch::{Arch, AddressDiff, Decoder, LengthedInstruction}; +use yaxpeax_arch::{Arch, AddressDiff, Decoder, LengthedInstruction, Reader, StandardDecodeError}; pub mod consts; pub mod display; -#[cfg(feature="use-serde")] -#[derive(Debug, Serialize, Deserialize)] -pub struct PIC18; - -#[cfg(not(feature="use-serde"))] +#[cfg_attr(feature="use-serde", derive(Serialize, Deserialize))] #[derive(Debug)] pub struct PIC18; impl Arch for PIC18 { type Address = u32; + type Word = u8; type Instruction = Instruction; - type DecodeError = DecodeError; + type DecodeError = StandardDecodeError; type Decoder = InstDecoder; type Operand = Operand; } @@ -56,29 +51,6 @@ impl LengthedInstruction for Instruction { } } -#[derive(Debug, PartialEq)] -pub enum DecodeError { - ExhaustedInput, - InvalidOpcode, - InvalidOperand, -} - -impl fmt::Display for DecodeError { - fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result { - match self { - DecodeError::ExhaustedInput => write!(f, "exhausted input"), - DecodeError::InvalidOpcode => write!(f, "invalid opcode"), - DecodeError::InvalidOperand => write!(f, "invalid operand"), - } - } -} - -impl yaxpeax_arch::DecodeError for DecodeError { - fn data_exhausted(&self) -> bool { self == &DecodeError::ExhaustedInput } - fn bad_opcode(&self) -> bool { self == &DecodeError::InvalidOpcode } - fn bad_operand(&self) -> bool { self == &DecodeError::InvalidOperand } -} - impl yaxpeax_arch::Instruction for Instruction { // TODO: this is wrong!! fn well_defined(&self) -> bool { true } @@ -193,15 +165,10 @@ pub enum Operand { #[derive(Default, Debug)] pub struct InstDecoder {} -impl Decoder for InstDecoder { - type Error = DecodeError; - - fn decode_into>(&self, inst: &mut Instruction, bytes: T) -> Result<(), Self::Error> { - let mut bytes_iter = bytes.into_iter(); - let word: Vec = bytes_iter.by_ref().take(2).collect(); - if word.len() != 2 { - return Err(DecodeError::ExhaustedInput); - } +impl Decoder for InstDecoder { + fn decode_into::Address, ::Word>>(&self, inst: &mut Instruction, words: &mut T) -> Result<(), ::DecodeError> { + let mut word = [0u8; 2]; + words.next_n(&mut word)?; // println!("Decoding {:x?}", word); match word[1] { @@ -289,7 +256,7 @@ impl Decoder for InstDecoder { }, _ => { inst.opcode = Opcode::Invalid(word[0], word[1]); - Err(DecodeError::InvalidOpcode) + Err(StandardDecodeError::InvalidOpcode) } } }, @@ -413,12 +380,10 @@ impl Decoder for InstDecoder { }, x if x >= 0b11000000 && x < 0b11010000 => { inst.opcode = Opcode::MOVFF; - let word2: Vec = bytes_iter.take(2).collect(); - if word2.len() != 2 { - return Err(DecodeError::ExhaustedInput); - } + let mut word2 = [0u8; 2]; + words.next_n(&mut word2)?; if word2[1] & 0xf0 != 0xf0 { - return Err(DecodeError::InvalidOperand); + return Err(StandardDecodeError::InvalidOperand); } let src = (word[0] as u16) | ((word[1] as u16 & 0x0f) << 8); @@ -452,13 +417,11 @@ impl Decoder for InstDecoder { }, 0xee => { let f_k_msb = word[0]; - let word2: Vec = bytes_iter.take(2).collect(); - if word2.len() != 2 { - return Err(DecodeError::ExhaustedInput); - } + let mut word2 = [0u8; 2]; + words.next_n(&mut word2)?; if (word2[1] & 0xf0) != 0xf0 { - return Err(DecodeError::InvalidOperand); + return Err(StandardDecodeError::InvalidOperand); } inst.opcode = Opcode::LFSR; @@ -475,13 +438,11 @@ impl Decoder for InstDecoder { 0xeb | 0xec => { // TODO: respect s bit let k_lsb = word[0]; - let word2: Vec = bytes_iter.take(2).collect(); - if word2.len() != 2 { - return Err(DecodeError::ExhaustedInput); - } + let mut word2 = [0u8; 2]; + words.next_n(&mut word2)?; if (word2[1] & 0xf0) != 0xf0 { - return Err(DecodeError::InvalidOperand); + return Err(StandardDecodeError::InvalidOperand); } let k_msb = (((word2[1] & 0xf) as u32) << 8) | word2[0] as u32; @@ -492,13 +453,11 @@ impl Decoder for InstDecoder { } 0xef => { let k_lsb = word[0]; - let word2: Vec = bytes_iter.take(2).collect(); - if word2.len() != 2 { - return Err(DecodeError::ExhaustedInput); - } + let mut word2 = [0u8; 2]; + words.next_n(&mut word2)?; if (word2[1] & 0xf0) != 0xf0 { - return Err(DecodeError::InvalidOperand); + return Err(StandardDecodeError::InvalidOperand); } let k_msb = (((word2[1] & 0xf) as u32) << 8) | word2[0] as u32; @@ -507,7 +466,7 @@ impl Decoder for InstDecoder { inst.operands[0] = Operand::ImmediateU32(((k_msb << 8) | k_lsb as u32) << 1); Ok(()) } - _ => Err(DecodeError::InvalidOpcode) + _ => Err(StandardDecodeError::InvalidOpcode) } } } -- cgit v1.1