Skip to content

dnsdist: squelch unused function warning #7952

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
Merged
rgacogne merged 1 commit into from
Jun 19, 2019
Merged

dnsdist: squelch unused function warning #7952

rgacogne merged 1 commit into from
Jun 19, 2019

Conversation

@pieterlexis
Copy link
Contributor

@pieterlexis pieterlexis commented Jun 19, 2019

Short description

Only compile the loadTLSCertificateAndKeys fucntion when DoT and/or DoH
are enabled.

Takes care of one of the warnings in #7950

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)

@pieterlexis
dnsdist: squelch unused function warning
3900614 
Only compile the loadTLSCertificateAndKeys fucntion when DoT and/or DoH
are enabled.
@pieterlexis pieterlexis requested a review from rgacogne June 19, 2019 12:10
@rgacogne rgacogne added this to the dnsdist-1.4.0 milestone Jun 19, 2019
@rgacogne rgacogne merged commit f973a9a into PowerDNS:master Jun 19, 2019
@pieterlexis pieterlexis deleted the dnsdist-unused-warning branch June 19, 2019 13:58
@fwsmit fwsmit mentioned this pull request Jun 19, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rgacogne rgacogne rgacogne approved these changes

Assignees

No one assigned

Labels

defect dnsdist

Projects

None yet

Milestone

dnsdist-1.4.0

Development

Successfully merging this pull request may close these issues.

2 participants

@pieterlexis @rgacogne