cvs: pear /Image_Transform/Driver Cairowrapper.php

[email protected] ("Christian Weiske")
Newsgroups php.pear.cvs
Message-ID <cvscweiske1209596460@cvsserver>
cweiske		Wed Apr 30 23:01:00 2008 UTC

  Modified files:              
    /pear/Image_Transform/Driver	Cairowrapper.php 
  Log:
  Return true on resize method and adjust docblocks
  
  
http://cvs.php.net/viewvc.cgi/pear/Image_Transform/Driver/Cairowrapper.php?r1=1.4&r2=1.5&diff_format=u
Index: pear/Image_Transform/Driver/Cairowrapper.php
diff -u pear/Image_Transform/Driver/Cairowrapper.php:1.4 pear/Image_Transform/Driver/Cairowrapper.php:1.5
--- pear/Image_Transform/Driver/Cairowrapper.php:1.4	Tue Apr 29 09:25:05 2008
+++ pear/Image_Transform/Driver/Cairowrapper.php	Wed Apr 30 23:01:00 2008
@@ -18,7 +18,7 @@
  * @author     Christian Weiske <[email protected]>
  * @copyright  2008 The PHP Group
  * @license    http://www.php.net/license/3_0.txt  PHP License 3.0
- * @version    CVS: $Id: Cairowrapper.php,v 1.4 2008/04/29 09:25:05 cweiske Exp $
+ * @version    CVS: $Id: Cairowrapper.php,v 1.5 2008/04/30 23:01:00 cweiske Exp $
  * @link       http://pear.php.net/package/Image_Transform
  */
 require_once 'Image/Transform.php';
@@ -67,8 +67,12 @@
     function __construct()
     {
         if (!PEAR::loadExtension('cairo_wrapper')) {
-            $this->isError(PEAR::raiseError("cairo_wrapper extension is not available.",
-                IMAGE_TRANSFORM_ERROR_UNSUPPORTED));
+            $this->isError(
+                PEAR::raiseError(
+                    "cairo_wrapper extension is not available.",
+                    IMAGE_TRANSFORM_ERROR_UNSUPPORTED
+                )
+            );
         }
     }
 
@@ -167,7 +171,7 @@
      * @param mixed  $type     ignored
      * @param mixed  $quality  ignored
      *
-     * @return void
+     * @return bool|PEAR_Error TRUE on success or PEAR_Error object on error
      *
      * @access public
      */
@@ -175,6 +179,7 @@
     {
         cairo_surface_write_to_png($this->surface, $filename);
         $this->free();
+        return true;
     }//function save(..)
 
 
@@ -212,7 +217,7 @@
 
 
     /**
-     * Mirrors the image horizontally
+     * Mirrors the image vertically
      * Uses an affine transformation matrix to flip the image.
      *
      * @return void
@@ -239,7 +244,7 @@
 
 
     /**
-     * Mirrors the image vertically.
+     * Mirrors the image horizontally.
      * Uses an affine transformation matrix to mirror the image.
      *
      * 123 -> 321
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.