Minor optimisation

Changed from preg_match_all to preg_match as we only need the first result
jamiebicknell · Mar 24, 2015 · ef4f339 · ef4f339
1 parent d6298c0
commit ef4f339
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/face.php b/face.php
@@ -46,8 +46,8 @@ function get_skin($user)
         $result = curl_exec($ch);
         $status = curl_getinfo($ch, CURLINFO_HTTP_CODE);
         if ($status == 301) {
-            preg_match_all('/location:(.*)/i', $result, $matches);
-            curl_setopt($ch, CURLOPT_URL, trim(array_pop($matches[1])));
+            preg_match('/location:(.*)/i', $result, $matches);
+            curl_setopt($ch, CURLOPT_URL, trim($matches[1]));
             curl_setopt($ch, CURLOPT_HEADER, 0);
             curl_setopt($ch, CURLOPT_NOBODY, 0);
             $result = curl_exec($ch);

diff --git a/skin.php b/skin.php
@@ -44,8 +44,8 @@ function get_skin($user)
         $result = curl_exec($ch);
         $status = curl_getinfo($ch, CURLINFO_HTTP_CODE);
         if ($status == 301) {
-            preg_match_all('/location:(.*)/i', $result, $matches);
-            curl_setopt($ch, CURLOPT_URL, trim(array_pop($matches[1])));
+            preg_match('/location:(.*)/i', $result, $matches);
+            curl_setopt($ch, CURLOPT_URL, trim($matches[1]));
             curl_setopt($ch, CURLOPT_HEADER, 0);
             curl_setopt($ch, CURLOPT_NOBODY, 0);
             $result = curl_exec($ch);