Some more formatting fixes for media classes
authorSiebrand Mazeland <s.mazeland@xs4all.nl>
Thu, 5 Dec 2013 20:55:30 +0000 (21:55 +0100)
committerSiebrand Mazeland <s.mazeland@xs4all.nl>
Sun, 8 Dec 2013 09:45:35 +0000 (10:45 +0100)
Based on Brian's review of https://gerrit.wikimedia.org/r/#/c/99344/

Change-Id: I35aea90da837d17bf8be5c29d22f5f0a3bcef145

includes/media/Bitmap.php
includes/media/XMPInfo.php

index d93168f..eeb8078 100644 (file)
@@ -67,7 +67,7 @@ class BitmapHandler extends ImageHandler {
 
                        if ( $srcWidth * $srcHeight > $wgMaxImageArea
                                && !( $image->getMimeType() == 'image/jpeg'
-                               && self::getScalerType( false, false ) == 'im' )
+                                       && self::getScalerType( false, false ) == 'im' )
                        ) {
                                # Only ImageMagick can efficiently downsize jpg images without loading
                                # the entire file in memory
index 3272fef..3700556 100644 (file)
@@ -53,20 +53,21 @@ class XMPInfo {
         * each containing an array of tags
         * each tag is an array of information about the
         * tag, including:
-        *   * map_group - what group (used for precedence during conflicts)
-        *   * mode - What type of item (self::MODE_SIMPLE usually, see above
-        *     for all values)
-        *   * validate - method to validate input. Could also post-process the
+        *   * map_group - What group (used for precedence during conflicts).
+        *   * mode - What type of item (self::MODE_SIMPLE usually, see above for
+        *     all values).
+        *   * validate - Method to validate input. Could also post-process the
         *     input. A string value is assumed to be a static method of
         *     XMPValidate. Can also take a array( 'className', 'methodName' ).
-        *   * choices  - array of potential values (format of 'value' => true ).
-        *     Only used with validateClosed
-        *   * rangeLow and rangeHigh - alternative to choices for numeric ranges.
+        *   * choices - Array of potential values (format of 'value' => true ).
+        *     Only used with validateClosed.
+        *   * rangeLow and rangeHigh - Alternative to choices for numeric ranges.
         *     Again for validateClosed only.
-        *     * children - for MODE_STRUCT items, allowed children.
-        *   * structPart - Indicates that this element can only appear as a member of a structure.
+        *   * children - For MODE_STRUCT items, allowed children.
+        *   * structPart - Indicates that this element can only appear as a member
+        *     of a structure.
         *
-        * Currently this just has a bunch of exif values as this class is only half-done
+        * Currently this just has a bunch of EXIF values as this class is only half-done.
         */
        static private $items = array(
                'http://ns.adobe.com/exif/1.0/' => array(