Skip to content

Commit

Permalink
Replace calls to array_merge with array_values
Browse files Browse the repository at this point in the history
  • Loading branch information
igorw committed Sep 24, 2012
1 parent 6073011 commit b3f8578
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions Protocol/Parser.php
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ public function parseHeader(Message $message)
$header = substr($message->data, 0, 12);
$message->consumed += 12;

list($id, $fields, $qdCount, $anCount, $nsCount, $arCount) = array_merge(unpack('n*', $header));
list($id, $fields, $qdCount, $anCount, $nsCount, $arCount) = array_values(unpack('n*', $header));

$rcode = $fields & bindec('1111');
$z = ($fields >> 4) & bindec('111');
Expand Down Expand Up @@ -86,7 +86,7 @@ public function parseQuestion(Message $message)
return;
}

list($type, $class) = array_merge(unpack('n*', substr($message->data, $consumed, 4)));
list($type, $class) = array_values(unpack('n*', substr($message->data, $consumed, 4)));
$consumed += 4;

$message->consumed = $consumed;
Expand Down Expand Up @@ -122,13 +122,13 @@ public function parseAnswer(Message $message)
return;
}

list($type, $class) = array_merge(unpack('n*', substr($message->data, $consumed, 4)));
list($type, $class) = array_values(unpack('n*', substr($message->data, $consumed, 4)));
$consumed += 4;

list($ttl) = array_merge(unpack('N', substr($message->data, $consumed, 4)));
list($ttl) = array_values(unpack('N', substr($message->data, $consumed, 4)));
$consumed += 4;

list($rdLength) = array_merge(unpack('n', substr($message->data, $consumed, 2)));
list($rdLength) = array_values(unpack('n', substr($message->data, $consumed, 2)));
$consumed += 2;

$rdata = null;
Expand Down Expand Up @@ -208,15 +208,15 @@ public function getCompressedLabel($data, $consumed)
public function isCompressedLabel($data, $consumed)
{
$mask = 0xc000; // 1100000000000000
list($peek) = array_merge(unpack('n', substr($data, $consumed, 2)));
list($peek) = array_values(unpack('n', substr($data, $consumed, 2)));

return (bool) ($peek & $mask);
}

public function getCompressedLabelOffset($data, $consumed)
{
$mask = 0x3fff; // 0011111111111111
list($peek) = array_merge(unpack('n', substr($data, $consumed, 2)));
list($peek) = array_values(unpack('n', substr($data, $consumed, 2)));

return array($peek & $mask, $consumed + 2);
}
Expand Down

0 comments on commit b3f8578

Please sign in to comment.