Changeset 16300 for trunk/wp-includes/functions.php

Timestamp:

11/11/2010 04:10:16 PM (14 years ago)

Author:

ryan

Message:

Faster is_serialized(). Props duck_, hakre, Denis-de-Bernardy. see #14429

File:

• trunk/wp-includes/functions.php (modified) (1 diff)

trunk/wp-includes/functions.php

@@ -247 +247 @@
 function is_serialized( $data ) {
     // if it isn't a string, it isn't serialized
-    if ( !is_string( $data ) )
+    if ( ! is_string( $data ) )
         return false;
     $data = trim( $data );
     if ( 'N;' == $data )
         return true;
-    if ( function_exists('strpbrk') ) {
-        if ( strlen($data) > 1 && strpbrk($data,'adObis') == $data && $data[1] == ':' ) {
-            $badions = array();
-            $badions[1] = $data[0];
-        } else {
-            return false;
-        }
-    } elseif ( !preg_match( '/^([adObis]):/', $data, $badions ) ) {
+    $length = strlen( $data );
+    if ( $length < 4 )
         return false;
-    }
-    switch ( $badions[1] ) {
+    if ( ':' !== $data[1] )
+        return false;
+    $lastc = $data[$length-1];
+    if ( ';' !== $lastc && '}' !== $lastc )
+        return false;
+    $token = $data[0];
+    switch ( $token ) {
+        case 's' :
+            if ( '"' !== $data[$length-2] )
+                return false;
         case 'a' :
         case 'O' :
-        case 's' :
-            if ( preg_match( "/^{$badions[1]}:[0-9]+:.*[;}]\$/s", $data ) )
-                return true;
-            break;
+            return (bool) preg_match( "/^{$token}:[0-9]+:/s", $data );
         case 'b' :
         case 'i' :
         case 'd' :
-            if ( preg_match( "/^{$badions[1]}:[0-9.E-]+;\$/", $data ) )
-                return true;
-            break;
+            return (bool) preg_match( "/^{$token}:[0-9.E-]+;\$/", $data );
     }
     return false;