Skip to content

Commit a103e9e

Browse files
committed
part/fpt: rename errors
Change the order into verb-object-error as suggested in https://rust-lang.github.io/api-guidelines/naming.html#c-word-order Signed-off-by: Daniel Maslowski <info@orangecms.org>
1 parent ef4e982 commit a103e9e

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

src/part/fpt.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -139,8 +139,8 @@ impl Display for FPTHeader {
139139

140140
#[derive(Debug)]
141141
pub enum FptError<'a> {
142-
HeaderParseError(SizeError<&'a [u8], FPTHeader>),
143-
EntryParseError(
142+
ParseHeaderError(SizeError<&'a [u8], FPTHeader>),
143+
ParseEntryError(
144144
ConvertError<
145145
AlignmentError<&'a [u8], [FPTEntry]>,
146146
SizeError<&'a [u8], [FPTEntry]>,
@@ -242,14 +242,14 @@ impl<'a> FPT {
242242
let d = &data[offset..];
243243
let header = match FPTHeader::read_from_prefix(d) {
244244
Ok((h, _)) => h,
245-
Err(e) => return Some(Err(FptError::HeaderParseError(e))),
245+
Err(e) => return Some(Err(FptError::ParseHeaderError(e))),
246246
};
247247
// NOTE: Skip $FPT (header) itself
248248
let slice = &d[FPT_HEADER_SIZE..];
249249
let count = header.entries as usize;
250250
let entries = match Ref::<_, [FPTEntry]>::from_prefix_with_elems(slice, count) {
251251
Ok((r, _)) => r,
252-
Err(e) => return Some(Err(FptError::EntryParseError(e))),
252+
Err(e) => return Some(Err(FptError::ParseEntryError(e))),
253253
};
254254

255255
let original_size = pre_header.len() + FPT_HEADER_SIZE + FPT_ENTRY_SIZE * entries.len();
@@ -327,7 +327,7 @@ fn parse_size_error() {
327327
let parsed = FPT::parse(&DATA[..70]);
328328
assert!(parsed.is_some());
329329
let fpt_res = parsed.unwrap();
330-
assert!(matches!(fpt_res, Err(FptError::EntryParseError(_))));
330+
assert!(matches!(fpt_res, Err(FptError::ParseEntryError(_))));
331331
}
332332

333333
#[test]

0 commit comments

Comments
 (0)