Minor code cleanups (#6169)

* Minor code cleanups

* Update claims_based_security_impl.hpp

Undo UniqueAmqpCbsHandle removal - it is right, but will create problems for Larry working in a feature branch when merging it back.

* Update claims_based_security_impl.hpp

---------

Co-authored-by: Anton Kolesnyk <antkmsft@users.noreply.github.com>

sdk/core/azure-core/src/base64.cpp

@@ -3,8 +3,6 @@
 
 #include "azure/core/base64.hpp"
 
-#include "azure/core/platform.hpp"
-
 #include <string>
 #include <vector>
 
@@ -336,15 +334,11 @@ std::string Base64Encode(uint8_t const* const data, size_t length)
   {
     int32_t result = Base64EncodeAndPadTwo(&data[sourceIndex]);
     Base64WriteIntAsFourBytes(destination, result);
-    destination += 4;
-    sourceIndex += 1;
   }
   else if (sourceIndex + 2 == inputSize)
   {
     int32_t result = Base64EncodeAndPadOne(&data[sourceIndex]);
     Base64WriteIntAsFourBytes(destination, result);
-    destination += 4;
-    sourceIndex += 2;
   }
 
   return encodedResult;
@@ -453,7 +447,6 @@ std::vector<uint8_t> Base64Decode(const std::string& text)
     }
 
     Base64WriteThreeLowOrderBytes(destinationPtr, i0);
-    destinationPtr += 3;
   }
   else if (i2 != EncodingPad)
   {
@@ -469,7 +462,6 @@ std::vector<uint8_t> Base64Decode(const std::string& text)
 
     destinationPtr[1] = static_cast<uint8_t>(i0 >> 8);
     destinationPtr[0] = static_cast<uint8_t>(i0 >> 16);
-    destinationPtr += 2;
   }
   else
   {
@@ -479,7 +471,6 @@ std::vector<uint8_t> Base64Decode(const std::string& text)
     }
 
     destinationPtr[0] = static_cast<uint8_t>(i0 >> 16);
-    destinationPtr += 1;
   }
 
   return destination;

sdk/core/azure-core/src/datetime.cpp

@@ -4,7 +4,6 @@
 #include "azure/core/datetime.hpp"
 
 #include "azure/core/internal/strings.hpp"
-#include "azure/core/platform.hpp"
 
 #include <algorithm>
 #include <ctime>

sdk/core/azure-core/src/http/curl/curl.cpp

@@ -14,21 +14,9 @@
 #endif
 #endif
 
-#include "azure/core/platform.hpp"
-
-#if defined(AZ_PLATFORM_WINDOWS)
-#if !defined(WIN32_LEAN_AND_MEAN)
-#define WIN32_LEAN_AND_MEAN
-#endif
-#if !defined(NOMINMAX)
-#define NOMINMAX
-#endif
-#endif
-
 #include "azure/core/http/curl_transport.hpp"
 #include "azure/core/http/http.hpp"
 #include "azure/core/http/policies/policy.hpp"
-#include "azure/core/http/transport.hpp"
 #include "azure/core/internal/diagnostics/log.hpp"
 #include "azure/core/internal/strings.hpp"
 

sdk/core/azure-core/src/http/http.cpp

@@ -3,13 +3,11 @@
 
 #include "azure/core/http/http.hpp"
 
-#include "azure/core/http/policies/policy.hpp"
 #include "azure/core/internal/strings.hpp"
 #include "azure/core/url.hpp"
 
 #include <algorithm>
 #include <unordered_set>
-#include <utility>
 
 using namespace Azure::Core;
 using namespace Azure::Core::Http;

sdk/core/azure-core/src/http/log_policy.cpp

@@ -4,11 +4,8 @@
 #include "azure/core/http/policies/policy.hpp"
 #include "azure/core/internal/diagnostics/log.hpp"
 
-#include <algorithm>
 #include <chrono>
-#include <iterator>
 #include <sstream>
-#include <type_traits>
 
 using Azure::Core::Context;
 using namespace Azure::Core;