diff options
| author | Maxime Coste <mawww@kakoune.org> | 2018-03-25 16:53:27 +1100 |
|---|---|---|
| committer | Maxime Coste <mawww@kakoune.org> | 2018-03-25 16:53:27 +1100 |
| commit | a6b1d142fa1aef8824318c8647a74503d5fef015 (patch) | |
| tree | ecb0c2e9c2a3c5862c93f3a941a07304b9100d29 /src/client_manager.cc | |
| parent | 49c52b025f2cb5bd66f729f873d2e690b613fb39 (diff) | |
Cleanup client name validation code
Diffstat (limited to 'src/client_manager.cc')
| -rw-r--r-- | src/client_manager.cc | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/client_manager.cc b/src/client_manager.cc index 8fe19ece..cfc44cbc 100644 --- a/src/client_manager.cc +++ b/src/client_manager.cc @@ -33,7 +33,7 @@ String ClientManager::generate_name() const for (int i = 0; true; ++i) { String name = format("unnamed{}", i); - if (validate_client_name(name)) + if (not client_name_exists(name)) return name; } } @@ -172,10 +172,9 @@ void ClientManager::clear_client_trash() m_client_trash.clear(); } -bool ClientManager::validate_client_name(StringView name) const +bool ClientManager::client_name_exists(StringView name) const { - return all_of(name, is_identifier) and - const_cast<ClientManager*>(this)->get_client_ifp(name) == nullptr; + return const_cast<ClientManager*>(this)->get_client_ifp(name) != nullptr; } Client* ClientManager::get_client_ifp(StringView name) |