fix: mark $previous parameter as explicitly nullable for PHP 8.4

[xelan]

Implicitly nullable parameters missing the nullable modifier (e.g. function($param = null) instead of function(?$param = null)) are deprecated in PHP 8.4.

This PR fixes the deprecation, only TcpSocketException::__construct() is currently affected by this issue.

[qodo-slickdeals]

PR Summary by Qodo

Declare TcpSocketException previous cause nullable for PHP 8.4

🐞 Bug fix 🕐 Less than 5 minutes

AI Description

• Explicitly marks the optional previous exception as nullable.
• Prevents implicit-nullability deprecation warnings under PHP 8.4.

High-Level Assessment

The explicit nullable type is the optimal approach because it preserves the constructor API and existing null default while satisfying PHP 8.4. Removing the default or accepting only Exception would introduce an unnecessary compatibility break.

Files changed (1) +1 / -1

Bug fix (1) +1 / -1

TcpSocketException.phpExplicitly allow a nullable previous exception +1/-1

Explicitly allow a nullable previous exception

• Changes the '$previous' constructor parameter from '\Exception' to '?\Exception'. This preserves existing behavior while avoiding PHP 8.4's implicit-nullability deprecation.

src/Connection/TcpSocketException.php

[qodo-slickdeals]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Tip of the day

💡 Did you know, you can show, collapse, or hide each part of a finding: code, evidence, and all

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

[xelan]

I deliberately kept this PR as small as possible for PHP 8.4 support without deprecations, but keeping BC with the currently supported PHP versions.