don't unnecessarily construct bytearrays over and over again

glyph · Kriechi · commit f4c1b0cade24 · 2025-11-20T19:13:11.000+01:00
diff --git a/src/wsproto/connection.py b/src/wsproto/connection.py
@@ -191,7 +191,7 @@ def events(self) -> Generator[Event, None, None]:
                 elif frame.opcode is Opcode.BINARY:
                     assert isinstance(frame.payload, (bytes, bytearray))
                     yield BytesMessage(
-                        data=bytearray(frame.payload),
+                        data=frame.payload,
                         frame_finished=frame.frame_finished,
                         message_finished=frame.message_finished,
                     )

Original file line number	Diff line number	Diff line change
`@@ -191,7 +191,7 @@ def events(self) -> Generator[Event, None, None]:`
`191`	`191`	`elif frame.opcode is Opcode.BINARY:`
`192`	`192`	`assert isinstance(frame.payload, (bytes, bytearray))`
`193`	`193`	`yield BytesMessage(`
`194`		`- data=bytearray(frame.payload),`
	`194`	`+ data=frame.payload,`
`195`	`195`	`frame_finished=frame.frame_finished,`
`196`	`196`	`message_finished=frame.message_finished,`
`197`	`197`	`)`