Removed all instances of pragma region and pragma endregion

Author: ras0219-msft

Release/include/cpprest/filestream.h

@@ -737,7 +737,6 @@ namespace details {
         }
 #endif
 
-#pragma region Completion callback interface implementations
         class _filestream_callback_open : public details::_filestream_callback
         {
         public:
@@ -942,7 +941,6 @@ namespace details {
             _file_info *m_info;
             pplx::task_completion_event<int_type> m_op;
         };
-#pragma endregion
 
         _file_info *m_info;
         async_operation_queue m_readOps;

Release/include/cpprest/interopstream.h

@@ -57,7 +57,6 @@ namespace Concurrency { namespace streams {
     template<typename CharType> class stdio_ostream;
     template<typename CharType> class stdio_istream;
 
-#pragma region Asynchronous streams on top of synchronous stream buffers.
     namespace details {
 
     /// <summary>
@@ -228,10 +227,6 @@ namespace Concurrency { namespace streams {
         stdio_istream & operator =(const stdio_istream &other) { basic_istream<CharType>::operator=(other); return *this; }
     };
 
-#pragma endregion
-
-#pragma region Synchronous streams on top of asynchronous stream buffers.
-
     namespace details {
 
     /// <summary>
@@ -504,10 +499,8 @@ namespace Concurrency { namespace streams {
     private:
         details::basic_async_streambuf<CharType> m_strbuf;
     };
-#pragma endregion
 
 #if defined(__cplusplus_winrt)
-#pragma region WinRT streams interop
 
     /// <summary>
     /// Static class containing factory functions for WinRT streams implemented on top of Casablanca async streams.
@@ -554,7 +547,6 @@ namespace Concurrency { namespace streams {
     };
 
 #endif
-#pragma endregion
 
 }} // namespaces
 #pragma warning(pop) // 4100

Release/src/json/json.cpp

@@ -47,10 +47,6 @@ utility::istream_t& web::json::operator >> (utility::istream_t &is, json::value
     return is;
 }
 
-#if defined(_MSC_VER)
-#pragma region json::value Constructors
-#endif
-
 web::json::value::value() : 
     m_value(utility::details::make_unique<web::json::details::_Null>())
 #ifdef ENABLE_JSON_VALUE_VISUALIZER
@@ -151,13 +147,6 @@ web::json::value &web::json::value::operator=(web::json::value &&other)
     return *this;
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-
-#pragma region Static Factories
-#endif
-
 web::json::value web::json::value::null()
 {
     return web::json::value();
@@ -250,10 +239,6 @@ web::json::value web::json::value::array(std::vector<value> elements)
             );
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-#endif
-
 web::json::number web::json::value::as_number() const
 {
     return m_value->as_number();

Release/src/uri/uri.cpp

@@ -101,10 +101,6 @@ utility::string_t _uri_components::join()
 
 using namespace details;
 
-#if defined(_MSC_VER)
-#pragma region constructor
-#endif
-
 uri::uri(const utility::string_t &uri_string)
 {
     if (!details::uri_parser().parse(uri_string, m_components))
@@ -123,13 +119,6 @@ uri::uri(const utility::char_t *uri_string): m_uri(uri_string)
     m_uri = m_components.join();
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-
-#pragma region encoding
-#endif
-
 utility::string_t uri::encode_impl(const utility::string_t &raw, const std::function<bool(int)>& should_encode)
 {
     const utility::char_t * const hex = _XPLATSTR("0123456789ABCDEF");
@@ -268,12 +257,6 @@ utility::string_t uri::decode(const utility::string_t &encoded)
     return to_string_t(utf8raw);
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region splitting
-#endif
-
 std::vector<utility::string_t> uri::split_path(const utility::string_t &path)
 {
     std::vector<utility::string_t> results;
@@ -329,23 +312,11 @@ std::map<utility::string_t, utility::string_t> uri::split_query(const utility::s
     return results;
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region validation
-#endif
-
 bool uri::validate(const utility::string_t &uri_string)
 {
     return uri_parser().validate(uri_string);
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region accessors
-#endif
-
 uri uri::authority() const
 {
         return uri_builder().set_scheme(this->scheme()).set_host(this->host()).set_port(this->port()).set_user_info(this->user_info()).to_uri();
@@ -356,12 +327,6 @@ uri uri::resource() const
         return uri_builder().set_path(this->path()).set_query(this->query()).set_fragment(this->fragment()).to_uri();
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region operators
-#endif
-
 bool uri::operator == (const uri &other) const
 {
     // Each individual URI component must be decoded before performing comparison.
@@ -409,8 +374,4 @@ bool uri::operator == (const uri &other) const
     return true;
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-#endif
-
 }

Release/src/uri/uri_builder.cpp

@@ -30,14 +30,6 @@
 namespace web 
 {
 
-#if defined(_MSC_VER)
-#pragma region Validation
-
-#pragma endregion
-
-#pragma region Appending
-#endif
-
 uri_builder &uri_builder::append_path(const utility::string_t &path, bool is_encode)
 {
     if(path.empty() || path == _XPLATSTR("/"))
@@ -113,12 +105,6 @@ uri_builder &uri_builder::append(const http::uri &relative_uri)
     return *this;
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region URI Creation
-#endif
-
 utility::string_t uri_builder::to_string()
 {
     return to_uri().to_string();
@@ -134,9 +120,5 @@ bool uri_builder::is_valid()
     return uri::validate(m_uri.join());
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-#endif
-
 } // namespace web
 

Release/src/utilities/asyncrt_utils.cpp

@@ -157,10 +157,6 @@ scoped_c_thread_locale::~scoped_c_thread_locale()
 #endif
 }
 
-#if defined(_MSC_VER)
-#pragma region error categories
-#endif
-
 namespace details
 {
 
@@ -264,12 +260,6 @@ const std::error_category & __cdecl linux_category()
 
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region conversions
-#endif
-
 utf16string __cdecl conversions::utf8_to_utf16(const std::string &s)
 {
     if(s.empty())
@@ -595,12 +585,6 @@ utf16string __cdecl conversions::to_utf16string(const std::string &value) { retu
 
 utf16string __cdecl conversions::to_utf16string(utf16string value) { return std::move(value); }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region datetime
-#endif
-
 #ifndef WIN32
 datetime datetime::timeval_to_datetime(struct timeval time)
 {
@@ -1038,12 +1022,6 @@ datetime __cdecl datetime::from_string(const utility::string_t& dateString, date
 #endif
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-
-#pragma region "timespan"
-#endif
-
 /// <summary>
 /// Converts a timespan/interval in seconds to xml duration string as specified by
 /// http://www.w3.org/TR/xmlschema-2/#duration
@@ -1152,11 +1130,6 @@ utility::seconds __cdecl timespan::xml_duration_to_seconds(utility::string_t tim
     return utility::seconds(numSecs);
 }
 
-#if defined(_MSC_VER)
-#pragma endregion
-#endif
-
-
 const utility::string_t nonce_generator::c_allowed_chars(_XPLATSTR("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789"));
 
 utility::string_t nonce_generator::generate()