Remove debug logging from hot paths

CHANGELOG.md

@@ -6,6 +6,7 @@ All notable changes to `php-vips` will be documented in this file.
 
 - better ffi startup diagnostics [ping-localhost]
 - add setBlock() and setBlockUntrusted() to control operation blocking [jcupitt]
+- remove debug logging from hot paths [guillaume-sainthillier]
 
 ## 2.6.1 - 2025-12-10
 

src/Config.php

@@ -52,7 +52,6 @@
  */
 class Config
 {
-
     /**
      * The logger instance.
      */

src/FFI.php

@@ -50,7 +50,6 @@
  */
 class FFI
 {
-
     /**
      * The FFI handle we use for the glib binary.
      *

src/Introspect.php

@@ -165,14 +165,14 @@ public function __construct($operation_name)
                 !($flags & ArgumentFlags::DEPRECATED)) {
                 $this->required_output[] = $name;
             }
- 
+
             # we let deprecated optional args through, but warn about them
             # if they get used, see below
             if (($flags & ArgumentFlags::INPUT) &&
                 !($flags & ArgumentFlags::REQUIRED)) {
                 $this->optional_input[] = $name;
             }
- 
+
             if (($flags & ArgumentFlags::OUTPUT) &&
                 !($flags & ArgumentFlags::REQUIRED)) {
                 $this->optional_output[] = $name;

src/VipsObject.php

@@ -162,8 +162,6 @@ public function get(string $name)
             g_object_get_property($this->gObject, $name, $gvalue->pointer);
         $value = $gvalue->get();
 
-        Utils::debugLog("get", [$name => var_export($value, true)]);
-
         return $value;
     }
 
@@ -172,8 +170,6 @@ public function get(string $name)
      */
     public function set(string $name, $value): void
     {
-        Utils::debugLog("set", [$name => $value]);
-
         $name = str_replace("-", "_", $name);
         $gvalue = new GValue();
         $gvalue->setType($this->getType($name));

src/VipsOperation.php

@@ -217,11 +217,6 @@ public static function callBase(
         ?Image $instance,
         array $arguments
     ) {
-        Utils::debugLog($operation_name, [
-            'instance' => $instance,
-            'arguments' => $arguments
-        ]);
-
         $operation = self::newFromName($operation_name);
         $operation->introspect = self::introspect($operation_name);
         $introspect = $operation->introspect;
@@ -339,10 +334,6 @@ public static function callBase(
 
         $result = self::wrapResult($result);
 
-        Utils::debugLog($operation_name, [
-            'result' => var_export($result, true)
-        ]);
-
         return $result;
     }