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

dnsdist: Fix warnings reported by clang's analyzer and cppcheck #6407

Merged
merged 7 commits into from Mar 28, 2018
Merged

dnsdist: Fix warnings reported by clang's analyzer and cppcheck #6407

merged 7 commits into from Mar 28, 2018

Commits on Mar 27, 2018

  1. LuaWrapper: Don't move function since we still access it later

    @rgacogne
    rgacogne committed Mar 27, 2018
    Copy the full SHA
    34abd71 View commit details
    Browse the repository at this point in the history

  2. Use pre-increment instead of post-increment for non-primitive types

    @rgacogne
    rgacogne committed Mar 27, 2018
    Copy the full SHA
    4b5da56 View commit details
    Browse the repository at this point in the history

  3. Update the lock status in the move constructors

    @rgacogne
    rgacogne committed Mar 27, 2018
    Copy the full SHA
    f10f2f2 View commit details
    Browse the repository at this point in the history

  4. Use an initialization list instead of assigning values in the ctor body

    @rgacogne
    rgacogne committed Mar 27, 2018
    Copy the full SHA
    5708a72 View commit details
    Browse the repository at this point in the history

  5. dnsdist: Don't capture the client var when we don't need it

    @rgacogne
    rgacogne committed Mar 27, 2018
    Copy the full SHA
    7f2f825 View commit details
    Browse the repository at this point in the history

  6. dnsdist: Pass shared pointers by reference whenever possible

    @rgacogne
    rgacogne committed Mar 27, 2018
    Copy the full SHA
    f3b1a1e View commit details
    Browse the repository at this point in the history

Commits on Mar 28, 2018

  1. dnsdist: Constify the function refs passed to Lua{,Response}Action

    @rgacogne
    rgacogne committed Mar 28, 2018
    Copy the full SHA
    41a33fa View commit details
    Browse the repository at this point in the history