Ros-diffs January 2025

ros-diffs@reactos.org

23 participants
278 discussions

[reactos] 28/33: [UCRT] Hack some template code, so that retarded compilers can handle it

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=693f9f51a04084b13b26b… commit 693f9f51a04084b13b26bfe0600e1735eaebd005 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Mon Oct 14 00:12:32 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Hack some template code, so that retarded compilers can handle it --- sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h | 64 ++++++++++++------------ 1 file changed, 32 insertions(+), 32 deletions(-) diff --git a/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h b/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h index 6f9878a52cc..86d7ffb4ecf 100644 --- a/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h +++ b/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h @@ -1009,7 +1009,7 @@ protected: __crt_cached_ptd_host& ptd, va_list const arglist ) throw() - : common_data{ptd}, + : common_data<Character>{ptd}, _output_adapter(output_adapter) { // We initialize several base class data members here, so that we can @@ -1209,7 +1209,7 @@ protected: #endif typedef positional_parameter_base self_type; - typedef format_validation_base base_type; + typedef format_validation_base<Character, OutputAdapter> base_type; typedef __crt_char_traits<Character> char_traits; template <typename... Ts> @@ -1296,7 +1296,7 @@ protected: { if (_format_mode == mode::nonpositional) { - return base_type::extract_argument_from_va_list<RequestedParameterType>(result); + return base_type::template extract_argument_from_va_list<RequestedParameterType>(result); } _UCRT_VALIDATE_RETURN(_ptd, _type_index >= 0 && _type_index < _ARGMAX, EINVAL, false); @@ -2274,7 +2274,7 @@ private: if (is_wide_character_specifier(_options, _format_char, _length)) { wchar_t wide_character{}; - if (!extract_argument_from_va_list<wchar_t>(wide_character)) + if (!this->template extract_argument_from_va_list<wchar_t>(wide_character)) { return false; } @@ -2286,7 +2286,7 @@ private: // Convert to multibyte. If the conversion fails, we suppress the // output operation but we do not fail the entire operation: - errno_t const status{_wctomb_internal(&_string_length, _buffer.data<char>(), _buffer.count<char>(), wide_character, _ptd)}; + errno_t const status{_wctomb_internal(&_string_length, _buffer.template data<char>(), _buffer.template count<char>(), wide_character, _ptd)}; if (status != 0) { _suppress_output = true; @@ -2296,7 +2296,7 @@ private: // to the output, as-is. else { - if (!extract_argument_from_va_list<unsigned short>(_buffer.data<char>()[0])) + if (!this->template extract_argument_from_va_list<unsigned short>(_buffer.template data<char>()[0])) { return false; } @@ -2309,7 +2309,7 @@ private: _string_length = 1; } - _narrow_string = _buffer.data<char>(); + _narrow_string = _buffer.template data<char>(); return true; } @@ -2320,7 +2320,7 @@ private: _string_is_wide = true; wchar_t wide_character{}; - if (!extract_argument_from_va_list<wchar_t>(wide_character)) + if (!this->template extract_argument_from_va_list<wchar_t>(wide_character)) return false; if (!should_format()) @@ -2334,7 +2334,7 @@ private: // output operation. char const local_buffer[2]{ static_cast<char>(wide_character & 0x00ff), '\0' }; int const mbc_length{_mbtowc_internal( - _buffer.data<wchar_t>(), + _buffer.template data<wchar_t>(), local_buffer, _ptd.get_locale()->locinfo->_public._locale_mb_cur_max, _ptd @@ -2346,10 +2346,10 @@ private: } else { - _buffer.data<wchar_t>()[0] = wide_character; + _buffer.template data<wchar_t>()[0] = wide_character; } - _wide_string = _buffer.data<wchar_t>(); + _wide_string = _buffer.template data<wchar_t>(); _string_length = 1; return true; } @@ -2371,7 +2371,7 @@ private: }; ansi_string* string{}; - if (!extract_argument_from_va_list<ansi_string*>(string)) + if (!this->template extract_argument_from_va_list<ansi_string*>(string)) return false; if (!should_format()) @@ -2406,7 +2406,7 @@ private: // of the length of the C string and the given precision. Note that the // string needs not be null-terminated if a precision is given, so we // cannot call strlen to compute the length of the string. - if (!extract_argument_from_va_list<char*>(_narrow_string)) + if (!this->template extract_argument_from_va_list<char*>(_narrow_string)) return false; if (!should_format()) @@ -2505,18 +2505,18 @@ private: _precision = 1; // Per C Standard Library specification. } - if (!_buffer.ensure_buffer_is_big_enough<char>(_CVTBUFSIZE + _precision, _ptd)) + if (!_buffer.template ensure_buffer_is_big_enough<char>(_CVTBUFSIZE + _precision, _ptd)) { // If we fail to enlarge the buffer, cap precision so that the // statically-sized buffer may be used for the formatting: - _precision = static_cast<int>(_buffer.count<char>() - _CVTBUFSIZE); + _precision = static_cast<int>(_buffer.template count<char>() - _CVTBUFSIZE); } - _narrow_string = _buffer.data<char>(); + _narrow_string = _buffer.template data<char>(); // Note that we separately handle the FORMAT_POSSCAN_PASS above. _CRT_DOUBLE tmp{}; - if (!extract_argument_from_va_list<_CRT_DOUBLE>(tmp)) + if (!this->template extract_argument_from_va_list<_CRT_DOUBLE>(tmp)) { return false; } @@ -2524,10 +2524,10 @@ private: // Format the number into the buffer: __acrt_fp_format( &tmp.x, - _buffer.data<char>(), - _buffer.count<char>(), - _buffer.scratch_data<char>(), - _buffer.scratch_count<char>(), + _buffer.template data<char>(), + _buffer.template count<char>(), + _buffer.template scratch_data<char>(), + _buffer.template scratch_count<char>(), static_cast<char>(_format_char), _precision, _options, @@ -2639,23 +2639,23 @@ private: { case sizeof(int8_t): extraction_result = has_flag(FL_SIGNED) - ? extract_argument_from_va_list<int8_t >(original_number) - : extract_argument_from_va_list<uint8_t>(original_number); + ? this->template extract_argument_from_va_list<int8_t >(original_number) + : this->template extract_argument_from_va_list<uint8_t>(original_number); break; case sizeof(int16_t): extraction_result = has_flag(FL_SIGNED) - ? extract_argument_from_va_list<int16_t >(original_number) - : extract_argument_from_va_list<uint16_t>(original_number); + ? this->template extract_argument_from_va_list<int16_t >(original_number) + : this->template extract_argument_from_va_list<uint16_t>(original_number); break; case sizeof(int32_t): extraction_result = has_flag(FL_SIGNED) - ? extract_argument_from_va_list<int32_t >(original_number) - : extract_argument_from_va_list<uint32_t>(original_number); + ? this->template extract_argument_from_va_list<int32_t >(original_number) + : this->template extract_argument_from_va_list<uint32_t>(original_number); break; case sizeof(int64_t): extraction_result = has_flag(FL_SIGNED) - ? extract_argument_from_va_list<int64_t >(original_number) - : extract_argument_from_va_list<uint64_t>(original_number); + ? this->template extract_argument_from_va_list<int64_t >(original_number) + : this->template extract_argument_from_va_list<uint64_t>(original_number); break; default: _UCRT_VALIDATE_RETURN(_ptd, ("Invalid integer length modifier", 0), EINVAL, false); @@ -2695,7 +2695,7 @@ private: else { unset_flag(FL_LEADZERO); - _buffer.ensure_buffer_is_big_enough<Character>(_precision, _ptd); + _buffer.template ensure_buffer_is_big_enough<Character>(_precision, _ptd); } // If the number is zero, we do not want to print the hex prefix ("0x"), @@ -2741,7 +2741,7 @@ private: // buffer at the end of the formatting buffer, which allows us to perform // the formatting from least to greatest magnitude, which maps well to // the math. - Character* const last_digit{_buffer.data<Character>() + _buffer.count<Character>() - 1}; + Character* const last_digit{_buffer.template data<Character>() + _buffer.template count<Character>() - 1}; Character*& string_pointer = tchar_string(); @@ -2773,7 +2773,7 @@ private: bool type_case_n() throw() { void* p{nullptr}; - if (!extract_argument_from_va_list<void*>(p)) + if (!this->template extract_argument_from_va_list<void*>(p)) return false; if (!should_format())

2 weeks, 6 days

[reactos] 27/33: [UCRT] Use a C cast instead of const_cast for GCC

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f9dc185ede4a0a3c9bda1… commit f9dc185ede4a0a3c9bda1a4a77d034ceebda8d41 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Sun Oct 13 17:02:51 2024 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Use a C cast instead of const_cast for GCC This is because GCC will generate errors for both const_cast and static_cast depending on the template parameter: - With const_cast: error: invalid 'const_cast' from type 'std::nullptr_t' to type 'const wchar_t**' - With static_cast: error: invalid 'static_cast' from type 'wchar_t** const' to type 'const wchar_t**' --- sdk/lib/ucrt/inc/corecrt_internal_strtox.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sdk/lib/ucrt/inc/corecrt_internal_strtox.h b/sdk/lib/ucrt/inc/corecrt_internal_strtox.h index 102724dda7d..d3c4a0c0fb7 100644 --- a/sdk/lib/ucrt/inc/corecrt_internal_strtox.h +++ b/sdk/lib/ucrt/inc/corecrt_internal_strtox.h @@ -1870,7 +1870,7 @@ c_string_character_source<Character> __cdecl make_c_string_character_source( EndPointer const end ) throw() { - return c_string_character_source<Character>(string, const_cast<Character const**>(end)); + return c_string_character_source<Character>(string, (Character const**)(end)); } template <typename Integer, typename Character, typename EndPointer>

2 weeks, 6 days

[reactos] 26/33: [UCRT] No __declspec(spectre(nomitigation)) for GCC/Clang

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=47e3f49e5266ce9e4668b… commit 47e3f49e5266ce9e4668bc96140161877ed0ba41 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Mon Oct 14 00:08:12 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] No __declspec(spectre(nomitigation)) for GCC/Clang --- sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h b/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h index bb4d545d88c..6f9878a52cc 100644 --- a/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h +++ b/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h @@ -576,7 +576,10 @@ public: // Instead of using an lfence mitigation, we can fill the table to a power of two, // then bitwise-and all values used to index into the array. - __declspec(spectre(nomitigation)) T const& operator[](size_t const index) const + #if defined(_MSC_VER) && !defined(__clang__) + __declspec(spectre(nomitigation)) + #endif + T const& operator[](size_t const index) const { return m_array[index & mask]; }

2 weeks, 6 days

[reactos] 25/33: [UCRT] Fix GCC/Clang build of SSE2/AVX code

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=97e20ef342300158940d6… commit 97e20ef342300158940d61892cdc70ed63bf93cd Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Sun Nov 10 14:57:01 2024 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Fix GCC/Clang build of SSE2/AVX code On GCC the code using extended ISA cannot be inlined and must either be marked with a function attribute or compiled with the proper target options ('-mavx2' on the command line or '#pragma GCC target("avx2")' inside the code) --- sdk/lib/ucrt/inc/corecrt_internal_simd.h | 20 ++++++++++++++++++++ sdk/lib/ucrt/string/strnlen.cpp | 6 +++++- 2 files changed, 25 insertions(+), 1 deletion(-) diff --git a/sdk/lib/ucrt/inc/corecrt_internal_simd.h b/sdk/lib/ucrt/inc/corecrt_internal_simd.h index 5770418e034..01a92229438 100644 --- a/sdk/lib/ucrt/inc/corecrt_internal_simd.h +++ b/sdk/lib/ucrt/inc/corecrt_internal_simd.h @@ -18,6 +18,24 @@ #if defined _CRT_SIMD_SUPPORT_AVAILABLE +#if defined(__clang__) +#define _UCRT_ENABLE_EXTENDED_ISA \ + _Pragma("clang attribute push(__attribute__((target(\"sse2,avx,avx2\"))), apply_to=function)") +#define _UCRT_RESTORE_DEFAULT_ISA \ + _Pragma("clang attribute pop") +#elif defined(__GNUC__) +#define _UCRT_ENABLE_EXTENDED_ISA \ + _Pragma("GCC push_options") \ + _Pragma("GCC target(\"avx2\")") +#define _UCRT_RESTORE_DEFAULT_ISA \ + _Pragma("GCC pop_options") +#else +#define _UCRT_ENABLE_EXTENDED_ISA +#define _UCRT_RESTORE_DEFAULT_ISA +#endif + +_UCRT_ENABLE_EXTENDED_ISA + extern "C" int __isa_available; enum class __crt_simd_isa @@ -155,4 +173,6 @@ } }; +_UCRT_RESTORE_DEFAULT_ISA + #endif // _CRT_SIMD_SUPPORT_AVAILABLE diff --git a/sdk/lib/ucrt/string/strnlen.cpp b/sdk/lib/ucrt/string/strnlen.cpp index 6cee4e85e0f..302086cb9f9 100644 --- a/sdk/lib/ucrt/string/strnlen.cpp +++ b/sdk/lib/ucrt/string/strnlen.cpp @@ -78,11 +78,13 @@ static __forceinline size_t __cdecl common_strnlen_c( #ifdef _CRT_SIMD_SUPPORT_AVAILABLE +_UCRT_ENABLE_EXTENDED_ISA + template <strnlen_mode Mode, __crt_simd_isa Isa, typename Element> _Check_return_ _When_(maximum_count > _String_length_(string), _Post_satisfies_(return == _String_length_(string))) _When_(maximum_count <= _String_length_(string), _Post_satisfies_(return == maximum_count)) - static __forceinline size_t __cdecl common_strnlen_simd( + static __inline size_t __cdecl common_strnlen_simd( Element const* const string, size_t const maximum_count ) throw() @@ -168,6 +170,8 @@ static __forceinline size_t __cdecl common_strnlen_c( return static_cast<size_t>(it - string); } +_UCRT_RESTORE_DEFAULT_ISA + #endif // _CRT_SIMD_SUPPORT_AVAILABLE template <strnlen_mode Mode, typename Element>

2 weeks, 6 days

[reactos] 24/33: [UCRT] Add GCC compatible __FUNCTIONW__

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=51e6ce3aa7c880709f8f2… commit 51e6ce3aa7c880709f8f29483d8df85c22c560ca Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Sun Oct 13 10:55:31 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Add GCC compatible __FUNCTIONW__ --- sdk/include/ucrt/corecrt.h | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/sdk/include/ucrt/corecrt.h b/sdk/include/ucrt/corecrt.h index b89f0a94c22..4fe20566df9 100644 --- a/sdk/include/ucrt/corecrt.h +++ b/sdk/include/ucrt/corecrt.h @@ -280,7 +280,18 @@ extern "C++" #define _CRT_SIZE_MAX ((size_t)-1) #define __FILEW__ _CRT_WIDE(__FILE__) + +#ifdef _MSC_VER #define __FUNCTIONW__ _CRT_WIDE(__FUNCTION__) +#else // _MSC_VER +#define __FUNCTIONW__ \ +({ \ + static wchar_t __funcw__[sizeof(__func__)]; \ + for (size_t i = 0; i < sizeof(__func__); i++) \ + __funcw__[i] = __func__[i]; \ + __funcw__; \ +}) +#endif // _MSC_VER #ifdef __cplusplus #ifndef _STATIC_ASSERT

2 weeks, 6 days

[reactos] 23/33: [UCRT] Include intrin.h for __debugbreak()

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=589bbcaa086b77f7dde51… commit 589bbcaa086b77f7dde51a691f4318255091c1b6 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Sun Oct 13 16:53:29 2024 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Include intrin.h for __debugbreak() --- sdk/include/ucrt/crtdbg.h | 1 + 1 file changed, 1 insertion(+) diff --git a/sdk/include/ucrt/crtdbg.h b/sdk/include/ucrt/crtdbg.h index 90925ff4266..e9f6952aced 100644 --- a/sdk/include/ucrt/crtdbg.h +++ b/sdk/include/ucrt/crtdbg.h @@ -11,6 +11,7 @@ #include <corecrt.h> #include <vcruntime_new_debug.h> +#include <intrin.h> // for __debugbreak() on GCC #pragma warning(push) #pragma warning(disable: _UCRT_DISABLED_WARNINGS)

2 weeks, 6 days

[reactos] 22/33: [UCRT] Make SEH blocks ReactOS PSEH compatible

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e98e9000c7e67e8293628… commit e98e9000c7e67e8293628c82e8342a6fab69697e Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Sat Oct 12 00:05:27 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Make SEH blocks ReactOS PSEH compatible --- sdk/lib/ucrt/conio/cgets.cpp | 1 + sdk/lib/ucrt/conio/cgetws.cpp | 1 + sdk/lib/ucrt/conio/cputs.cpp | 1 + sdk/lib/ucrt/conio/cputws.cpp | 1 + sdk/lib/ucrt/conio/getch.cpp | 5 +++++ sdk/lib/ucrt/conio/getwch.cpp | 4 ++++ sdk/lib/ucrt/conio/pipe.cpp | 2 ++ sdk/lib/ucrt/conio/popen.cpp | 2 ++ sdk/lib/ucrt/env/getenv.cpp | 3 +++ sdk/lib/ucrt/env/putenv.cpp | 1 + sdk/lib/ucrt/heap/debug_heap.cpp | 14 ++++++++++++++ sdk/lib/ucrt/heap/heapwalk.cpp | 1 + sdk/lib/ucrt/heap/new_handler.cpp | 2 ++ sdk/lib/ucrt/inc/internal_shared.h | 1 + sdk/lib/ucrt/locale/locale_refcounting.cpp | 1 + sdk/lib/ucrt/locale/wsetlocale.cpp | 2 ++ sdk/lib/ucrt/lowio/dup.cpp | 1 + sdk/lib/ucrt/lowio/dup2.cpp | 1 + sdk/lib/ucrt/lowio/filelength.cpp | 1 + sdk/lib/ucrt/lowio/ioinit.cpp | 1 + sdk/lib/ucrt/lowio/locking.cpp | 1 + sdk/lib/ucrt/lowio/lseek.cpp | 1 + sdk/lib/ucrt/lowio/open.cpp | 2 ++ sdk/lib/ucrt/lowio/osfinfo.cpp | 3 +++ sdk/lib/ucrt/lowio/read.cpp | 1 + sdk/lib/ucrt/lowio/setmode.cpp | 1 + sdk/lib/ucrt/lowio/write.cpp | 1 + sdk/lib/ucrt/mbstring/mbctype.cpp | 1 + sdk/lib/ucrt/misc/dbgrptt.cpp | 4 ++++ sdk/lib/ucrt/misc/perror.cpp | 1 + sdk/lib/ucrt/misc/signal.cpp | 3 +++ sdk/lib/ucrt/misc/terminate.cpp | 1 + sdk/lib/ucrt/startup/exit.cpp | 1 + sdk/lib/ucrt/startup/thread.cpp | 1 + sdk/lib/ucrt/stdio/clearerr.cpp | 1 + sdk/lib/ucrt/stdio/closeall.cpp | 1 + sdk/lib/ucrt/stdio/fclose.cpp | 1 + sdk/lib/ucrt/stdio/fdopen.cpp | 1 + sdk/lib/ucrt/stdio/fgetc.cpp | 1 + sdk/lib/ucrt/stdio/fgets.cpp | 1 + sdk/lib/ucrt/stdio/fgetwc.cpp | 1 + sdk/lib/ucrt/stdio/fopen.cpp | 1 + sdk/lib/ucrt/stdio/fputc.cpp | 1 + sdk/lib/ucrt/stdio/fputwc.cpp | 1 + sdk/lib/ucrt/stdio/fread.cpp | 1 + sdk/lib/ucrt/stdio/freopen.cpp | 1 + sdk/lib/ucrt/stdio/fseek.cpp | 1 + sdk/lib/ucrt/stdio/ftell.cpp | 1 + sdk/lib/ucrt/stdio/gets.cpp | 1 + sdk/lib/ucrt/stdio/getw.cpp | 1 + sdk/lib/ucrt/stdio/putw.cpp | 1 + sdk/lib/ucrt/stdio/rewind.cpp | 1 + sdk/lib/ucrt/stdio/rmtmp.cpp | 2 ++ sdk/lib/ucrt/stdio/stream.cpp | 1 + sdk/lib/ucrt/stdio/tempnam.cpp | 1 + sdk/lib/ucrt/stdio/tmpfile.cpp | 3 +++ sdk/lib/ucrt/stdio/ungetc.cpp | 1 + sdk/lib/ucrt/stdio/ungetwc.cpp | 1 + sdk/lib/ucrt/time/tzset.cpp | 3 +++ 59 files changed, 98 insertions(+) diff --git a/sdk/lib/ucrt/conio/cgets.cpp b/sdk/lib/ucrt/conio/cgets.cpp index fe3fffea718..7ec4ae6dcfe 100644 --- a/sdk/lib/ucrt/conio/cgets.cpp +++ b/sdk/lib/ucrt/conio/cgets.cpp @@ -91,6 +91,7 @@ extern "C" errno_t __cdecl _cgets_s(char* const source_string, size_t const size { __acrt_unlock(__acrt_conio_lock); } + __endtry *string++ = '\0'; diff --git a/sdk/lib/ucrt/conio/cgetws.cpp b/sdk/lib/ucrt/conio/cgetws.cpp index 0149e82b2b4..3b40b722a3c 100644 --- a/sdk/lib/ucrt/conio/cgetws.cpp +++ b/sdk/lib/ucrt/conio/cgetws.cpp @@ -121,6 +121,7 @@ extern "C" errno_t __cdecl _cgetws_s(wchar_t* const string_buffer, size_t const { __acrt_unlock(__acrt_conio_lock); } + __endtry return retval; } diff --git a/sdk/lib/ucrt/conio/cputs.cpp b/sdk/lib/ucrt/conio/cputs.cpp index 58e312d37c6..96e2232b544 100644 --- a/sdk/lib/ucrt/conio/cputs.cpp +++ b/sdk/lib/ucrt/conio/cputs.cpp @@ -34,5 +34,6 @@ extern "C" int __cdecl _cputs(char const* const string) { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } diff --git a/sdk/lib/ucrt/conio/cputws.cpp b/sdk/lib/ucrt/conio/cputws.cpp index 2a16e4fac0a..fcede25a0cb 100644 --- a/sdk/lib/ucrt/conio/cputws.cpp +++ b/sdk/lib/ucrt/conio/cputws.cpp @@ -58,6 +58,7 @@ extern "C" int __cdecl _cputws(wchar_t const* string) { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } diff --git a/sdk/lib/ucrt/conio/getch.cpp b/sdk/lib/ucrt/conio/getch.cpp index 0a8699b8053..9850aaf0cbc 100644 --- a/sdk/lib/ucrt/conio/getch.cpp +++ b/sdk/lib/ucrt/conio/getch.cpp @@ -254,6 +254,7 @@ extern "C" int __cdecl _getch() { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } @@ -269,6 +270,7 @@ extern "C" int __cdecl _getche() { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } @@ -353,6 +355,7 @@ extern "C" int __cdecl _getch_nolock() // Restore the previous console mode: __dcrt_set_input_console_mode(old_console_mode); } + __endtry return result; } @@ -394,6 +397,7 @@ extern "C" int __cdecl _kbhit() { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } @@ -472,6 +476,7 @@ extern "C" int __cdecl _ungetch(int const c) { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } diff --git a/sdk/lib/ucrt/conio/getwch.cpp b/sdk/lib/ucrt/conio/getwch.cpp index cfc0a8ab7ac..f8fe8dd1a36 100644 --- a/sdk/lib/ucrt/conio/getwch.cpp +++ b/sdk/lib/ucrt/conio/getwch.cpp @@ -51,6 +51,7 @@ extern "C" wint_t __cdecl _getwch() { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } @@ -67,6 +68,7 @@ extern "C" wint_t __cdecl _getwche() { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } @@ -140,6 +142,7 @@ extern "C" wint_t __cdecl _getwch_nolock() // Restore the previous console mode: __dcrt_set_input_console_mode(old_console_mode); } + __endtry return result; } @@ -185,6 +188,7 @@ extern "C" wint_t __cdecl _ungetwch(wint_t const c) { __acrt_unlock(__acrt_conio_lock); } + __endtry return result; } diff --git a/sdk/lib/ucrt/conio/pipe.cpp b/sdk/lib/ucrt/conio/pipe.cpp index 17bb5b0ee83..33b5ffd4b69 100644 --- a/sdk/lib/ucrt/conio/pipe.cpp +++ b/sdk/lib/ucrt/conio/pipe.cpp @@ -61,6 +61,7 @@ extern "C" int __cdecl _pipe(int* const phandles, unsigned const psize, int cons { __acrt_lowio_unlock_fh(crt_read_handle); } + __endtry // Create the CRT write handle for the pipe: int const crt_write_handle = _alloc_osfhnd(); @@ -84,6 +85,7 @@ extern "C" int __cdecl _pipe(int* const phandles, unsigned const psize, int cons { __acrt_lowio_unlock_fh(crt_write_handle); } + __endtry // Figure out which textmode the file gets: int fmode = 0; diff --git a/sdk/lib/ucrt/conio/popen.cpp b/sdk/lib/ucrt/conio/popen.cpp index 77d73495530..f1a524cfae4 100644 --- a/sdk/lib/ucrt/conio/popen.cpp +++ b/sdk/lib/ucrt/conio/popen.cpp @@ -400,6 +400,7 @@ static FILE* __cdecl common_popen( { __acrt_unlock(__acrt_popen_lock); } + __endtry return return_value; } @@ -483,6 +484,7 @@ extern "C" int __cdecl _pclose(FILE* const stream) { __acrt_unlock(__acrt_popen_lock); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/env/getenv.cpp b/sdk/lib/ucrt/env/getenv.cpp index 5e9c24894ef..05faf989d6a 100644 --- a/sdk/lib/ucrt/env/getenv.cpp +++ b/sdk/lib/ucrt/env/getenv.cpp @@ -76,6 +76,7 @@ static Character* __cdecl common_getenv(Character const* const name) throw() { __acrt_unlock(__acrt_environment_lock); } + __endtry return result; } @@ -160,6 +161,7 @@ static errno_t __cdecl common_getenv_s( { __acrt_unlock(__acrt_environment_lock); } + __endtry return status; } @@ -275,6 +277,7 @@ static errno_t __cdecl common_dupenv_s( { __acrt_unlock(__acrt_environment_lock); } + __endtry return status; } diff --git a/sdk/lib/ucrt/env/putenv.cpp b/sdk/lib/ucrt/env/putenv.cpp index 0c8a03df005..09e504eed39 100644 --- a/sdk/lib/ucrt/env/putenv.cpp +++ b/sdk/lib/ucrt/env/putenv.cpp @@ -212,6 +212,7 @@ static int __cdecl common_putenv( { __acrt_unlock(__acrt_environment_lock); } + __endtry return status; } diff --git a/sdk/lib/ucrt/heap/debug_heap.cpp b/sdk/lib/ucrt/heap/debug_heap.cpp index 0a8841e0c5e..0c3f9d0153a 100644 --- a/sdk/lib/ucrt/heap/debug_heap.cpp +++ b/sdk/lib/ucrt/heap/debug_heap.cpp @@ -241,6 +241,7 @@ static bool __cdecl is_bad_read_pointer(void const* const p, size_t const size) { return true; } + __endtry return false; } @@ -285,6 +286,7 @@ static void __cdecl validate_heap_if_required_nolock() throw() { heap_validation_pending = false; } + __endtry __acrt_check_counter = 0; } @@ -431,6 +433,7 @@ static void* __cdecl heap_alloc_dbg_internal( { __acrt_unlock(__acrt_heap_lock); } + __endtry return block; } @@ -763,6 +766,7 @@ extern "C" __declspec(noinline) void* __cdecl _realloc_dbg( { __acrt_unlock(__acrt_heap_lock); } + __endtry return new_block; } @@ -835,6 +839,7 @@ extern "C" __declspec(noinline) void* __cdecl _expand_dbg( { __acrt_unlock(__acrt_heap_lock); } + __endtry return new_block; } @@ -1035,6 +1040,7 @@ extern "C" __declspec(noinline) void __cdecl _free_dbg(void* const block, int co { __acrt_unlock(__acrt_heap_lock); } + __endtry } @@ -1070,6 +1076,7 @@ extern "C" __declspec(noinline) size_t __cdecl _msize_dbg(void* const block, int { __acrt_unlock(__acrt_heap_lock); } + __endtry return size; } @@ -1115,6 +1122,7 @@ extern "C" void __cdecl _CrtSetDbgBlockType( { __acrt_unlock(__acrt_heap_lock); } + __endtry } @@ -1301,6 +1309,7 @@ extern "C" int __cdecl _CrtCheckMemory() { __acrt_unlock(__acrt_heap_lock); } + __endtry return all_okay ? TRUE : FALSE; } @@ -1344,6 +1353,7 @@ extern "C" int __cdecl _CrtSetDbgFlag(int const new_bits) { __acrt_unlock(__acrt_heap_lock); } + __endtry return old_bits; } @@ -1374,6 +1384,7 @@ extern "C" void __cdecl _CrtDoForAllClientObjects( { __acrt_unlock(__acrt_heap_lock); } + __endtry } @@ -1469,6 +1480,7 @@ extern "C" int __cdecl _CrtIsMemoryBlock( { __acrt_unlock(__acrt_heap_lock); } + __endtry return result; } @@ -1547,6 +1559,7 @@ extern "C" void __cdecl _CrtMemCheckpoint(_CrtMemState* const state) { __acrt_unlock(__acrt_heap_lock); } + __endtry } @@ -1701,6 +1714,7 @@ extern "C" void __cdecl _CrtMemDumpAllObjectsSince(_CrtMemState const* const sta { __acrt_unlock(__acrt_heap_lock); } + __endtry _RPT0(_CRT_WARN, "Object dump complete.\n"); } diff --git a/sdk/lib/ucrt/heap/heapwalk.cpp b/sdk/lib/ucrt/heap/heapwalk.cpp index dfaf91181ed..27b6745beb7 100644 --- a/sdk/lib/ucrt/heap/heapwalk.cpp +++ b/sdk/lib/ucrt/heap/heapwalk.cpp @@ -28,6 +28,7 @@ static int __cdecl try_walk(PROCESS_HEAP_ENTRY* const win32_entry) throw() { return _HEAPBADNODE; } + __endtry } // Walks the heap, returning information on one entry at a time. If there are diff --git a/sdk/lib/ucrt/heap/new_handler.cpp b/sdk/lib/ucrt/heap/new_handler.cpp index a0866eeb914..801da85897a 100644 --- a/sdk/lib/ucrt/heap/new_handler.cpp +++ b/sdk/lib/ucrt/heap/new_handler.cpp @@ -35,6 +35,7 @@ _PNH __cdecl _set_new_handler(_PNH new_handler) { __acrt_unlock(__acrt_heap_lock); } + __endtry return result; } @@ -65,6 +66,7 @@ _PNH __cdecl _query_new_handler() { __acrt_unlock(__acrt_heap_lock); } + __endtry return result; } diff --git a/sdk/lib/ucrt/inc/internal_shared.h b/sdk/lib/ucrt/inc/internal_shared.h index a5f50f0f681..d4234e7cdf7 100644 --- a/sdk/lib/ucrt/inc/internal_shared.h +++ b/sdk/lib/ucrt/inc/internal_shared.h @@ -158,6 +158,7 @@ struct __crt_seh_guarded_call { cleanup(); } + __endtry } }; diff --git a/sdk/lib/ucrt/locale/locale_refcounting.cpp b/sdk/lib/ucrt/locale/locale_refcounting.cpp index c949b2036c7..b9731136f47 100644 --- a/sdk/lib/ucrt/locale/locale_refcounting.cpp +++ b/sdk/lib/ucrt/locale/locale_refcounting.cpp @@ -280,6 +280,7 @@ extern "C" __crt_locale_data* __cdecl __acrt_update_thread_locale_data() { __acrt_unlock(__acrt_locale_lock); } + __endtry } else { diff --git a/sdk/lib/ucrt/locale/wsetlocale.cpp b/sdk/lib/ucrt/locale/wsetlocale.cpp index da3b6d834f6..f7b309dab71 100644 --- a/sdk/lib/ucrt/locale/wsetlocale.cpp +++ b/sdk/lib/ucrt/locale/wsetlocale.cpp @@ -168,6 +168,7 @@ void __cdecl _free_locale( { __acrt_unlock(__acrt_multibyte_cp_lock); } + __endtry if (plocinfo->locinfo != nullptr) { @@ -191,6 +192,7 @@ void __cdecl _free_locale( { __acrt_unlock(__acrt_locale_lock); } + __endtry } _free_crt(plocinfo); diff --git a/sdk/lib/ucrt/lowio/dup.cpp b/sdk/lib/ucrt/lowio/dup.cpp index dc170b62f49..0bea1ae87ad 100644 --- a/sdk/lib/ucrt/lowio/dup.cpp +++ b/sdk/lib/ucrt/lowio/dup.cpp @@ -72,6 +72,7 @@ static int __cdecl _dup_nolock_internal(int const fh, __crt_cached_ptd_host& ptd __acrt_lowio_unlock_fh(new_fh); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/lowio/dup2.cpp b/sdk/lib/ucrt/lowio/dup2.cpp index 1b38c7c6b7b..453bd9b5614 100644 --- a/sdk/lib/ucrt/lowio/dup2.cpp +++ b/sdk/lib/ucrt/lowio/dup2.cpp @@ -105,6 +105,7 @@ static int __cdecl _dup2_internal(int const source_fh, int const target_fh, __cr __acrt_lowio_unlock_fh(source_fh); __acrt_lowio_unlock_fh(target_fh); } + __endtry return result; } diff --git a/sdk/lib/ucrt/lowio/filelength.cpp b/sdk/lib/ucrt/lowio/filelength.cpp index 595b8906ca8..f9b6577ac16 100644 --- a/sdk/lib/ucrt/lowio/filelength.cpp +++ b/sdk/lib/ucrt/lowio/filelength.cpp @@ -48,6 +48,7 @@ static Integer __cdecl common_filelength(int const fh) throw() { __acrt_lowio_unlock_fh(fh); } + __endtry return end; } diff --git a/sdk/lib/ucrt/lowio/ioinit.cpp b/sdk/lib/ucrt/lowio/ioinit.cpp index 5a3e858607c..452f4011339 100644 --- a/sdk/lib/ucrt/lowio/ioinit.cpp +++ b/sdk/lib/ucrt/lowio/ioinit.cpp @@ -242,6 +242,7 @@ extern "C" bool __cdecl __acrt_initialize_lowio() { __acrt_unlock(__acrt_lowio_index_lock); } + __endtry return result; } diff --git a/sdk/lib/ucrt/lowio/locking.cpp b/sdk/lib/ucrt/lowio/locking.cpp index 6f62a514da9..8f7dda92b51 100644 --- a/sdk/lib/ucrt/lowio/locking.cpp +++ b/sdk/lib/ucrt/lowio/locking.cpp @@ -116,5 +116,6 @@ extern "C" int __cdecl _locking(int const fh, int const locking_mode, long const { __acrt_lowio_unlock_fh(fh); } + __endtry return result; } diff --git a/sdk/lib/ucrt/lowio/lseek.cpp b/sdk/lib/ucrt/lowio/lseek.cpp index 35f41aec86b..e0bbd9f0967 100644 --- a/sdk/lib/ucrt/lowio/lseek.cpp +++ b/sdk/lib/ucrt/lowio/lseek.cpp @@ -123,6 +123,7 @@ static Integer __cdecl common_lseek(int const fh, Integer const offset, int cons { __acrt_lowio_unlock_fh(fh); } + __endtry return result; } diff --git a/sdk/lib/ucrt/lowio/open.cpp b/sdk/lib/ucrt/lowio/open.cpp index 941f331b91e..62e92d108c5 100644 --- a/sdk/lib/ucrt/lowio/open.cpp +++ b/sdk/lib/ucrt/lowio/open.cpp @@ -110,6 +110,7 @@ static int __cdecl common_open( __acrt_lowio_unlock_fh(fh); } } + __endtry if (error_code != 0) { @@ -182,6 +183,7 @@ static errno_t __cdecl common_sopen_dispatch( __acrt_lowio_unlock_fh(*pfh); } } + __endtry if (error_code != 0) { diff --git a/sdk/lib/ucrt/lowio/osfinfo.cpp b/sdk/lib/ucrt/lowio/osfinfo.cpp index 89a866b9de3..e7687732f71 100644 --- a/sdk/lib/ucrt/lowio/osfinfo.cpp +++ b/sdk/lib/ucrt/lowio/osfinfo.cpp @@ -93,6 +93,7 @@ extern "C" errno_t __cdecl __acrt_lowio_ensure_fh_exists(int const fh) { __acrt_unlock(__acrt_lowio_index_lock); } + __endtry return status; } @@ -183,6 +184,7 @@ extern "C" int __cdecl _alloc_osfhnd() { __acrt_unlock(__acrt_lowio_index_lock); } + __endtry return result; } @@ -328,6 +330,7 @@ extern "C" int __cdecl _open_osfhandle(intptr_t const osfhandle, int const sourc __acrt_lowio_unlock_fh(fh); } + __endtry return fh; } diff --git a/sdk/lib/ucrt/lowio/read.cpp b/sdk/lib/ucrt/lowio/read.cpp index ee5a3cf2256..d9814556876 100644 --- a/sdk/lib/ucrt/lowio/read.cpp +++ b/sdk/lib/ucrt/lowio/read.cpp @@ -399,6 +399,7 @@ extern "C" int __cdecl _read(int const fh, void* const buffer, unsigned const bu { __acrt_lowio_unlock_fh(fh); } + __endtry return result; } diff --git a/sdk/lib/ucrt/lowio/setmode.cpp b/sdk/lib/ucrt/lowio/setmode.cpp index 414a4e5a705..5c465f4bc16 100644 --- a/sdk/lib/ucrt/lowio/setmode.cpp +++ b/sdk/lib/ucrt/lowio/setmode.cpp @@ -46,6 +46,7 @@ extern "C" int __cdecl _setmode(int const fh, int const mode) { __acrt_lowio_unlock_fh(fh); } + __endtry return result; } diff --git a/sdk/lib/ucrt/lowio/write.cpp b/sdk/lib/ucrt/lowio/write.cpp index 302d6de5f97..c251a53d4a5 100644 --- a/sdk/lib/ucrt/lowio/write.cpp +++ b/sdk/lib/ucrt/lowio/write.cpp @@ -68,6 +68,7 @@ extern "C" int __cdecl _write_internal(int const fh, void const* const buffer, u { __acrt_lowio_unlock_fh(fh); } + __endtry return result; } diff --git a/sdk/lib/ucrt/mbstring/mbctype.cpp b/sdk/lib/ucrt/mbstring/mbctype.cpp index 4d2f7f1e35f..c02db844d60 100644 --- a/sdk/lib/ucrt/mbstring/mbctype.cpp +++ b/sdk/lib/ucrt/mbstring/mbctype.cpp @@ -343,6 +343,7 @@ static __crt_multibyte_data* __cdecl update_thread_multibyte_data_internal( { __acrt_unlock(__acrt_multibyte_cp_lock); } + __endtry } else { diff --git a/sdk/lib/ucrt/misc/dbgrptt.cpp b/sdk/lib/ucrt/misc/dbgrptt.cpp index f46f9488efa..ee3a23ef76a 100644 --- a/sdk/lib/ucrt/misc/dbgrptt.cpp +++ b/sdk/lib/ucrt/misc/dbgrptt.cpp @@ -380,6 +380,7 @@ int __cdecl _VCrtDbgReportA( { __acrt_unlock(__acrt_debug_lock); } + __endtry } if (handled) @@ -427,6 +428,7 @@ int __cdecl _VCrtDbgReportA( _InterlockedDecrement(&_crtAssertBusy); } } + __endtry return retval; } @@ -602,6 +604,7 @@ int __cdecl _VCrtDbgReportW { __acrt_unlock(__acrt_debug_lock); } + __endtry } if (handled) @@ -680,6 +683,7 @@ int __cdecl _VCrtDbgReportW _InterlockedDecrement(&_crtAssertBusy); } } + __endtry return retval; } diff --git a/sdk/lib/ucrt/misc/perror.cpp b/sdk/lib/ucrt/misc/perror.cpp index 319649bbba9..2fa4c2e0465 100644 --- a/sdk/lib/ucrt/misc/perror.cpp +++ b/sdk/lib/ucrt/misc/perror.cpp @@ -62,6 +62,7 @@ static void __cdecl _perror_internal(char const* const user_prefix, __crt_cached { __acrt_lowio_unlock_fh( fh ); } + __endtry } extern "C" void __cdecl perror(char const* const user_prefix) diff --git a/sdk/lib/ucrt/misc/signal.cpp b/sdk/lib/ucrt/misc/signal.cpp index 457e2b6504f..9be5453c6bc 100644 --- a/sdk/lib/ucrt/misc/signal.cpp +++ b/sdk/lib/ucrt/misc/signal.cpp @@ -195,6 +195,7 @@ static BOOL WINAPI ctrlevent_capture(DWORD const ctrl_type) throw() { __acrt_unlock(__acrt_signal_lock); } + __endtry // The default signal action leaves the event unhandled, so return false to // indicate such: @@ -333,6 +334,7 @@ extern "C" __crt_signal_handler_t __cdecl signal(int signum, __crt_signal_handle { __acrt_unlock(__acrt_signal_lock); } + __endtry if (set_console_ctrl_error) return signal_failed(signum); @@ -525,6 +527,7 @@ extern "C" int __cdecl raise(int const signum) if (action_is_global) __acrt_unlock(__acrt_signal_lock); } + __endtry if (return0) return 0; diff --git a/sdk/lib/ucrt/misc/terminate.cpp b/sdk/lib/ucrt/misc/terminate.cpp index b62854c7d64..10cd6504064 100644 --- a/sdk/lib/ucrt/misc/terminate.cpp +++ b/sdk/lib/ucrt/misc/terminate.cpp @@ -51,6 +51,7 @@ extern "C" void __cdecl terminate() throw() { ; // Deliberately do nothing } + __endtry } // If the terminate handler returned, faulted, or otherwise failed to end diff --git a/sdk/lib/ucrt/startup/exit.cpp b/sdk/lib/ucrt/startup/exit.cpp index cc680c175ff..0b3ee5ba814 100644 --- a/sdk/lib/ucrt/startup/exit.cpp +++ b/sdk/lib/ucrt/startup/exit.cpp @@ -228,6 +228,7 @@ static void __cdecl common_exit( { terminate(); } + __endtry #ifndef CRTDLL // When the CRT is statically linked, we are responsible for executing diff --git a/sdk/lib/ucrt/startup/thread.cpp b/sdk/lib/ucrt/startup/thread.cpp index 00ce20d8b21..63884e65199 100644 --- a/sdk/lib/ucrt/startup/thread.cpp +++ b/sdk/lib/ucrt/startup/thread.cpp @@ -107,6 +107,7 @@ static unsigned long WINAPI thread_start(void* const parameter) throw() // Execution should never reach here: _exit(GetExceptionCode()); } + __endtry // This return statement will never be reached. All execution paths result // in the thread or process exiting. diff --git a/sdk/lib/ucrt/stdio/clearerr.cpp b/sdk/lib/ucrt/stdio/clearerr.cpp index d3a0afcdd77..8ce8a4f05d1 100644 --- a/sdk/lib/ucrt/stdio/clearerr.cpp +++ b/sdk/lib/ucrt/stdio/clearerr.cpp @@ -25,6 +25,7 @@ extern "C" errno_t __cdecl clearerr_s(FILE* const public_stream) { _unlock_file(stream.public_stream()); } + __endtry return 0; } diff --git a/sdk/lib/ucrt/stdio/closeall.cpp b/sdk/lib/ucrt/stdio/closeall.cpp index cb537957fe2..c857169230d 100644 --- a/sdk/lib/ucrt/stdio/closeall.cpp +++ b/sdk/lib/ucrt/stdio/closeall.cpp @@ -39,6 +39,7 @@ extern "C" int __cdecl _fcloseall() { __acrt_unlock(__acrt_stdio_index_lock); } + __endtry return count; } diff --git a/sdk/lib/ucrt/stdio/fclose.cpp b/sdk/lib/ucrt/stdio/fclose.cpp index b4c120cb686..a538f4e187e 100644 --- a/sdk/lib/ucrt/stdio/fclose.cpp +++ b/sdk/lib/ucrt/stdio/fclose.cpp @@ -75,6 +75,7 @@ static int __cdecl _fclose_internal(FILE* const public_stream, __crt_cached_ptd_ { _unlock_file(stream.public_stream()); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fdopen.cpp b/sdk/lib/ucrt/stdio/fdopen.cpp index 9edbf11d316..ab5062b78b3 100644 --- a/sdk/lib/ucrt/stdio/fdopen.cpp +++ b/sdk/lib/ucrt/stdio/fdopen.cpp @@ -53,6 +53,7 @@ static FILE* __cdecl common_fdopen( { stream.unlock(); } + __endtry return stream.public_stream(); } diff --git a/sdk/lib/ucrt/stdio/fgetc.cpp b/sdk/lib/ucrt/stdio/fgetc.cpp index ba589ec13cc..d18888ac482 100644 --- a/sdk/lib/ucrt/stdio/fgetc.cpp +++ b/sdk/lib/ucrt/stdio/fgetc.cpp @@ -55,6 +55,7 @@ extern "C" int __cdecl fgetc(FILE* const public_stream) { _unlock_file(stream.public_stream()); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fgets.cpp b/sdk/lib/ucrt/stdio/fgets.cpp index 332c9df3ad7..0f7d17a7738 100644 --- a/sdk/lib/ucrt/stdio/fgets.cpp +++ b/sdk/lib/ucrt/stdio/fgets.cpp @@ -75,6 +75,7 @@ static Character* __cdecl common_fgets( { _unlock_file(stream.public_stream()); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fgetwc.cpp b/sdk/lib/ucrt/stdio/fgetwc.cpp index bdf0efa5236..715410553f4 100644 --- a/sdk/lib/ucrt/stdio/fgetwc.cpp +++ b/sdk/lib/ucrt/stdio/fgetwc.cpp @@ -110,6 +110,7 @@ extern "C" wint_t __cdecl fgetwc(FILE* const stream) { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fopen.cpp b/sdk/lib/ucrt/stdio/fopen.cpp index 2ca0eba7ba3..aace2ba7eae 100644 --- a/sdk/lib/ucrt/stdio/fopen.cpp +++ b/sdk/lib/ucrt/stdio/fopen.cpp @@ -60,6 +60,7 @@ static FILE* __cdecl common_fsopen( stream.unlock(); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fputc.cpp b/sdk/lib/ucrt/stdio/fputc.cpp index f4e3453f9be..e191289990c 100644 --- a/sdk/lib/ucrt/stdio/fputc.cpp +++ b/sdk/lib/ucrt/stdio/fputc.cpp @@ -60,6 +60,7 @@ static int __cdecl _fputc_internal(int const c, FILE* const stream, __crt_cached { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fputwc.cpp b/sdk/lib/ucrt/stdio/fputwc.cpp index d5cfc5acef7..be357b2db67 100644 --- a/sdk/lib/ucrt/stdio/fputwc.cpp +++ b/sdk/lib/ucrt/stdio/fputwc.cpp @@ -110,6 +110,7 @@ static wint_t __cdecl _fputwc_internal(wchar_t const c, FILE* const stream, __cr { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fread.cpp b/sdk/lib/ucrt/stdio/fread.cpp index 04326dbdbca..5e607ad567d 100644 --- a/sdk/lib/ucrt/stdio/fread.cpp +++ b/sdk/lib/ucrt/stdio/fread.cpp @@ -59,6 +59,7 @@ extern "C" size_t __cdecl fread_s( { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/freopen.cpp b/sdk/lib/ucrt/stdio/freopen.cpp index fe6bf2cc4c7..c8a3b26e366 100644 --- a/sdk/lib/ucrt/stdio/freopen.cpp +++ b/sdk/lib/ucrt/stdio/freopen.cpp @@ -79,6 +79,7 @@ static errno_t __cdecl common_freopen( { _unlock_file(stream.public_stream()); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/fseek.cpp b/sdk/lib/ucrt/stdio/fseek.cpp index b635bc31cfb..4a6ea859e6c 100644 --- a/sdk/lib/ucrt/stdio/fseek.cpp +++ b/sdk/lib/ucrt/stdio/fseek.cpp @@ -194,6 +194,7 @@ static int __cdecl common_fseek( { _unlock_file(stream.public_stream()); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/ftell.cpp b/sdk/lib/ucrt/stdio/ftell.cpp index b1b4321300b..e93c19c0884 100644 --- a/sdk/lib/ucrt/stdio/ftell.cpp +++ b/sdk/lib/ucrt/stdio/ftell.cpp @@ -362,6 +362,7 @@ static Integer __cdecl common_ftell(__crt_stdio_stream const stream, __crt_cache { _unlock_file(stream.public_stream()); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/gets.cpp b/sdk/lib/ucrt/stdio/gets.cpp index b42ba038803..9a2436092e0 100644 --- a/sdk/lib/ucrt/stdio/gets.cpp +++ b/sdk/lib/ucrt/stdio/gets.cpp @@ -111,6 +111,7 @@ static Character* __cdecl common_gets( { _unlock_file(stdin); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/getw.cpp b/sdk/lib/ucrt/stdio/getw.cpp index 9c2a4fcc4d4..9f79e2a705f 100644 --- a/sdk/lib/ucrt/stdio/getw.cpp +++ b/sdk/lib/ucrt/stdio/getw.cpp @@ -43,6 +43,7 @@ extern "C" int __cdecl _getw(FILE* const stream) { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/putw.cpp b/sdk/lib/ucrt/stdio/putw.cpp index 3f264c8d838..301e8e52615 100644 --- a/sdk/lib/ucrt/stdio/putw.cpp +++ b/sdk/lib/ucrt/stdio/putw.cpp @@ -38,6 +38,7 @@ extern "C" int __cdecl _putw(int const value, FILE* const stream) { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/rewind.cpp b/sdk/lib/ucrt/stdio/rewind.cpp index 11fcb4ba81c..3cb1d26f419 100644 --- a/sdk/lib/ucrt/stdio/rewind.cpp +++ b/sdk/lib/ucrt/stdio/rewind.cpp @@ -51,6 +51,7 @@ static void __cdecl _rewind_internal(FILE* const public_stream, __crt_cached_ptd { _unlock_file(stream.public_stream()); } + __endtry } extern "C" void __cdecl rewind(FILE* const public_stream) diff --git a/sdk/lib/ucrt/stdio/rmtmp.cpp b/sdk/lib/ucrt/stdio/rmtmp.cpp index 8a3a2229bbd..0aed77d32ab 100644 --- a/sdk/lib/ucrt/stdio/rmtmp.cpp +++ b/sdk/lib/ucrt/stdio/rmtmp.cpp @@ -58,12 +58,14 @@ extern "C" int __cdecl _rmtmp() { _unlock_file(stream.public_stream()); } + __endtry } } __finally { __acrt_unlock(__acrt_stdio_index_lock); } + __endtry return count; } diff --git a/sdk/lib/ucrt/stdio/stream.cpp b/sdk/lib/ucrt/stdio/stream.cpp index 69ea56a0f8e..713556fc0e8 100644 --- a/sdk/lib/ucrt/stdio/stream.cpp +++ b/sdk/lib/ucrt/stdio/stream.cpp @@ -89,6 +89,7 @@ __crt_stdio_stream __cdecl __acrt_stdio_allocate_stream() throw() { __acrt_unlock(__acrt_stdio_index_lock); } + __endtry return stream; } diff --git a/sdk/lib/ucrt/stdio/tempnam.cpp b/sdk/lib/ucrt/stdio/tempnam.cpp index dd213f16b26..f9041d46abb 100644 --- a/sdk/lib/ucrt/stdio/tempnam.cpp +++ b/sdk/lib/ucrt/stdio/tempnam.cpp @@ -172,6 +172,7 @@ static bool __cdecl compute_name( { __acrt_unlock(__acrt_tempnam_lock); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/tmpfile.cpp b/sdk/lib/ucrt/stdio/tmpfile.cpp index a58a8e48bfe..d0786c8007a 100644 --- a/sdk/lib/ucrt/stdio/tmpfile.cpp +++ b/sdk/lib/ucrt/stdio/tmpfile.cpp @@ -309,6 +309,7 @@ static errno_t common_tmpnam( { __acrt_unlock(__acrt_tempnam_lock); } + __endtry return return_value; } @@ -408,6 +409,7 @@ static errno_t __cdecl common_tmpfile_nolock(_Out_ FILE** const stream, int cons local_stream.unlock(); } + __endtry return result; } @@ -435,6 +437,7 @@ static errno_t __cdecl common_tmpfile(_Out_ FILE** const stream, int const sh_fl { __acrt_unlock(__acrt_tempnam_lock); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/ungetc.cpp b/sdk/lib/ucrt/stdio/ungetc.cpp index 211216ec119..eabd8908941 100644 --- a/sdk/lib/ucrt/stdio/ungetc.cpp +++ b/sdk/lib/ucrt/stdio/ungetc.cpp @@ -28,6 +28,7 @@ extern "C" int __cdecl ungetc(int const c, FILE* const stream) { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/stdio/ungetwc.cpp b/sdk/lib/ucrt/stdio/ungetwc.cpp index 831ac543d2c..ceef236d5a7 100644 --- a/sdk/lib/ucrt/stdio/ungetwc.cpp +++ b/sdk/lib/ucrt/stdio/ungetwc.cpp @@ -28,6 +28,7 @@ extern "C" wint_t __cdecl ungetwc(wint_t const c, FILE* const stream) { _unlock_file(stream); } + __endtry return return_value; } diff --git a/sdk/lib/ucrt/time/tzset.cpp b/sdk/lib/ucrt/time/tzset.cpp index bd3e68931dc..2e7ae9254b3 100644 --- a/sdk/lib/ucrt/time/tzset.cpp +++ b/sdk/lib/ucrt/time/tzset.cpp @@ -381,6 +381,7 @@ extern "C" void __cdecl _tzset() { __acrt_unlock(__acrt_time_lock); } + __endtry } @@ -413,6 +414,7 @@ extern "C" void __cdecl __tzset() { __acrt_unlock(__acrt_time_lock); } + __endtry } @@ -707,6 +709,7 @@ extern "C" int __cdecl _isindst(tm* const tb) { __acrt_unlock(__acrt_time_lock); } + __endtry return retval; }

2 weeks, 6 days

[reactos] 21/33: [UCRT] Fix GCC/Clang build of enum_system_locales_ex_nolock

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=7bb9f6bf288a444f0c70e… commit 7bb9f6bf288a444f0c70ed59cd393935d850b539 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Wed Oct 16 13:02:08 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Fix GCC/Clang build of enum_system_locales_ex_nolock Add static function for Clang. Add missing __stdcall in a lambda for GCC. MSVC does this automagically. See https://devblogs.microsoft.com/oldnewthing/20150220-00/?p=44623. --- sdk/lib/ucrt/internal/winapi_thunks.cpp | 22 ++++++++++++++++++++-- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/sdk/lib/ucrt/internal/winapi_thunks.cpp b/sdk/lib/ucrt/internal/winapi_thunks.cpp index 15e9d63cfa1..d0e50cfaa58 100644 --- a/sdk/lib/ucrt/internal/winapi_thunks.cpp +++ b/sdk/lib/ucrt/internal/winapi_thunks.cpp @@ -438,6 +438,14 @@ extern "C" int WINAPI __acrt_CompareStringEx( return CompareStringW(__acrt_LocaleNameToLCID(locale_name, 0), flags, string1, string1_count, string2, string2_count); } +#ifdef __clang__ +static LOCALE_ENUMPROCEX static_enum_proc; +static BOOL CALLBACK LocaleEnumProcW(LPWSTR locale_string) +{ + return __crt_fast_decode_pointer(static_enum_proc)(locale_string, 0, 0); +} +#endif + // This has been split into its own function to work around a bug in the Dev12 // C++ compiler where nested captureless lambdas are not convertible to the // required function pointer type. @@ -445,11 +453,21 @@ static BOOL enum_system_locales_ex_nolock( LOCALE_ENUMPROCEX const enum_proc ) throw() { +#ifndef __clang__ static LOCALE_ENUMPROCEX static_enum_proc; +#endif static_enum_proc = __crt_fast_encode_pointer(enum_proc); - BOOL const result = EnumSystemLocalesW( - [](LPWSTR locale_string) { return __crt_fast_decode_pointer(static_enum_proc)(locale_string, 0, 0); }, + BOOL const result = EnumSystemLocalesW((LOCALE_ENUMPROCW) +#ifdef __clang__ + LocaleEnumProcW, +#else + [](LPWSTR locale_string) + #if defined(__GNUC__) && !defined(__clang__) + __stdcall + #endif // __GNUC__ + { return __crt_fast_decode_pointer(static_enum_proc)(locale_string, 0, 0); }, +#endif LCID_INSTALLED); static_enum_proc = __crt_fast_encode_pointer(nullptr);

2 weeks, 6 days

[reactos] 20/33: [UCRT] Silence warning about returning noreturn function

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=2cc6699f0d8b3593cd4c6… commit 2cc6699f0d8b3593cd4c6703c4e38c044640a4c6 Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Mon Oct 14 11:14:34 2024 +0200 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Silence warning about returning noreturn function --- sdk/lib/ucrt/misc/invalid_parameter.cpp | 1 + sdk/lib/ucrt/startup/exit.cpp | 4 ++++ 2 files changed, 5 insertions(+) diff --git a/sdk/lib/ucrt/misc/invalid_parameter.cpp b/sdk/lib/ucrt/misc/invalid_parameter.cpp index 7865ca156db..435115b3664 100644 --- a/sdk/lib/ucrt/misc/invalid_parameter.cpp +++ b/sdk/lib/ucrt/misc/invalid_parameter.cpp @@ -276,6 +276,7 @@ extern "C" __declspec(noreturn) void __cdecl _invalid_parameter_noinfo_noreturn( EXCEPTION_NONCONTINUABLE); TerminateProcess(GetCurrentProcess(), STATUS_INVALID_CRUNTIME_PARAMETER); + UNREACHABLE; } #else // ^^^ (_M_IX86 || _M_X64) && !_UCRT_ENCLAVE_BUILD ^^^ // vvv Newer Architectures vvv // diff --git a/sdk/lib/ucrt/startup/exit.cpp b/sdk/lib/ucrt/startup/exit.cpp index e46e0395d01..cc680c175ff 100644 --- a/sdk/lib/ucrt/startup/exit.cpp +++ b/sdk/lib/ucrt/startup/exit.cpp @@ -285,21 +285,25 @@ extern "C" int __cdecl _is_c_termination_complete() extern "C" void __cdecl exit(int const return_code) { common_exit(return_code, _crt_exit_full_cleanup, _crt_exit_terminate_process); + UNREACHABLE; } extern "C" void __cdecl _exit(int const return_code) { common_exit(return_code, _crt_exit_no_cleanup, _crt_exit_terminate_process); + UNREACHABLE; } extern "C" void __cdecl _Exit(int const return_code) { common_exit(return_code, _crt_exit_no_cleanup, _crt_exit_terminate_process); + UNREACHABLE; } extern "C" void __cdecl quick_exit(int const return_code) { common_exit(return_code, _crt_exit_quick_cleanup, _crt_exit_terminate_process); + UNREACHABLE; } extern "C" void __cdecl _cexit()

2 weeks, 6 days

[reactos] 19/33: [UCRT] Add workaround for va_arg warning about type promotion

by Timo Kreuzer

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a21a206c89409c3d1418e… commit a21a206c89409c3d1418e1e69e0562f621517dcf Author: Timo Kreuzer <timo.kreuzer(a)reactos.org> AuthorDate: Mon Oct 14 00:03:54 2024 +0300 Commit: Timo Kreuzer <timo.kreuzer(a)reactos.org> CommitDate: Thu Jan 16 14:18:53 2025 +0200 [UCRT] Add workaround for va_arg warning about type promotion --- sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h b/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h index cde4af0e080..bb4d545d88c 100644 --- a/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h +++ b/sdk/lib/ucrt/inc/corecrt_internal_stdio_output.h @@ -30,16 +30,33 @@ namespace __crt_stdio_output { // reads the next argument of type T from the varargs list and updates the // va_list, just like va_arg does. The 'peek' function returns the next argument // of type T, but does not modify the va_list. +#if defined(__GNUC__) || defined(__clang__) +template<typename T> struct _va_arg_promoted_tye { using type = T; }; +template<> struct _va_arg_promoted_tye<signed char> { using type = int; }; +template<> struct _va_arg_promoted_tye<unsigned char> { using type = int; }; +template<> struct _va_arg_promoted_tye<wchar_t> { using type = int; }; +template<> struct _va_arg_promoted_tye<short int> { using type = int; }; +template<> struct _va_arg_promoted_tye<short unsigned int> { using type = int; }; +#endif + template <typename T> T read_va_arg(va_list& arglist) throw() { +#if defined(__GNUC__) || defined(__clang__) + return (T)(va_arg(arglist, typename _va_arg_promoted_tye<T>::type)); +#else return va_arg(arglist, T); +#endif } template <typename T> T peek_va_arg(va_list arglist) throw() { +#if defined(__GNUC__) || defined(__clang__) + return (T)(va_arg(arglist, typename _va_arg_promoted_tye<T>::type)); +#else return va_arg(arglist, T); +#endif }

2 weeks, 6 days

← Newer
1
...
16
17
18
19
20
21
22
...
28
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs January 2025