Remove error suppression in Exif::isRational
authorumherirrender <umherirrender_de.wp@web.de>
Sun, 17 Mar 2013 14:57:31 +0000 (15:57 +0100)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 18 Mar 2013 07:50:39 +0000 (07:50 +0000)
preg_match throws warning, when getting a array as input, but there is a
is_array check, so no need for error suppression

Change-Id: Ie7f180f45c04e8e396186cd6a5a9e40e2c94a80a

includes/media/Exif.php

index 20b84d5..1767180 100644 (file)
@@ -677,7 +677,7 @@ class Exif {
         */
        private function isRational( $in ) {
                $m = array();
-               if ( !is_array( $in ) && @preg_match( '/^(\d+)\/(\d+[1-9]|[1-9]\d*)$/', $in, $m ) ) { # Avoid division by zero
+               if ( !is_array( $in ) && preg_match( '/^(\d+)\/(\d+[1-9]|[1-9]\d*)$/', $in, $m ) ) { # Avoid division by zero
                        return $this->isLong( $m[1] ) && $this->isLong( $m[2] );
                } else {
                        $this->debug( $in, __FUNCTION__, 'fed a non-fraction value' );