Changeset 52717 for trunk/src/wp-includes/media.php

Timestamp:

02/12/2022 01:06:17 PM (3 years ago)

Author:

SergeyBiryukov

Message:

Coding Standards: Remove unnecessary try/catch block in wp_get_webp_info().

This appears to be redundant, as none of the functions used inside the block throw an exception.

Follow-up to [50810].

See #54728.

File:

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

trunk/src/wp-includes/media.php

@@ -5236 +5236 @@
     }
 
-    try {
-        $handle = fopen( $filename, 'rb' );
-        if ( $handle ) {
-            $magic = fread( $handle, 40 );
-            fclose( $handle );
-
-            // Make sure we got enough bytes.
-            if ( strlen( $magic ) < 40 ) {
-                return compact( 'width', 'height', 'type' );
-            }
-
-            // The headers are a little different for each of the three formats.
-            // Header values based on WebP docs, see https://developers.google.com/speed/webp/docs/riff_container.
-            switch ( substr( $magic, 12, 4 ) ) {
-                // Lossy WebP.
-                case 'VP8 ':
-                    $parts  = unpack( 'v2', substr( $magic, 26, 4 ) );
-                    $width  = (int) ( $parts[1] & 0x3FFF );
-                    $height = (int) ( $parts[2] & 0x3FFF );
-                    $type   = 'lossy';
-                    break;
-                // Lossless WebP.
-                case 'VP8L':
-                    $parts  = unpack( 'C4', substr( $magic, 21, 4 ) );
-                    $width  = (int) ( $parts[1] | ( ( $parts[2] & 0x3F ) << 8 ) ) + 1;
-                    $height = (int) ( ( ( $parts[2] & 0xC0 ) >> 6 ) | ( $parts[3] << 2 ) | ( ( $parts[4] & 0x03 ) << 10 ) ) + 1;
-                    $type   = 'lossless';
-                    break;
-                // Animated/alpha WebP.
-                case 'VP8X':
-                    // Pad 24-bit int.
-                    $width = unpack( 'V', substr( $magic, 24, 3 ) . "\x00" );
-                    $width = (int) ( $width[1] & 0xFFFFFF ) + 1;
-                    // Pad 24-bit int.
-                    $height = unpack( 'V', substr( $magic, 27, 3 ) . "\x00" );
-                    $height = (int) ( $height[1] & 0xFFFFFF ) + 1;
-                    $type   = 'animated-alpha';
-                    break;
-            }
-        }
-    } catch ( Exception $e ) {
+    $handle = fopen( $filename, 'rb' );
+
+    if ( false === $handle ) {
+        return compact( 'width', 'height', 'type' );
+    }
+
+    $magic = fread( $handle, 40 );
+    fclose( $handle );
+
+    // Make sure we got enough bytes.
+    if ( strlen( $magic ) < 40 ) {
+        return compact( 'width', 'height', 'type' );
+    }
+
+    // The headers are a little different for each of the three formats.
+    // Header values based on WebP docs, see https://developers.google.com/speed/webp/docs/riff_container.
+    switch ( substr( $magic, 12, 4 ) ) {
+        // Lossy WebP.
+        case 'VP8 ':
+            $parts  = unpack( 'v2', substr( $magic, 26, 4 ) );
+            $width  = (int) ( $parts[1] & 0x3FFF );
+            $height = (int) ( $parts[2] & 0x3FFF );
+            $type   = 'lossy';
+            break;
+        // Lossless WebP.
+        case 'VP8L':
+            $parts  = unpack( 'C4', substr( $magic, 21, 4 ) );
+            $width  = (int) ( $parts[1] | ( ( $parts[2] & 0x3F ) << 8 ) ) + 1;
+            $height = (int) ( ( ( $parts[2] & 0xC0 ) >> 6 ) | ( $parts[3] << 2 ) | ( ( $parts[4] & 0x03 ) << 10 ) ) + 1;
+            $type   = 'lossless';
+            break;
+        // Animated/alpha WebP.
+        case 'VP8X':
+            // Pad 24-bit int.
+            $width = unpack( 'V', substr( $magic, 24, 3 ) . "\x00" );
+            $width = (int) ( $width[1] & 0xFFFFFF ) + 1;
+            // Pad 24-bit int.
+            $height = unpack( 'V', substr( $magic, 27, 3 ) . "\x00" );
+            $height = (int) ( $height[1] & 0xFFFFFF ) + 1;
+            $type   = 'animated-alpha';
+            break;
     }