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

util: improve util inspect performance #22503

Closed
wants to merge 3 commits into from
Closed

util: improve util inspect performance #22503

wants to merge 3 commits into from
+242 −273

Commits on Sep 6, 2018

  1. util: remove TODO 

    I do not plan on doing this anymore.
    @BridgeAR
    BridgeAR committed Sep 6, 2018
    Configuration menu
    Copy the full SHA
    c6d9a4d View commit details
    Browse the repository at this point in the history

  2. benchmark: refactor util benchmarks 

    This significantly reduces the benchmark runtime. It removes to many
variations that do not provide any benefit and reduces the iterations.
    @BridgeAR
    BridgeAR committed Sep 6, 2018
    Configuration menu
    Copy the full SHA
    5f7d097 View commit details
    Browse the repository at this point in the history

  3. util: improve inspect performance 

    This significantly improves the inspection performance for all array
types. From now on only the visible elements cause work instead of
having to process all array keys no matter how many entries are
visible.

This also moves some code out of the main function to reduce the
overall function complexity.
    @BridgeAR
    BridgeAR committed Sep 6, 2018
    Configuration menu
    Copy the full SHA
    2b1c2a4 View commit details
    Browse the repository at this point in the history