Skip to content

asyncrt_utils.h: include xlocale on macOS#1804

Open
barracuda156 wants to merge 1 commit intomicrosoft:masterfrom
barracuda156:darwin
Open

asyncrt_utils.h: include xlocale on macOS#1804
barracuda156 wants to merge 1 commit intomicrosoft:masterfrom
barracuda156:darwin

Conversation

@barracuda156
Copy link

Nothing seems to define HAVE_XLOCALE_H, at least on macOS, but including that header may be required for the build to work.

@barracuda156
Copy link
Author

Failure without the fix:

In file included from /opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/json.h:18,
                 from /opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/src/pch/stdafx.h:88:
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h: At global scope:
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h:317:13: error: 'locale_t' does not name a type
  317 |     typedef locale_t xplat_locale;
      |             ^~~~~~~~
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h:320:24: error: 'xplat_locale' does not name a type
  320 |     static _ASYNCRTIMP xplat_locale __cdecl c_locale();
      |                        ^~~~~~~~~~~~
/opt/local/var/macports/build/_opt_PPCSnowLeopardPorts_www_cpprestsdk/cpprestsdk/work/cpprestsdk-2.10.19/Release/include/cpprest/asyncrt_utils.h:327:5: error: 'locale_t' does not name a type
  327 |     locale_t m_prevLocale;
      |     ^~~~~~~~

@ryandesign
Copy link

The error this fixes was previously reported at #1381 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants