Skip to content
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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update FirePHP.class.php #71

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update FirePHP.class.php #71

wants to merge 1 commit into from

Commits on Oct 15, 2021

  1. Update FirePHP.class.php 

    adding error_reporting() as 2nd parameter (error_levels), when registering the errorhandler, results in not having to have a call to errorHandler() whenever PHP throwing an error, but only for those, that are activated.

E_WARNINGs and E_NOTICEs can sum up very fast in hundreds to thousands of unnecessary calls to errorHandler(), just to have errorHandler() to decide not to throw an exception.

Drawback of this version: if you change error_reporting after registerErrorHandler(), you have to call registerErrorHandler again, in order to get the new error_reporting into count.
For backward compatibility, a new parameter ($use_error_reporting) is introduced, defaulting to false, providing the functions old behaviour
    @stSaHiB
    stSaHiB authored Oct 15, 2021
    Configuration menu
    Copy the full SHA
    f4333d0 View commit details
    Browse the repository at this point in the history