eliott/ladybird
1
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird synced 2026-04-25 17:25:08 +02:00
Code Issues Packages Projects Releases Wiki Activity

LibWebSocket: Remove meaningless const&& Message constructor

Browse Source 
This results in an ambiguity error in an upcoming commit.
This commit is contained in:
Timothy Flynn
2025-03-08 12:10:27 -05:00
committed by Tim Flynn
parent 5f76324af5
commit 8f6169859d
Notes: github-actions[bot] 2025-03-09 15:15:36 +00:00 
Author: https://github.com/trflynn89
Commit: https://github.com/LadybirdBrowser/ladybird/commit/8f6169859d0
Pull-request: https://github.com/LadybirdBrowser/ladybird/pull/3865
2 changed files with 3 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
Libraries/LibWebSocket/Message.h
View File

@@ -20,13 +20,7 @@ public:
{
}
explicit Message(ByteBuffer data, bool is_text)
: m_is_text(is_text)
, m_data(move(data))
{
}
explicit Message(ByteBuffer const&& data, bool is_text)
Message(ByteBuffer data, bool is_text)
: m_is_text(is_text)
, m_data(move(data))
{

4
Libraries/LibWebSocket/WebSocket.cpp
View File

@@ -525,11 +525,11 @@ void WebSocket::read_frame()
m_fragmented_data_buffer.clear();
}
if (op_code == WebSocket::OpCode::Text) {
notify_message(Message(payload, true));
notify_message(Message(move(payload), true));
return;
}
if (op_code == WebSocket::OpCode::Binary) {
notify_message(Message(payload, false));
notify_message(Message(move(payload), false));
return;
}
dbgln("Websocket: Found unknown opcode {}", (u8)op_code);