hfw / socket

src/WebSocket/WebSocketClient.php

Spacing +15 added lines, -15 removed lines

@@ -51,7 +51,7 @@ 
      * @param $resource
      * @param WebSocketServer $server
      */
-    public function __construct ($resource, WebSocketServer $server) {
+    public function __construct($resource, WebSocketServer $server) {
         parent::__construct($resource);
         $this->server = $server;
     }
@@ -74,7 +74,7 @@ 
      * @param string $reason
      * @return StreamClient|void
      */
-    public function close (int $code = null, string $reason = '') {
+    public function close(int $code = null, string $reason = '') {
         try {
             if ($code >= 1000 and $this->state === self::STATE_OK) {
                 $this->getFrameHandler()->writeClose($code, $reason);
@@ -91,49 +91,49 @@ 
     /**
      * @return FrameHandler
      */
-    public function getFrameHandler () {
+    public function getFrameHandler() {
         return $this->frameHandler ?? $this->frameHandler = new FrameHandler($this);
     }
 
     /**
      * @return FrameReader
      */
-    public function getFrameReader () {
+    public function getFrameReader() {
         return $this->frameReader ?? $this->frameReader = new FrameReader($this);
     }
 
     /**
      * @return HandShake
      */
-    public function getHandshake () {
+    public function getHandshake() {
         return $this->handshake ?? $this->handshake = new HandShake($this);
     }
 
     /**
      * @return MessageHandler
      */
-    public function getMessageHandler () {
+    public function getMessageHandler() {
         return $this->messageHandler ?? $this->messageHandler = new MessageHandler($this);
     }
 
     /**
      * @return WebSocketServer
      */
-    public function getServer () {
+    public function getServer() {
         return $this->server;
     }
 
     /**
      * @return int
      */
-    public function getState (): int {
+    public function getState(): int {
         return $this->state;
     }
 
     /**
      * @return bool
      */
-    final public function isOk (): bool {
+    final public function isOk(): bool {
         return $this->state === self::STATE_OK;
     }
 
@@ -142,7 +142,7 @@ 
      *
      * The RFC says the connection must be dropped if any unsupported activity occurs.
      */
-    final public function onOutOfBand (): void {
+    final public function onOutOfBand(): void {
         $this->close(Frame::CLOSE_PROTOCOL_ERROR, "Received out-of-band data.");
     }
 
@@ -151,7 +151,7 @@ 
      *
      * @throws Exception
      */
-    public function onReadable (): void {
+    public function onReadable(): void {
         if (!strlen($this->recv(1, MSG_PEEK))) { // peer has shut down writing, or closed.
             $this->close();
             return;
@@ -187,7 +187,7 @@ 
     /**
      * Stub.
      */
-    protected function onStateOk (): void {
+    protected function onStateOk(): void {
 
     }
 
@@ -195,7 +195,7 @@ 
      * @param FrameHandler $frameHandler
      * @return $this
      */
-    public function setFrameHandler (FrameHandler $frameHandler) {
+    public function setFrameHandler(FrameHandler $frameHandler) {
         $this->frameHandler = $frameHandler;
         return $this;
     }
@@ -204,7 +204,7 @@ 
      * @param FrameReader $frameReader
      * @return $this
      */
-    public function setFrameReader (FrameReader $frameReader) {
+    public function setFrameReader(FrameReader $frameReader) {
         $this->frameReader = $frameReader;
         return $this;
     }
@@ -213,7 +213,7 @@ 
      * @param MessageHandler $messageHandler
      * @return $this
      */
-    public function setMessageHandler (MessageHandler $messageHandler) {
+    public function setMessageHandler(MessageHandler $messageHandler) {
         $this->messageHandler = $messageHandler;
         return $this;
     }

Braces +3 added lines, -6 removed lines

@@ -80,8 +80,7 @@ 
                 $this->getFrameHandler()->writeClose($code, $reason);
                 $this->shutdown(self::CH_WRITE);
             }
-        }
-        finally {
+        } finally {
             $this->state = self::STATE_CLOSE;
             $this->server->remove($this);
             parent::close();
@@ -173,12 +172,10 @@ 
                 case self::STATE_CLOSE:
                     return;
             }
-        }
-        catch (WebSocketError $e) {
+        } catch (WebSocketError $e) {
             $this->close($e->getCode(), $e->getMessage());
             throw $e;
-        }
-        catch (Exception $e) {
+        } catch (Exception $e) {
             $this->close(Frame::CLOSE_INTERNAL_ERROR);
             throw $e;
         }

src/WebSocket/MessageHandler.php

Spacing +4 added lines, -4 removed lines

@@ -12,14 +12,14 @@ 
     /**
      * @param WebSocketClient $client
      */
-    public function __construct (WebSocketClient $client) {
+    public function __construct(WebSocketClient $client) {
         $this->client = $client;
     }
 
     /**
      * @param string $binary
      */
-    public function onBinary (string $binary): void {
+    public function onBinary(string $binary): void {
         unset($binary);
         throw new WebSocketError(Frame::CLOSE_UNHANDLED_DATA, "I don't handle binary data.");
     }
@@ -27,7 +27,7 @@ 
     /**
      * @param string $text
      */
-    public function onText (string $text): void {
+    public function onText(string $text): void {
         $this->onText_CheckUtf8($text);
         throw new WebSocketError(Frame::CLOSE_UNHANDLED_DATA, "I don't handle text.");
     }
@@ -37,7 +37,7 @@ 
      *
      * @param string $text
      */
-    protected function onText_CheckUtf8 (string $text): void {
+    protected function onText_CheckUtf8(string $text): void {
         if (!mb_detect_encoding($text, 'UTF-8', true)) {
             throw new WebSocketError(Frame::CLOSE_BAD_DATA, "Received TEXT is not UTF-8.");
         }

src/WebSocket/FrameReader.php

Spacing +6 added lines, -6 removed lines

@@ -46,14 +46,14 @@ 
      */
     protected $maxLength = 10 * 1024 * 1024;
 
-    public function __construct (WebSocketClient $client) {
+    public function __construct(WebSocketClient $client) {
         $this->client = $client;
     }
 
     /**
      * @return Frame|null
      */
-    protected function getFrame () {
+    protected function getFrame() {
         if (!$this->head) {
             if (preg_match(self::REGEXP, $this->buffer, $head)) {
                 [, $op, $len] = unpack('C2', $head[0]);
@@ -95,7 +95,7 @@ 
      *
      * @return Generator|Frame[]
      */
-    public function getFrames () {
+    public function getFrames() {
         $this->buffer .= $this->client->recvAll();
         while ($frame = $this->getFrame()) {
             yield $frame;
@@ -105,7 +105,7 @@ 
     /**
      * @return int
      */
-    public function getMaxLength (): int {
+    public function getMaxLength(): int {
         return $this->maxLength;
     }
 
@@ -113,7 +113,7 @@ 
      * @param int $bytes
      * @return $this
      */
-    public function setMaxLength (int $bytes) {
+    public function setMaxLength(int $bytes) {
         if ($bytes < self::MAX_LENGTH_RANGE[0] or $bytes > self::MAX_LENGTH_RANGE[1]) {
             throw new InvalidArgumentException('Max length must be within range [125,2^63-1]');
         }
@@ -126,7 +126,7 @@ 
      *
      * @throws WebSocketError
      */
-    protected function validate (): void {
+    protected function validate(): void {
         if ($this->head['length'] > $this->maxLength) {
             throw new WebSocketError(Frame::CLOSE_TOO_LARGE, "Payload would exceed {$this->maxLength} bytes");
         }

Braces +3 added lines, -6 removed lines

@@ -67,11 +67,9 @@ 
                 ];
                 $this->buffer = substr($this->buffer, strlen($head[0]));
                 $this->validate();
-            }
-            elseif (strlen($this->buffer) >= 14) { // max head room
+            } elseif (strlen($this->buffer) >= 14) { // max head room
                 throw new WebSocketError(Frame::CLOSE_PROTOCOL_ERROR, 'Bad frame.');
-            }
-            else {
+            } else {
                 return null;
             }
         }
@@ -139,8 +137,7 @@ 
             if (!$this->head['final']) {
                 throw new WebSocketError(Frame::CLOSE_PROTOCOL_ERROR, "Received fragmented {$name}");
             }
-        }
-        elseif ($opCode > Frame::OP_BINARY) { // data
+        } elseif ($opCode > Frame::OP_BINARY) { // data
             throw new WebSocketError(Frame::CLOSE_PROTOCOL_ERROR, "Received {$name}");
         }
     }

Doc Comments +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
     /**
      * Constructs and yields all available frames from the peer.
      *
-     * @return Generator|Frame[]
+     * @return Generator
      */
     public function getFrames () {
         $this->buffer .= $this->client->recvAll();

src/AbstractServer.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
      *
      * @return string
      */
-    public function __toString () {
+    public function __toString() {
         try {
             return implode(':', $this->getSockName());
         }
@@ -40,7 +40,7 @@ 
      * @return $this
      * @throws SocketError
      */
-    public function bind (string $address, int $port = 0) {
+    public function bind(string $address, int $port = 0) {
         if (!@socket_bind($this->resource, $address, $port)) {
             throw new SocketError($this->resource, SOCKET_EOPNOTSUPP);
         }

Braces +1 added lines, -2 removed lines

@@ -21,8 +21,7 @@
     public function __toString () {
         try {
             return implode(':', $this->getSockName());
-        }
-        catch (Throwable $e) {
+        } catch (Throwable $e) {
             return "?{$this->resource}";
         }
     }

src/StreamServer.php

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
      *
      * @return int
      */
-    final public static function getType (): int {
+    final public static function getType(): int {
         return SOCK_STREAM;
     }
 
@@ -25,7 +25,7 @@ 
      * @return StreamClient
      * @throws SocketError
      */
-    public function accept () {
+    public function accept() {
         if (!$resource = @socket_accept($this->resource)) {
             throw new SocketError($this->resource); // reliable errno
         }
@@ -45,7 +45,7 @@ 
      * @return $this
      * @throws SocketError
      */
-    public function listen (int $backlog = 0) {
+    public function listen(int $backlog = 0) {
         if (!@socket_listen($this->resource, $backlog)) {
             throw new SocketError($this->resource); // reliable errno
         }
@@ -58,7 +58,7 @@ 
      * @param resource $resource The accepted connection.
      * @return StreamClient
      */
-    protected function newClient ($resource) {
+    protected function newClient($resource) {
         return new StreamClient($resource);
     }
 

src/DatagramServer.php

Spacing +3 added lines, -3 removed lines

@@ -12,7 +12,7 @@ 
      *
      * @return int
      */
-    final public static function getType (): int {
+    final public static function getType(): int {
         return SOCK_DGRAM;
     }
 
@@ -28,12 +28,12 @@ 
      * @return string
      * @throws SocketError
      */
-    public function recv (int $length, int $flags = 0, string &$name = null, int &$port = 0): string {
+    public function recv(int $length, int $flags = 0, string &$name = null, int &$port = 0): string {
         $count = @socket_recvfrom($this->resource, $data, $length, $flags, $name, $port);
         if ($count === false) {
             throw new SocketError($this->resource, SOCKET_EOPNOTSUPP);
         }
-        return (string)$data; // cast needed, will be null if 0 bytes are read
+        return (string) $data; // cast needed, will be null if 0 bytes are read
     }
 
 }
\ No newline at end of file

src/WebSocket/Frame.php

Spacing +33 added lines, -33 removed lines

@@ -41,15 +41,15 @@ 
         0x0f => 'RESERVED CONTROL 0x0f',
     ];
 
-    const CLOSE_NORMAL = 1000;              // mutual closure
-    const CLOSE_INTERRUPT = 1001;           // abrupt closure due to hangups, reboots, "going away"
-    const CLOSE_PROTOCOL_ERROR = 1002;      // invalid behavior / framing
-    const CLOSE_UNHANDLED_DATA = 1003;      // message handler doesn't want the payload
-    const CLOSE_BAD_DATA = 1007;            // message handler can't understand the payload
-    const CLOSE_POLICY_VIOLATION = 1008;    // generic "access denied"
-    const CLOSE_TOO_LARGE = 1009;           // unacceptable payload size
-    const CLOSE_EXPECTATION = 1010;         // peer closed because it wants extensions (listed in the reason)
-    const CLOSE_INTERNAL_ERROR = 1011;      // like http 500
+    const CLOSE_NORMAL = 1000; // mutual closure
+    const CLOSE_INTERRUPT = 1001; // abrupt closure due to hangups, reboots, "going away"
+    const CLOSE_PROTOCOL_ERROR = 1002; // invalid behavior / framing
+    const CLOSE_UNHANDLED_DATA = 1003; // message handler doesn't want the payload
+    const CLOSE_BAD_DATA = 1007; // message handler can't understand the payload
+    const CLOSE_POLICY_VIOLATION = 1008; // generic "access denied"
+    const CLOSE_TOO_LARGE = 1009; // unacceptable payload size
+    const CLOSE_EXPECTATION = 1010; // peer closed because it wants extensions (listed in the reason)
+    const CLOSE_INTERNAL_ERROR = 1011; // like http 500
 
     /**
      * @var bool
@@ -79,7 +79,7 @@ 
      * @param int $opCode
      * @param string $payload
      */
-    public function __construct (bool $final, int $rsv, int $opCode, string $payload) {
+    public function __construct(bool $final, int $rsv, int $opCode, string $payload) {
         $this->final = $final;
         $this->rsv = $rsv;
         $this->opCode = $opCode;
@@ -91,7 +91,7 @@ 
      *
      * @return string
      */
-    public function __toString () {
+    public function __toString() {
         if ($this->isClose()) {
             return $this->getCloseReason();
         }
@@ -105,7 +105,7 @@ 
      *
      * @return int
      */
-    final public function getCloseCode (): int {
+    final public function getCloseCode(): int {
         $code = substr($this->payload, 0, 2);
         return isset($code[1]) ? unpack('n', $code)[1] : self::CLOSE_NORMAL;
     }
@@ -117,126 +117,126 @@ 
      *
      * @return string
      */
-    final public function getCloseReason (): string {
+    final public function getCloseReason(): string {
         return substr($this->payload, 2);
     }
 
     /**
      * @return int
      */
-    final public function getLength (): int {
+    final public function getLength(): int {
         return strlen($this->payload);
     }
 
     /**
      * @return string
      */
-    public function getName (): string {
+    public function getName(): string {
         return self::NAMES[$this->opCode];
     }
 
     /**
      * @return int
      */
-    final public function getOpCode (): int {
+    final public function getOpCode(): int {
         return $this->opCode;
     }
 
     /**
      * @return string
      */
-    final public function getPayload (): string {
+    final public function getPayload(): string {
         return $this->payload;
     }
 
     /**
      * @return int
      */
-    final public function getRsv (): int {
+    final public function getRsv(): int {
         return $this->rsv;
     }
 
     /**
      * @return bool
      */
-    final public function hasRsv1 (): bool {
-        return (bool)($this->rsv & self::RSV1);
+    final public function hasRsv1(): bool {
+        return (bool) ($this->rsv & self::RSV1);
     }
 
     /**
      * @return bool
      */
-    final public function hasRsv2 (): bool {
-        return (bool)($this->rsv & self::RSV2);
+    final public function hasRsv2(): bool {
+        return (bool) ($this->rsv & self::RSV2);
     }
 
     /**
      * @return bool
      */
-    final public function hasRsv3 (): bool {
-        return (bool)($this->rsv & self::RSV3);
+    final public function hasRsv3(): bool {
+        return (bool) ($this->rsv & self::RSV3);
     }
 
     /**
      * @return bool
      */
-    final public function isBinary (): bool {
+    final public function isBinary(): bool {
         return $this->opCode === self::OP_BINARY;
     }
 
     /**
      * @return bool
      */
-    final public function isClose (): bool {
+    final public function isClose(): bool {
         return $this->opCode === self::OP_CLOSE;
     }
 
     /**
      * @return bool
      */
-    final public function isContinue (): bool {
+    final public function isContinue(): bool {
         return $this->opCode === self::OP_CONTINUE;
     }
 
     /**
      * @return bool
      */
-    final public function isControl (): bool {
+    final public function isControl(): bool {
         return $this->opCode >= self::OP_CLOSE;
     }
 
     /**
      * @return bool
      */
-    final public function isData (): bool {
+    final public function isData(): bool {
         return $this->opCode < self::OP_CLOSE;
     }
 
     /**
      * @return bool
      */
-    final public function isFinal (): bool {
+    final public function isFinal(): bool {
         return $this->final;
     }
 
     /**
      * @return bool
      */
-    final public function isPing (): bool {
+    final public function isPing(): bool {
         return $this->opCode === self::OP_PING;
     }
 
     /**
      * @return bool
      */
-    final public function isPong (): bool {
+    final public function isPong(): bool {
         return $this->opCode === self::OP_PONG;
     }
 
     /**
      * @return bool
      */
-    final public function isText (): bool {
+    final public function isText(): bool {
         return $this->opCode === self::OP_TEXT;
     }
 

src/StreamClient.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
      * @see https://php.net/socket_create_pair
      *
      * @param array $extra Variadic constructor arguments.
-     * @return static[] Two instances at indices `0` and `1`.
+     * @return StreamClient[] Two instances at indices `0` and `1`.
      * @throws SocketError
      */
     public static function newUnixPair (...$extra) {

Spacing +6 added lines, -6 removed lines

@@ -12,7 +12,7 @@ 
      *
      * @return int
      */
-    final public static function getType (): int {
+    final public static function getType(): int {
         return SOCK_STREAM;
     }
 
@@ -25,7 +25,7 @@ 
      * @return static[] Two instances at indices `0` and `1`.
      * @throws SocketError
      */
-    public static function newUnixPair (...$extra) {
+    public static function newUnixPair(...$extra) {
         if (!@socket_create_pair(AF_UNIX, SOCK_STREAM, 0, $fd)) {
             throw new SocketError; // reliable errno
         }
@@ -44,7 +44,7 @@ 
      * @return string
      * @throws SocketError The partially read data is attached.
      */
-    public function read (int $length): string {
+    public function read(int $length): string {
         try {
             $data = '';
             do {
@@ -71,11 +71,11 @@ 
      * @return string
      * @throws SocketError
      */
-    public function recv (int $maxLength, int $flags = 0): string {
+    public function recv(int $maxLength, int $flags = 0): string {
         if (false === @socket_recv($this->resource, $data, $maxLength, $flags)) {
             throw new SocketError($this->resource, SOCKET_EINVAL);
         }
-        return (string)$data; // cast needed, will be null if 0 bytes are read.
+        return (string) $data; // cast needed, will be null if 0 bytes are read.
     }
 
     /**
@@ -85,7 +85,7 @@ 
      * @return string
      * @throws SocketError
      */
-    public function recvAll (int $flags = 0): string {
+    public function recvAll(int $flags = 0): string {
         $flags = ($flags & ~MSG_WAITALL) | MSG_DONTWAIT;
         $length = $this->getOption(SO_RCVBUF);
         try {

Braces +2 added lines, -4 removed lines

@@ -52,8 +52,7 @@ 
                 $length -= $chunkSize = strlen($chunk);
             } while ($chunkSize and $length);
             return $data;
-        }
-        catch (SocketError $e) {
+        } catch (SocketError $e) {
             $e->setExtra($data);
             throw $e;
         }
@@ -90,8 +89,7 @@ 
         $length = $this->getOption(SO_RCVBUF);
         try {
             return $this->recv($length, $flags);
-        }
-        catch (SocketError $e) {
+        } catch (SocketError $e) {
             if ($e->getCode() === SOCKET_EAGAIN) { // would block
                 return '';
             }

src/WebSocket/FrameHandler.php

Spacing +26 added lines, -26 removed lines

@@ -46,21 +46,21 @@ 
      */
     protected $maxLength = 10 * 1024 * 1024;
 
-    public function __construct (WebSocketClient $client) {
+    public function __construct(WebSocketClient $client) {
         $this->client = $client;
     }
 
     /**
      * @return int
      */
-    public function getFragmentSize (): int {
+    public function getFragmentSize(): int {
         return $this->fragmentSize;
     }
 
     /**
      * @return int
      */
-    public function getMaxLength (): int {
+    public function getMaxLength(): int {
         return $this->maxLength;
     }
 
@@ -72,7 +72,7 @@ 
      * @param Frame $binary
      * @throws WebSocketError
      */
-    protected function onBinary (Frame $binary): void {
+    protected function onBinary(Frame $binary): void {
         $this->buffer .= $binary->getPayload();
         if ($binary->isFinal()) {
             $message = $this->buffer;
@@ -92,7 +92,7 @@ 
      *
      * @param Frame $close
      */
-    protected function onClose (Frame $close): void {
+    protected function onClose(Frame $close): void {
         $this->client->close($close->getCloseCode());
     }
 
@@ -102,7 +102,7 @@ 
      * @param Frame $frame
      * @throws WebSocketError
      */
-    protected function onContinue (Frame $frame): void {
+    protected function onContinue(Frame $frame): void {
         try {
             switch ($this->continue) {
                 case Frame::OP_TEXT:
@@ -135,7 +135,7 @@ 
      *
      * @param Frame $control
      */
-    protected function onControl (Frame $control): void {
+    protected function onControl(Frame $control): void {
         if ($control->isClose()) {
             $this->onClose($control);
         }
@@ -152,7 +152,7 @@ 
      *
      * @param Frame $data
      */
-    protected function onData (Frame $data): void {
+    protected function onData(Frame $data): void {
         $this->onData_SetContinue($data);
         if ($data->isText()) {
             $this->onText($data);
@@ -162,7 +162,7 @@ 
         }
     }
 
-    protected function onData_SetContinue (Frame $data): void {
+    protected function onData_SetContinue(Frame $data): void {
         if ($this->continue) {
             $existing = Frame::NAMES[$this->continue];
             throw new WebSocketError(
@@ -183,7 +183,7 @@ 
      *
      * @param Frame $frame
      */
-    public function onFrame (Frame $frame): void {
+    public function onFrame(Frame $frame): void {
         $this->onFrame_CheckRsv($frame);
         $this->onFrame_CheckLength($frame);
         if ($frame->isControl()) {
@@ -200,7 +200,7 @@ 
     /**
      * @param Frame $frame
      */
-    protected function onFrame_CheckLength (Frame $frame): void {
+    protected function onFrame_CheckLength(Frame $frame): void {
         if ($frame->isData()) {
             $length = strlen($this->buffer);
             if ($length + $frame->getLength() > $this->maxLength) {
@@ -219,7 +219,7 @@ 
      * @param Frame $frame
      * @throws WebSocketError
      */
-    protected function onFrame_CheckRsv (Frame $frame): void {
+    protected function onFrame_CheckRsv(Frame $frame): void {
         if ($badRsv = $frame->getRsv() & ~$this->client->getHandshake()->getRsv()) {
             $badRsv = str_pad(base_convert($badRsv >> 4, 10, 2), 3, '0', STR_PAD_LEFT);
             throw new WebSocketError(Frame::CLOSE_PROTOCOL_ERROR, "Received unknown RSV bits: 0b{$badRsv}");
@@ -233,7 +233,7 @@ 
      *
      * @param Frame $ping
      */
-    protected function onPing (Frame $ping): void {
+    protected function onPing(Frame $ping): void {
         $this->writePong($ping->getPayload());
     }
 
@@ -244,7 +244,7 @@ 
      *
      * @param Frame $pong
      */
-    protected function onPong (Frame $pong): void {
+    protected function onPong(Frame $pong): void {
         // stub
     }
 
@@ -256,7 +256,7 @@ 
      * @param Frame $text
      * @throws WebSocketError
      */
-    protected function onText (Frame $text): void {
+    protected function onText(Frame $text): void {
         $this->buffer .= $text->getPayload();
         if ($text->isFinal()) {
             $message = $this->buffer;
@@ -269,7 +269,7 @@ 
      * @param int $bytes
      * @return $this
      */
-    public function setFragmentSize (int $bytes) {
+    public function setFragmentSize(int $bytes) {
         $this->fragmentSize = $bytes;
         return $this;
     }
@@ -278,7 +278,7 @@ 
      * @param int $bytes
      * @return $this
      */
-    public function setMaxLength (int $bytes) {
+    public function setMaxLength(int $bytes) {
         $this->maxLength = $bytes;
         return $this;
     }
@@ -289,7 +289,7 @@ 
      * @param int $opCode
      * @param string $payload
      */
-    public function write (int $opCode, string $payload): void {
+    public function write(int $opCode, string $payload): void {
         $offset = 0;
         $total = strlen($payload);
         do {
@@ -305,7 +305,7 @@ 
     /**
      * @param string $payload
      */
-    public function writeBinary (string $payload): void {
+    public function writeBinary(string $payload): void {
         $this->write(Frame::OP_BINARY, $payload);
     }
 
@@ -313,8 +313,8 @@ 
      * @param int $code
      * @param string $reason
      */
-    public function writeClose (int $code = Frame::CLOSE_NORMAL, string $reason = ''): void {
-        $this->writeFrame(true, Frame::OP_CLOSE, pack('n', $code) . $reason);
+    public function writeClose(int $code = Frame::CLOSE_NORMAL, string $reason = ''): void {
+        $this->writeFrame(true, Frame::OP_CLOSE, pack('n', $code).$reason);
     }
 
     /**
@@ -324,7 +324,7 @@ 
      * @param int $opCode
      * @param string $payload
      */
-    protected function writeFrame (bool $final, int $opCode, string $payload): void {
+    protected function writeFrame(bool $final, int $opCode, string $payload): void {
         if ($opCode & 0x08 and !$final) {
             throw new WebSocketError(
                 Frame::CLOSE_INTERNAL_ERROR,
@@ -344,27 +344,27 @@ 
         else {
             $head .= chr($length);
         }
-        $this->client->write($head . $payload);
+        $this->client->write($head.$payload);
     }
 
     /**
      * @param string $payload
      */
-    public function writePing (string $payload = ''): void {
+    public function writePing(string $payload = ''): void {
         $this->writeFrame(true, Frame::OP_PING, $payload);
     }
 
     /**
      * @param string $payload
      */
-    public function writePong (string $payload = ''): void {
+    public function writePong(string $payload = ''): void {
         $this->writeFrame(true, Frame::OP_PONG, $payload);
     }
 
     /**
      * @param string $payload
      */
-    public function writeText (string $payload): void {
+    public function writeText(string $payload): void {
         $this->write(Frame::OP_TEXT, $payload);
     }
 }
\ No newline at end of file

Braces +8 added lines, -16 removed lines

@@ -118,8 +118,7 @@ 
                         $frame
                     );
             }
-        }
-        finally {
+        } finally {
             if ($frame->isFinal()) {
                 $this->continue = null;
             }
@@ -138,11 +137,9 @@ 
     protected function onControl (Frame $control): void {
         if ($control->isClose()) {
             $this->onClose($control);
-        }
-        elseif ($control->isPing()) {
+        } elseif ($control->isPing()) {
             $this->onPing($control);
-        }
-        elseif ($control->isPong()) {
+        } elseif ($control->isPong()) {
             $this->onPong($control);
         }
     }
@@ -156,8 +153,7 @@ 
         $this->onData_SetContinue($data);
         if ($data->isText()) {
             $this->onText($data);
-        }
-        elseif ($data->isBinary()) {
+        } elseif ($data->isBinary()) {
             $this->onBinary($data);
         }
     }
@@ -188,11 +184,9 @@ 
         $this->onFrame_CheckLength($frame);
         if ($frame->isControl()) {
             $this->onControl($frame);
-        }
-        elseif ($frame->isContinue()) {
+        } elseif ($frame->isContinue()) {
             $this->onContinue($frame);
-        }
-        else {
+        } else {
             $this->onData($frame);
         }
     }
@@ -336,12 +330,10 @@ 
         if ($length > 65535) {
             $head .= chr(127);
             $head .= pack('J', $length);
-        }
-        elseif ($length >= 126) {
+        } elseif ($length >= 126) {
             $head .= chr(126);
             $head .= pack('n', $length);
-        }
-        else {
+        } else {
             $head .= chr($length);
         }
         $this->client->write($head . $payload);