hfw / socket

src/Reactor.php

Spacing +5 added lines, -5 removed lines

@@ -22,7 +22,7 @@ 
      * @param ReactiveInterface $socket
      * @return $this
      */
-    public function add (ReactiveInterface $socket) {
+    public function add(ReactiveInterface $socket) {
         $this->sockets[$socket->getId()] = $socket;
         return $this;
     }
@@ -32,14 +32,14 @@ 
      *
      * @return int
      */
-    public function count () {
+    public function count() {
         return count($this->sockets);
     }
 
     /**
      * @return SocketInterface[]
      */
-    public function getSockets () {
+    public function getSockets() {
         return $this->sockets;
     }
 
@@ -50,7 +50,7 @@ 
      * @throws Error
      * @return int Number of sockets that reacted.
      */
-    public function react ($timeout = null) {
+    public function react($timeout = null) {
         /** @var ReactiveInterface[][] $rwe */
         $rwe = [$this->sockets, [], $this->sockets];
         $count = AbstractSocket::select($rwe[0], $rwe[1], $rwe[2], $timeout);
@@ -69,7 +69,7 @@ 
      * @param int $id
      * @return $this
      */
-    public function remove ($id) {
+    public function remove($id) {
         unset($this->sockets[$id]);
         return $this;
     }

src/StreamClient.php

Braces +1 added lines, -2 removed lines

@@ -52,8 +52,7 @@
                 if ($chunk === '') {
                     return $data;
                 }
-            }
-            catch (Error $error) {
+            } catch (Error $error) {
                 $error->setExtra($data);
                 throw $error;
             }

Spacing +4 added lines, -4 removed lines

@@ -17,7 +17,7 @@ 
      * @return static[] Two instances at indices `0` and `1`.
      * @throws Error
      */
-    public static function newUnixPair (...$extra) {
+    public static function newUnixPair(...$extra) {
         if (!@socket_create_pair(AF_UNIX, SOCK_STREAM, 0, $fd)) {
             throw new Error; // reliable errno
         }
@@ -32,7 +32,7 @@ 
      *
      * @return int
      */
-    final public static function getType () {
+    final public static function getType() {
         return SOCK_STREAM;
     }
 
@@ -46,7 +46,7 @@ 
      * @return string
      * @throws Error The error's extra data is set to what was partially read.
      */
-    public function read ($length) {
+    public function read($length) {
         $data = '';
         while ($length > 0) {
             try {
@@ -76,7 +76,7 @@ 
      * @throws Error
      * @return string
      */
-    public function recv ($length, $flags = 0) {
+    public function recv($length, $flags = 0) {
         $count = @socket_recv($this->resource, $data, $length, $flags);
         if ($count === false) {
             $error = new Error($this->resource, SOCKET_EINVAL);

src/AbstractSocket.php

Braces +3 added lines, -6 removed lines

@@ -96,11 +96,9 @@ 
     public function __construct ($resource) {
         if (!is_resource($resource) or get_resource_type($resource) !== 'Socket') {
             throw new InvalidArgumentException('Expected an open socket resource.', SOCKET_EBADF);
-        }
-        elseif (socket_get_option($resource, SOL_SOCKET, SO_TYPE) !== static::getType()) {
+        } elseif (socket_get_option($resource, SOL_SOCKET, SO_TYPE) !== static::getType()) {
             throw new InvalidArgumentException('The given socket resource is of the wrong type for ' . get_class(), SOCKET_ESOCKTNOSUPPORT);
-        }
-        elseif ($errno = Error::getLast($resource)) {
+        } elseif ($errno = Error::getLast($resource)) {
             $error = new Error(SOCKET_EBADFD);
             $error->setExtra($errno);
             throw $error;
@@ -331,8 +329,7 @@ 
     public function setBlocking ($blocking) {
         if ($blocking) {
             $success = @socket_set_block($this->resource);
-        }
-        else {
+        } else {
             $success = @socket_set_nonblock($this->resource);
         }
         if (!$success) {

Spacing +25 added lines, -25 removed lines

@@ -41,7 +41,7 @@ 
      * @return static
      * @throws Error
      */
-    public static function create ($domain = AF_INET, ...$extra) {
+    public static function create($domain = AF_INET, ...$extra) {
         if (!$resource = @socket_create($domain, static::getType(), 0)) { // auto-protocol
             throw new Error; // reliable errno
         }
@@ -60,7 +60,7 @@ 
      * @return int
      * @throws Error
      */
-    public static function select (array &$read, array &$write, array &$except, $timeout = null) {
+    public static function select(array &$read, array &$write, array &$except, $timeout = null) {
         $getResource = function(SocketInterface $socket) {
             return $socket->getResource();
         };
@@ -90,12 +90,12 @@ 
      * @throws Error (`SOCKET_EBADFD`) There was a pending error on the socket.
      * The preexisting error code is set as the exception's extra data.
      */
-    public function __construct ($resource) {
+    public function __construct($resource) {
         if (!is_resource($resource) or get_resource_type($resource) !== 'Socket') {
             throw new InvalidArgumentException('Expected an open socket resource.', SOCKET_EBADF);
         }
         elseif (socket_get_option($resource, SOL_SOCKET, SO_TYPE) !== static::getType()) {
-            throw new InvalidArgumentException('The given socket resource is of the wrong type for ' . get_class(), SOCKET_ESOCKTNOSUPPORT);
+            throw new InvalidArgumentException('The given socket resource is of the wrong type for '.get_class(), SOCKET_ESOCKTNOSUPPORT);
         }
         elseif ($errno = Error::getLast($resource)) {
             $error = new Error(SOCKET_EBADFD);
@@ -110,7 +110,7 @@ 
     /**
      * Closes the socket if it's open.
      */
-    public function __destruct () {
+    public function __destruct() {
         $this->close();
     }
 
@@ -125,7 +125,7 @@ 
      * @return $this
      * @throws Error
      */
-    public function await ($channel) {
+    public function await($channel) {
         $select = [$channel => [$this->resource]];
         $count = @socket_select($select[self::CH_READ], $select[self::CH_WRITE], $select[self::CH_EXCEPT], null);
         if (!$count) {
@@ -144,7 +144,7 @@ 
      *
      * @return $this
      */
-    public function close () {
+    public function close() {
         // socket_close() never errors, but we guard anyway to prevent
         // a redundant call to onClose(), and an incorrect call to it if
         // the constructor failed and the destructor is calling this.
@@ -158,14 +158,14 @@ 
     /**
      * @return int
      */
-    final public function getDomain () {
+    final public function getDomain() {
         return $this->domain;
     }
 
     /**
      * @return int
      */
-    final public function getId () {
+    final public function getId() {
         return (int)$this->resource;
     }
 
@@ -178,7 +178,7 @@ 
      * @throws Error
      * @return mixed The option's value. This is never `false`.
      */
-    public function getOption ($option) {
+    public function getOption($option) {
         $value = @socket_get_option($this->resource, SOL_SOCKET, $option);
         if ($value === false) {
             throw new Error($this->resource, SOCKET_EINVAL);
@@ -189,14 +189,14 @@ 
     /**
      * @return int
      */
-    final public function getProtocol () {
+    final public function getProtocol() {
         return $this->protocol;
     }
 
     /**
      * @return resource
      */
-    final public function getResource () {
+    final public function getResource() {
         return $this->resource;
     }
 
@@ -208,7 +208,7 @@ 
      * @throws Error
      * @return array String address/file and integer port at indices `0` and `1`.
      */
-    public function getSockName () {
+    public function getSockName() {
         if (!@socket_getsockname($this->resource, $addr, $port)) {
             throw new Error($this->resource, SOCKET_EOPNOTSUPP);
         }
@@ -220,7 +220,7 @@ 
      *
      * @return bool
      */
-    public function isOpen () {
+    public function isOpen() {
         return is_resource($this->resource);
     }
 
@@ -232,7 +232,7 @@ 
      * @return bool
      * @throws Error
      */
-    public function isOutOfBand () {
+    public function isOutOfBand() {
         return $this->isReady(self::CH_EXCEPT);
     }
 
@@ -244,7 +244,7 @@ 
      * @return bool
      * @throws Error
      */
-    final public function isReadable () {
+    final public function isReadable() {
         return $this->isReady(self::CH_READ);
     }
 
@@ -258,7 +258,7 @@ 
      * @return bool
      * @throws Error
      */
-    public function isReady ($channel, $timeout = 0.0) {
+    public function isReady($channel, $timeout = 0.0) {
         $usec = (int)(fmod($timeout, 1) * 1000000); // ignored if timeout is null
         $select = [$channel => [$this->resource]];
         $count = $count = @socket_select(
@@ -282,7 +282,7 @@ 
      * @return bool
      * @throws Error
      */
-    final public function isWritable () {
+    final public function isWritable() {
         return $this->isReady(self::CH_WRITE);
     }
 
@@ -291,7 +291,7 @@ 
      *
      * @return void
      */
-    protected function onClose () {
+    protected function onClose() {
     }
 
     /**
@@ -300,7 +300,7 @@ 
      * @param int $channel `SocketInterface` channel constant.
      * @return void
      */
-    protected function onShutdown ($channel) {
+    protected function onShutdown($channel) {
     }
 
     /**
@@ -308,7 +308,7 @@ 
      *
      * @return void
      */
-    protected function onTimeout () {
+    protected function onTimeout() {
     }
 
     /**
@@ -329,7 +329,7 @@ 
      * @throws Error
      * @return $this
      */
-    public function setBlocking ($blocking) {
+    public function setBlocking($blocking) {
         if ($blocking) {
             $success = @socket_set_block($this->resource);
         }
@@ -352,7 +352,7 @@ 
      * @throws Error
      * @return $this
      */
-    public function setOption ($option, $value) {
+    public function setOption($option, $value) {
         if (!@socket_set_option($this->resource, SOL_SOCKET, $option, $value)) {
             throw new Error($this->resource, SOCKET_EINVAL);
         }
@@ -366,7 +366,7 @@ 
      * @return $this
      * @throws Error
      */
-    public function setTimeout ($timeout) {
+    public function setTimeout($timeout) {
         $timeval = ['sec' => (int)$timeout, 'usec' => (int)(fmod($timeout, 1) * 1000000)];
         $this->setOption(SO_RCVTIMEO, $timeval);
         $this->setOption(SO_SNDTIMEO, $timeval);
@@ -393,7 +393,7 @@ 
      * @throws Error
      * @return $this
      */
-    public function shutdown ($channel) {
+    public function shutdown($channel) {
         $this->onShutdown($channel);
         if (!@socket_shutdown($this->resource, $channel)) {
             throw new Error($this->resource); // reliable errno

src/Error.php

Braces +2 added lines, -4 removed lines

@@ -57,14 +57,12 @@
     public function __construct ($subject = null, $fallback = 0) {
         if (is_int($subject)) {
             $errno = $subject;
-        }
-        else {
+        } else {
             $errno = static::getLast($subject);
         }
         if ($errno) {
             $message = socket_strerror($errno);
-        }
-        else {
+        } else {
             $errno = $fallback;
             $message = error_get_last()['message'];
         }

Spacing +4 added lines, -4 removed lines

@@ -27,7 +27,7 @@ 
      * @param resource $resource PHP socket resource, or `null` for the global error.
      * @return int If the resource is closed or not a socket, `SOCKET_EBADF` is returned.
      */
-    public static function getLast ($resource = null) {
+    public static function getLast($resource = null) {
         if (isset($resource)) {
             if (@get_resource_type($resource) !== 'Socket') {
                 return SOCKET_EBADF;
@@ -53,7 +53,7 @@ 
      *     PHP's last suppressed warning is used for the message.
      *
      */
-    public function __construct ($subject = null, $fallback = 0) {
+    public function __construct($subject = null, $fallback = 0) {
         if (is_int($subject)) {
             $errno = $subject;
         }
@@ -73,7 +73,7 @@ 
     /**
      * @return mixed
      */
-    public function getExtra () {
+    public function getExtra() {
         return $this->extra;
     }
 
@@ -81,7 +81,7 @@ 
      * @param mixed $extra
      * @return $this
      */
-    public function setExtra ($extra) {
+    public function setExtra($extra) {
         $this->extra = $extra;
         return $this;
     }

src/AbstractServer.php

Braces +1 added lines, -2 removed lines

@@ -17,8 +17,7 @@
     public function __toString () {
         try {
             return implode(':', $this->getSockName());
-        }
-        catch (\Exception $exception) {
+        } catch (\Exception $exception) {
             return ':';
         }
     }

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
      *
      * @return string
      */
-    public function __toString () {
+    public function __toString() {
         try {
             return implode(':', $this->getSockName());
         }
@@ -36,7 +36,7 @@ 
      * @throws Error
      * @return $this
      */
-    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 Error($this->resource, SOCKET_EOPNOTSUPP);
         }

src/AbstractClient.php

Spacing +5 added lines, -5 removed lines

@@ -14,7 +14,7 @@ 
      *
      * @return string
      */
-    public function __toString () {
+    public function __toString() {
         try {
             return implode(':', $this->getPeerName());
         }
@@ -38,7 +38,7 @@ 
      * @throws Error
      * @return $this
      */
-    public function connect ($address, $port = 0) {
+    public function connect($address, $port = 0) {
         if (!@socket_connect($this->resource, $address, $port)) {
             // ignore expected errors for non-blocking connections
             $errno = Error::getLast($this->resource);
@@ -58,7 +58,7 @@ 
      * @throws Error
      * @return array Address and port at indices `0` and `1`.
      */
-    public function getPeerName () {
+    public function getPeerName() {
         if ($this->domain === AF_UNIX) {
             return [$this->getOption(self::SO_PEERCRED), 0];
         }
@@ -80,7 +80,7 @@ 
      * @throws Error
      * @return int Total bytes sent.
      */
-    public function send ($data, $flags = 0) {
+    public function send($data, $flags = 0) {
         $count = @socket_send($this->resource, $data, PHP_INT_MAX, $flags);
         if ($count === false) {
             $error = new Error($this->resource); // reliable errno
@@ -99,7 +99,7 @@ 
      * @throws Error Total bytes sent is set as the error's extra data.
      * @return $this
      */
-    public function write ($data) {
+    public function write($data) {
         $data = (string)$data;
         $length = strlen($data);
         $total = 0;

Braces +2 added lines, -4 removed lines

@@ -19,8 +19,7 @@ 
     public function __toString () {
         try {
             return implode(':', $this->getPeerName());
-        }
-        catch (Exception $exception) {
+        } catch (Exception $exception) {
             return ':';
         }
     }
@@ -109,8 +108,7 @@ 
             try {
                 $total += $count = $this->await(self::CH_WRITE)->send($data);
                 $data = substr($data, $count); // 0 bytes sent = unaltered buffer
-            }
-            catch (Error $error) {
+            } catch (Error $error) {
                 $error->setExtra($total);
                 throw $error;
             }

src/DatagramServer.php

Spacing +2 added lines, -2 removed lines

@@ -12,7 +12,7 @@ 
      *
      * @return int
      */
-    final public static function getType () {
+    final public static function getType() {
         return SOCK_DGRAM;
     }
 
@@ -30,7 +30,7 @@ 
      * @throws Error
      * @return string
      */
-    public function recv ($length, $flags = 0, &$name = null, &$port = 0) {
+    public function recv($length, $flags = 0, &$name = null, &$port = 0) {
         $count = @socket_recvfrom($this->resource, $data, $length, $flags, $name, $port);
         if ($count === false) {
             $error = new Error($this->resource, SOCKET_EOPNOTSUPP);

src/DatagramClient.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      *
      * @return int
      */
-    final public static function getType () {
+    final public static function getType() {
         return SOCK_DGRAM;
     }
 

src/WebSocketFrame.php

Spacing +13 added lines, -13 removed lines

@@ -25,7 +25,7 @@ 
      * @param bool $final
      * @return string
      */
-    public static function pack (string $payload, int $opCode = self::OP_TEXT, bool $final = true): string {
+    public static function pack(string $payload, int $opCode = self::OP_TEXT, bool $final = true): string {
         $frame = chr(0x80 | $opCode);
         $len = strlen($payload);
         if ($len > 65535) {
@@ -39,14 +39,14 @@ 
         else {
             $frame .= chr($len);
         }
-        return $frame . $payload;
+        return $frame.$payload;
     }
 
-    public function __construct (string $data) {
+    public function __construct(string $data) {
         $this->data = $data;
     }
 
-    final public function getLength () {
+    final public function getLength() {
         switch ($char = ord($this->data[1]) & 0x7f) {
             case 127: // big-endian long-long
                 return unpack('J', substr($this->data, 2, 8));
@@ -60,7 +60,7 @@ 
     /**
      * @return int[]
      */
-    final public function getMask () {
+    final public function getMask() {
         if ($this->isMasked()) {
             switch (ord($this->data[1]) & 0x7f) {
                 case 127:
@@ -74,11 +74,11 @@ 
         return [0, 0, 0, 0];
     }
 
-    final public function getOpCode (): int {
+    final public function getOpCode(): int {
         return ord($this->data[0]) & 0x0f;
     }
 
-    final public function getPayload (): string {
+    final public function getPayload(): string {
         $length = $this->getLength();
         if ($length === 0) {
             return '';
@@ -102,27 +102,27 @@ 
         return $decoded;
     }
 
-    final public function isBinary (): bool {
+    final public function isBinary(): bool {
         return $this->getOpCode() === self::OP_BINARY;
     }
 
-    final public function isClose (): bool {
+    final public function isClose(): bool {
         return $this->getOpCode() === self::OP_CLOSE;
     }
 
-    final public function isFinal (): bool {
+    final public function isFinal(): bool {
         return (bool)(ord($this->data[0]) & 0x80);
     }
 
-    final public function isMasked (): bool {
+    final public function isMasked(): bool {
         return (bool)(ord($this->data[1]) & 0x80);
     }
 
-    final public function isPong (): bool {
+    final public function isPong(): bool {
         return $this->getOpCode() === self::OP_PONG;
     }
 
-    final public function isText (): bool {
+    final public function isText(): bool {
         return $this->getOpCode() === self::OP_TEXT;
     }
 

Braces +2 added lines, -4 removed lines

@@ -31,12 +31,10 @@
         if ($len > 65535) {
             $frame .= chr(127);
             $frame .= pack('J', $len);
-        }
-        elseif ($len >= 126) {
+        } elseif ($len >= 126) {
             $frame .= chr(126);
             $frame .= pack('n', $len);
-        }
-        else {
+        } else {
             $frame .= chr($len);
         }
         return $frame . $payload;