f_errors: remove BadIncomingData from ParseError

We remove last usage (an impl) of this variant.

This variant was the most awful, since it stringified many errors.
We fortunately can remove it now.

scylla-cql/src/frame/frame_errors.rs

@@ -64,8 +64,6 @@ pub enum FrameSerializationError {
 pub enum ParseError {
     #[error("Low-level deserialization failed: {0}")]
     LowLevelDeserializationError(#[from] LowLevelDeserializationError),
-    #[error("Could not deserialize frame: {0}")]
-    BadIncomingData(String),
     #[error(transparent)]
     TryFromIntError(#[from] std::num::TryFromIntError),
 }

scylla-cql/src/frame/request/batch.rs

@@ -4,7 +4,7 @@ use thiserror::Error;
 
 use crate::{
     frame::{
-        frame_errors::{CqlRequestSerializationError, ParseError},
+        frame_errors::CqlRequestSerializationError,
         request::{RequestOpcode, SerializableRequest},
         types::{self, SerialConsistency},
     },
@@ -52,12 +52,6 @@ pub struct BatchTypeParseError {
     value: u8,
 }
 
-impl From<BatchTypeParseError> for ParseError {
-    fn from(err: BatchTypeParseError) -> Self {
-        Self::BadIncomingData(format!("Bad BatchType value: {}", err.value))
-    }
-}
-
 impl TryFrom<u8> for BatchType {
     type Error = BatchTypeParseError;