From f067deb563d1073a7193b87d058fce37838ea5aa Mon Sep 17 00:00:00 2001 From: cyy Date: Thu, 1 Jul 2021 14:35:25 +0800 Subject: [PATCH] avoid unnecessary strlen (#3058) --- include/pybind11/attr.h | 2 +- include/pybind11/pybind11.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/include/pybind11/attr.h b/include/pybind11/attr.h index 50efdc7ce..97147904c 100644 --- a/include/pybind11/attr.h +++ b/include/pybind11/attr.h @@ -377,7 +377,7 @@ template <> struct process_attribute : process_attribu }; inline void process_kw_only_arg(const arg &a, function_record *r) { - if (!a.name || strlen(a.name) == 0) + if (!a.name || a.name[0] == '\0') pybind11_fail("arg(): cannot specify an unnamed argument after an kw_only() annotation"); ++r->nargs_kw_only; } diff --git a/include/pybind11/pybind11.h b/include/pybind11/pybind11.h index 1fb0d1fa1..9938b0b71 100644 --- a/include/pybind11/pybind11.h +++ b/include/pybind11/pybind11.h @@ -470,7 +470,7 @@ protected: signatures += it->signature; signatures += "\n"; } - if (it->doc && strlen(it->doc) > 0 && options::show_user_defined_docstrings()) { + if (it->doc && it->doc[0] != '\0' && options::show_user_defined_docstrings()) { // If we're appending another docstring, and aren't printing function signatures, we // need to append a newline first: if (!options::show_function_signatures()) {