Addressed code review comments.

Author: ras0219-msft

Release/include/cpprest/asyncrt_utils.h

@@ -250,11 +250,11 @@ namespace details
 class windows_category_impl : public std::error_category
 {
 public:
-    virtual const char *name() const _NOEXCEPT { return "windows"; }
+    virtual const char *name() const _noexcept { return "windows"; }
 
-    _ASYNCRTIMP virtual std::string message(int errorCode) const _NOEXCEPT;
+    _ASYNCRTIMP virtual std::string message(int errorCode) const _noexcept;
 
-    _ASYNCRTIMP virtual std::error_condition default_error_condition(int errorCode) const _NOEXCEPT;
+    _ASYNCRTIMP virtual std::error_condition default_error_condition(int errorCode) const _noexcept;
 };
 
 /// <summary>

Release/include/pplx/ioscheduler.h

@@ -149,7 +149,7 @@ class io_scheduler
     /// <summary>
     /// Get the I/O completion key to use with the scheduler.
     /// </summary>
-    DWORD get_key() const { return (((DWORD)(uintptr_t)this) & 0xFAFAFA00) + sizeof(EXTENDED_OVERLAPPED); }
+    DWORD get_key() const { return (static_cast<DWORD>((uintptr_t)this) & 0xFAFAFA00) + sizeof(EXTENDED_OVERLAPPED); }
 
     /// <summary>
     /// Get the I/O scheduler instance.

Release/src/build/vs14/casablanca140.vcxproj

@@ -40,7 +40,7 @@
   </ItemGroup>
   <ItemDefinitionGroup>
     <ClCompile>
-      <PreprocessorDefinitions>_ASYNCRT_EXPORT;AZURESTORAGESERVICES_EXPORTS;_PPLX_EXPORT;WIN32;_MBCS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
+      <PreprocessorDefinitions>_ASYNCRT_EXPORT;_PPLX_EXPORT;WIN32;_MBCS;_USRDLL;%(PreprocessorDefinitions)</PreprocessorDefinitions>
       <AdditionalIncludeDirectories>$(CasablancaIncludeDir);$(CasablancaSrcDir)\pch;$(WebsocketppIncludeDir);%(AdditionalIncludeDirectories)</AdditionalIncludeDirectories>
       <PrecompiledHeader>Use</PrecompiledHeader>
       <PrecompiledHeaderFile>stdafx.h</PrecompiledHeaderFile>

Release/src/utilities/asyncrt_utils.cpp

@@ -177,7 +177,7 @@ const std::error_category & __cdecl windows_category()
     return instance;
 }
 
-std::string windows_category_impl::message(int errorCode) const _NOEXCEPT
+std::string windows_category_impl::message(int errorCode) const _noexcept
 {
     const size_t buffer_size = 4096;
     DWORD dwFlags = FORMAT_MESSAGE_FROM_SYSTEM;
@@ -213,7 +213,7 @@ std::string windows_category_impl::message(int errorCode) const _NOEXCEPT
     return utility::conversions::to_utf8string(buffer);
 }
 
-std::error_condition windows_category_impl::default_error_condition(int errorCode) const _NOEXCEPT
+std::error_condition windows_category_impl::default_error_condition(int errorCode) const _noexcept
 {
     // First see if the STL implementation can handle the mapping for common cases.
     const std::error_condition errCondition = std::system_category().default_error_condition(errorCode);

Release/tests/functional/http/client/proxy_tests.cpp

@@ -113,7 +113,7 @@ TEST_FIXTURE(uri_address, proxy_with_credentials, "Ignore:Linux", "NYI", "Ignore
 		VERIFY_ARE_EQUAL(status_codes::OK, response.status_code());
 		response.content_ready().wait();
 	}
-	catch (web::http::http_exception& e)
+	catch (web::http::http_exception const& e)
 	{
 		if (e.error_code().value() == 12007) {
 			// The above "netproxy.redmond.corp.microsoft.com" is an internal site not generally accessible.