From 3cb49934238aad9a1615e4ce98dbf360d48158c6 Mon Sep 17 00:00:00 2001 From: Larry Osterman Date: Tue, 4 Apr 2023 11:31:37 -0700 Subject: [PATCH] Removed 'using namespace' directive from public header (#4514) * Removed 'using namespace' directive from public header * Shortened namespace names (Thanks Antkmsft) * Update sdk/keyvault/azure-security-keyvault-administration/inc/azure/keyvault/administration/settings_client.hpp Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com> --------- Co-authored-by: Anton Kolesnyk <41349689+antkmsft@users.noreply.github.com> --- .../azure/keyvault/administration/settings_client.hpp | 10 ++++------ .../src/settings_client.cpp | 1 + 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/sdk/keyvault/azure-security-keyvault-administration/inc/azure/keyvault/administration/settings_client.hpp b/sdk/keyvault/azure-security-keyvault-administration/inc/azure/keyvault/administration/settings_client.hpp index b73a4e054..1968c28ed 100644 --- a/sdk/keyvault/azure-security-keyvault-administration/inc/azure/keyvault/administration/settings_client.hpp +++ b/sdk/keyvault/azure-security-keyvault-administration/inc/azure/keyvault/administration/settings_client.hpp @@ -17,8 +17,6 @@ #include #include -using namespace Azure::Security::KeyVault::Administration::Models; - namespace Azure { namespace Security { namespace KeyVault { namespace Administration { /** * @brief Settings Client class. @@ -67,7 +65,7 @@ namespace Azure { namespace Security { namespace KeyVault { namespace Administra * * @returns Response containing the new updated setting. */ - Azure::Response UpdateSetting( + Azure::Response UpdateSetting( std::string const& name, std::string const& value, const Azure::Core::Context& context = Azure::Core::Context{}) const; @@ -80,7 +78,7 @@ namespace Azure { namespace Security { namespace KeyVault { namespace Administra * * @returns response containing the setting. */ - Azure::Response GetSetting( + Azure::Response GetSetting( std::string const& name, const Azure::Core::Context& context = Azure::Core::Context{}) const; @@ -91,11 +89,11 @@ namespace Azure { namespace Security { namespace KeyVault { namespace Administra * * @returns Response containing a list of settings. */ - Azure::Response GetSettings( + Azure::Response GetSettings( const Azure::Core::Context& context = Azure::Core::Context{}) const; private: - Setting ParseSetting(std::vector const& responseBody) const; + Models::Setting ParseSetting(std::vector const& responseBody) const; std::unique_ptr SendRequest( Azure::Core::Http::Request& request, diff --git a/sdk/keyvault/azure-security-keyvault-administration/src/settings_client.cpp b/sdk/keyvault/azure-security-keyvault-administration/src/settings_client.cpp index de106ec4e..195876bd7 100644 --- a/sdk/keyvault/azure-security-keyvault-administration/src/settings_client.cpp +++ b/sdk/keyvault/azure-security-keyvault-administration/src/settings_client.cpp @@ -19,6 +19,7 @@ using namespace Azure::Core::Http::Policies; using namespace Azure::Core::Http::Policies::_internal; using namespace Azure::Core::Json::_internal; using namespace Azure::Security::KeyVault::Administration; +using namespace Azure::Security::KeyVault::Administration::Models; using namespace Azure::Security::KeyVault::Administration::_detail; std::unique_ptr SettingsClient::SendRequest(