Skip to content

Commit

Permalink
Remove unnecessary #[allow(dead_code)]
Browse files Browse the repository at this point in the history
During the switch to new serialization API `#[allow(dead_code)]` was
added in several places as a temporary measure. Those can now be removed.
  • Loading branch information
Lorak-mmk committed Dec 11, 2023
1 parent 001fd2a commit bd5a6c1
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
2 changes: 0 additions & 2 deletions scylla-cql/src/types/serialize/row.rs
Original file line number Diff line number Diff line change
Expand Up @@ -677,7 +677,6 @@ impl SerializedValues {
}

/// Creates value list from the request frame
#[allow(dead_code)]
pub(crate) fn new_from_frame(buf: &mut &[u8]) -> Result<Self, ParseError> {
let values_num = types::read_short(buf)?;
let values_beg = *buf;
Expand All @@ -694,7 +693,6 @@ impl SerializedValues {
}

// Temporary function, to be removed when we implement new batching API (right now it is needed in frame::request::mod.rs tests)
#[allow(dead_code)]
pub fn to_old_serialized_values(&self) -> LegacySerializedValues {
let mut frame = Vec::new();
self.write_to_request(&mut frame);
Expand Down
1 change: 0 additions & 1 deletion scylla/src/statement/prepared_statement.rs
Original file line number Diff line number Diff line change
Expand Up @@ -338,7 +338,6 @@ impl PreparedStatement {
self.config.execution_profile_handle.as_ref()
}

#[allow(dead_code)]
pub(crate) fn serialize_values(
&self,
values: &impl SerializeRow,
Expand Down

0 comments on commit bd5a6c1

Please sign in to comment.