Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This should fix #132, though I don't have an Xdebug 2 profile to compare to. Based on the findings in #132 Xdebug 2 reported costs in microseconds, but Xdebug 3 now reports those costs in nanoseconds. This looks at the
events
header, which explicitly declares the time unit used for the costs, if defined, and dynamically adjusts the divisor based on the declared time unit.This seemed like the easiest fix. Since Xdebug 3 outputs much larger numbers due to the change to nanoseconds, the preprocessor is probably at greater risk of integer overflow, but that is a pre-existing problem.