mirror of
https://github.com/bitcoin/bitcoin.git
synced 2025-02-12 11:19:08 -05:00
Merge bitcoin/bitcoin#30868: refactor: add clang-tidy modernize-use-starts-ends-with
check
fc7b507e9a
tidy: add clang-tidy `modernize-use-starts-ends-with` check (Roman Zeyde) Pull request description: ACKs for top commit: jonatack: re-ACKfc7b507e9a
only change since my previous ACK is the commit message achow101: ACKfc7b507e9a
stickies-v: ACKfc7b507e9a
hebasto: ACKfc7b507e9a
, I have reviewed the code and it looks OK. Tree-SHA512: 334e0ff91b9b108a57cdfc12ee53685b792d377e11124c7c394b8f681a8168a8d65a56c7f884555238e65e97e9ad62ede52b79219ce258979e54abdd76721df1
This commit is contained in:
commit
8d000b85dd
4 changed files with 4 additions and 3 deletions
|
@ -14,6 +14,7 @@ modernize-use-emplace,
|
|||
modernize-use-equals-default,
|
||||
modernize-use-noexcept,
|
||||
modernize-use-nullptr,
|
||||
modernize-use-starts-ends-with,
|
||||
performance-*,
|
||||
-performance-avoid-endl,
|
||||
-performance-enum-size,
|
||||
|
|
|
@ -315,7 +315,7 @@ static void http_request_cb(struct evhttp_request* req, void* arg)
|
|||
if (i->exactMatch)
|
||||
match = (strURI == i->prefix);
|
||||
else
|
||||
match = (strURI.substr(0, i->prefix.size()) == i->prefix);
|
||||
match = strURI.starts_with(i->prefix);
|
||||
if (match) {
|
||||
path = strURI.substr(i->prefix.size());
|
||||
break;
|
||||
|
|
|
@ -1356,7 +1356,7 @@ bool AppInitMain(NodeContext& node, interfaces::BlockAndHeaderTipInfo* tip_info)
|
|||
if (!SplitHostPort(socket_addr, port_out, host_out)) {
|
||||
#ifdef HAVE_SOCKADDR_UN
|
||||
// Allow unix domain sockets for some options e.g. unix:/some/file/path
|
||||
if (!unix || socket_addr.find(ADDR_PREFIX_UNIX) != 0) {
|
||||
if (!unix || !socket_addr.starts_with(ADDR_PREFIX_UNIX)) {
|
||||
return InitError(InvalidPortErrMsg(arg, socket_addr));
|
||||
}
|
||||
#else
|
||||
|
|
|
@ -230,7 +230,7 @@ CService LookupNumeric(const std::string& name, uint16_t portDefault, DNSLookupF
|
|||
bool IsUnixSocketPath(const std::string& name)
|
||||
{
|
||||
#ifdef HAVE_SOCKADDR_UN
|
||||
if (name.find(ADDR_PREFIX_UNIX) != 0) return false;
|
||||
if (!name.starts_with(ADDR_PREFIX_UNIX)) return false;
|
||||
|
||||
// Split off "unix:" prefix
|
||||
std::string str{name.substr(ADDR_PREFIX_UNIX.length())};
|
||||
|
|
Loading…
Add table
Reference in a new issue