8000 minor #38033 Remove unnecessary silence operator (alexpott) · symfony/symfony@dad4e95 · GitHub
[go: up one dir, main page]
Skip to content
< 8000 script crossorigin="anonymous" defer="defer" type="application/javascript" src="https://github.githubassets.com/assets/vendors-node_modules_github_remote-form_dist_index_js-node_modules_delegated-events_dist_inde-94fd67-e789af5a4655.js">

Commit dad4e95

Browse files
fabpotfabpot
committed
minor #38033 Remove unnecessary silence operator (alexpott)
This PR was merged into the 5.2-dev branch. Discussion ---------- Remove unnecessary silence operator | Q | A | ------------- | --- | Branch? | master | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tickets | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead --> | License | MIT | Doc PR | symfony/symfony-docs#... <!-- required for new features --> A gobble-all-errors handler was added around the unserialize() call making the @ operator unnecessary. Commits ------- 03201f0 No longer need to silence errors as we're catching them all
2 parents a64b944 + 03201f0 commit dad4e95

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

‎src/Symfony/Bridge/PhpUnit/DeprecationErrorHandler/Deprecation.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ public function __construct($message, array $trace, $file)
6161
$this->triggeringFile = $file;
6262
if (isset($line['object']) || isset($line['class'])) {
6363
set_error_handler(function () {});
64-
$parsedMsg = @unserialize($this->message);
64+
$parsedMsg = unserialize($this->message);
6565
restore_error_handler();
6666
if ($parsedMsg && isset($parsedMsg['deprecation'])) {
6767
$this->message = $parsedMsg['deprecation'];

0 commit comments

Comments
 (0)
0