-
Notifications
You must be signed in to change notification settings - Fork 887
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.
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
clang-tidy fixes #13776
base: master
Are you sure you want to change the base?
clang-tidy fixes #13776
Conversation
Pull Request Test Coverage Report for Build 9377096475Details
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The endl -> '\n' change will change flushing behaviour, which is unwanted. We want the implicit flush.
c99170b
to
2420f31
Compare
Removed. |
pdns/sdig.cc
Outdated
@@ -422,7 +422,7 @@ try { | |||
Socket sock(dest.sin4.sin_family, SOCK_STREAM); | |||
sock.setNonBlocking(); | |||
setTCPNoDelay(sock.getHandle()); // disable NAGLE, which does not play nicely with delayed ACKs | |||
TCPIOHandler handler(subjectName, false, sock.releaseHandle(), timeout, std::move(tlsCtx)); | |||
TCPIOHandler handler(subjectName, false, sock.releaseHandle(), timeout, tlsCtx); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coverity does not seem to agree, see previous commit: 3b45a43
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coverity was right at the time, but 224085c changed the constructor.
pdns/pdnsutil.cc
Outdated
sd.db->abortTransaction(); | ||
cerr<<"Backend did not replace SOA record. Backend might not support this operation."<<endl; | ||
return -1; | ||
vector<DNSResourceRecord> rrs(rr); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vector<DNSResourceRecord> rrs(rr); | |
vector<DNSResourceRecord> rrs = {rr}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why =?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The initial syntax is wrong, there is no constructor in the std::vector
class that accepts a single element. Now if your question is why = {rr}
and not ({rr})
, I don't really have an opinion since performance is completely irrelevant here, so whatever compiles would work for me.
Signed-off-by: Rosen Penev <[email protected]>
Signed-off-by: Rosen Penev <[email protected]>
parameter is not passed by value but by reference. Treat it that way. Signed-off-by: Rosen Penev <[email protected]>
CI passes now. |
Some other stuff as well.