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

More code improvements based on static analysis #4258

Merged
merged 11 commits into from
Nov 24, 2023
Merged

More code improvements based on static analysis #4258

merged 11 commits into from
Nov 24, 2023

Commits on Nov 23, 2023

  1. Add operator= where copy ctor is present, drop implemetation of ctors… 

    … which can be defaulted
    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    69938dd View commit details
    Browse the repository at this point in the history

  2. Avoid possible NULL derefrence

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    62cc2e5 View commit details
    Browse the repository at this point in the history

  3. Avoid returning garbage for negatively sized stack declarations.

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    ed35736 View commit details
    Browse the repository at this point in the history

  4. Add missing va_end

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    9a7881c View commit details
    Browse the repository at this point in the history

  5. Avoid unnecessary copy, modernize

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    5045649 View commit details
    Browse the repository at this point in the history

  6. Avoid null dereference after error

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    55951c6 View commit details
    Browse the repository at this point in the history

  7. Provide assignment operator to match the copy constructor

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    1e254d2 View commit details
    Browse the repository at this point in the history

  8. Avoid doble-copying in frontend constructor/addDebugHook

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    812a5e5 View commit details
    Browse the repository at this point in the history

  9. Avoid extra copies

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    efbcd43 View commit details
    Browse the repository at this point in the history

  10. Fix formatting

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    252b157 View commit details
    Browse the repository at this point in the history

  11. Avoid sending null Node to ::error

    @vlstill
    vlstill committed Nov 23, 2023
    Configuration menu
    Copy the full SHA
    9d52e5c View commit details
    Browse the repository at this point in the history