Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[PhpUnitBridge] Provide debug_backtrace with proper args#28506
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
[PhpUnitBridge] Provide debug_backtrace with proper args#28506
Uh oh!
There was an error while loading.Please reload this page.
Conversation
3ffa6d1 tocfe6ae1Compare| $mode =$getMode(); | ||
| $trace =debug_backtrace(true); | ||
| $trace =debug_backtrace(\defined(DEBUG_BACKTRACE_PROVIDE_OBJECT) ?DEBUG_BACKTRACE_PROVIDE_OBJECT :true);// simplify when dropping php < 5.3.6 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Note thatDEBUG_BACKTRACE_PROVIDE_OBJECT is the default value for this argument.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
so let's remove it entirely instead
nicolas-grekas commentedSep 19, 2018 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
why do we care? we don't use strict_types (and we don't want to.) |
stof commentedSep 19, 2018
this is not a bugfix as there is no bug. This file is not in strict mode, and cannot become in strict mode unless you edit the source code yourselves (in which case,you are introducing a bug) |
This would fail if we were using strict mode with php 7, because true isonly a valid argument for php < 5.3.6.This was changed from PHP_VERSION_ID >= 50400 ?DEBUG_BACKTRACE_IGNORE_ARGS | DEBUG_BACKTRACE_PROVIDE_OBJECT : true insymfony#18272, but I do not understand why it was simplified, nor whyDEBUG_BACKTRACE_IGNORE_ARGS was there at that time.
cfe6ae1 to0d826aeComparenicolas-grekas commentedSep 20, 2018
Thank you@greg0ire. |
… (greg0ire)This PR was merged into the 2.8 branch.Discussion----------[PhpUnitBridge] Provide debug_backtrace with proper argsThis would fail if we were using strict mode with php 7, because true isonly a valid argument for php < 5.3.6.This was changed from PHP_VERSION_ID >= 50400 ?DEBUG_BACKTRACE_IGNORE_ARGS | DEBUG_BACKTRACE_PROVIDE_OBJECT : true in#18272, but I do not understand why it was simlified, nor whyDEBUG_BACKTRACE_IGNORE_ARGS was there at that time.| Q | A| ------------- | ---| Branch? | 2.8| Bug fix? | no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | n/a| License | MIT| Doc PR | n/aNot sure if this qualifies as a bugfix. If not, should I simplify the ternary and target master? My fear is that having the code diverge too much will make it harder to merge subsequent PRs. I know this looks small, but I'm kind of preparing a big PR on the bridge and I'd rather have it smaller and easier to understand by moving everything I can with small patches like this one.Commits-------0d826ae Provide debug_backtrace with proper args
Uh oh!
There was an error while loading.Please reload this page.
This would fail if we were using strict mode with php 7, because true is
only a valid argument for php < 5.3.6.
This was changed from PHP_VERSION_ID >= 50400 ?
DEBUG_BACKTRACE_IGNORE_ARGS | DEBUG_BACKTRACE_PROVIDE_OBJECT : true in
#18272, but I do not understand why it was simlified, nor why
DEBUG_BACKTRACE_IGNORE_ARGS was there at that time.
Not sure if this qualifies as a bugfix. If not, should I simplify the ternary and target master? My fear is that having the code diverge too much will make it harder to merge subsequent PRs. I know this looks small, but I'm kind of preparing a big PR on the bridge and I'd rather have it smaller and easier to understand by moving everything I can with small patches like this one.