remove TrailingBytes to avoid breaking change

crates/rlp/src/decode.rs

@@ -188,7 +188,8 @@ pub fn decode_exact<T: Decodable>(bytes: impl AsRef<[u8]>) -> Result<T> {
 
     // check if there are any remaining bytes after decoding
     if !buf.is_empty() {
-        return Err(Error::TrailingBytes);
+        // TODO: introduce a new variant TrailingBytes to better distinguish this error
+        return Err(Error::UnexpectedLength);
     }
 
     Ok(out)
@@ -382,7 +383,7 @@ mod tests {
             assert_eq!(decode_exact::<T>(&encoded), Ok(input));
             assert_eq!(
                 decode_exact::<T>([encoded, vec![0x00]].concat()),
-                Err(Error::TrailingBytes)
+                Err(Error::UnexpectedLength)
             );
         }
 

crates/rlp/src/error.rs

@@ -12,8 +12,6 @@ pub enum Error {
     LeadingZero,
     /// Overran input while decoding.
     InputTooShort,
-    /// Additional trailing bytes found after decoding.
-    TrailingBytes,
     /// Expected single byte, but got invalid value.
     NonCanonicalSingleByte,
     /// Expected size, but got invalid value.
@@ -44,7 +42,6 @@ impl fmt::Display for Error {
             Self::Overflow => f.write_str("overflow"),
             Self::LeadingZero => f.write_str("leading zero"),
             Self::InputTooShort => f.write_str("input too short"),
-            Self::TrailingBytes => f.write_str("trailing bytes"),
             Self::NonCanonicalSingleByte => f.write_str("non-canonical single byte"),
             Self::NonCanonicalSize => f.write_str("non-canonical size"),
             Self::UnexpectedLength => f.write_str("unexpected length"),