aquatic_ws: InMessage::to_ws_message: don't take ownership

This commit is contained in:
Joakim Frostegård 2020-08-11 23:24:46 +02:00
parent 63e5fe11c9
commit de1a299f1e
2 changed files with 8 additions and 7 deletions

View file

@ -276,7 +276,7 @@ pub fn send_out_messages(
use ::tungstenite::Error::Io;
let ws_message = out_message.into_ws_message();
let ws_message = out_message.to_ws_message();
match established_ws.ws.write_message(ws_message){
Ok(()) => {

View file

@ -257,6 +257,11 @@ pub enum InMessage {
impl InMessage {
#[inline]
pub fn to_ws_message(&self) -> ::tungstenite::Message {
::tungstenite::Message::from(::serde_json::to_string(&self).unwrap())
}
#[inline]
pub fn from_ws_message(
ws_message: tungstenite::Message
@ -272,10 +277,6 @@ impl InMessage {
return ::simd_json::serde::from_str(&mut text)
.context("deserialize with serde");
}
pub fn to_ws_message(&self) -> ::tungstenite::Message {
::tungstenite::Message::from(::serde_json::to_string(&self).unwrap())
}
}
@ -291,7 +292,7 @@ pub enum OutMessage {
impl OutMessage {
#[inline]
pub fn into_ws_message(self) -> tungstenite::Message {
pub fn to_ws_message(&self) -> tungstenite::Message {
::tungstenite::Message::from(::serde_json::to_string(&self).unwrap())
}
@ -545,7 +546,7 @@ mod tests {
#[quickcheck]
fn quickcheck_serde_identity_out_message(out_message_1: OutMessage) -> bool {
let ws_message = out_message_1.clone().into_ws_message();
let ws_message = out_message_1.to_ws_message();
let out_message_2 = OutMessage::from_ws_message(
ws_message.clone()