diff --git a/CHANGELOG.md b/CHANGELOG.md index 81d970bf..e4dd20e3 100644 --- a/CHANGELOG.md +++ b/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 diff --git a/src/Config.php b/src/Config.php index 48b58c0b..f24797e6 100644 --- a/src/Config.php +++ b/src/Config.php @@ -52,7 +52,6 @@ */ class Config { - /** * The logger instance. */ diff --git a/src/FFI.php b/src/FFI.php index 926f24c0..06c0bae3 100644 --- a/src/FFI.php +++ b/src/FFI.php @@ -50,7 +50,6 @@ */ class FFI { - /** * The FFI handle we use for the glib binary. * diff --git a/src/Introspect.php b/src/Introspect.php index 489364d7..c3871c60 100644 --- a/src/Introspect.php +++ b/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; diff --git a/src/VipsObject.php b/src/VipsObject.php index e0b18a59..3189f270 100644 --- a/src/VipsObject.php +++ b/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)); diff --git a/src/VipsOperation.php b/src/VipsOperation.php index cf212afa..4e4e6582 100644 --- a/src/VipsOperation.php +++ b/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; }