Skip to content

[CF] Add TARGET_OS_BSD in some places. #2590

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Dec 18, 2019

Conversation

3405691582
Copy link
Member

@3405691582 3405691582 commented Dec 17, 2019

These particular uses should hopefully be uncontroversial.

  • CFPlatform.c: to use the _CFIsMainThread pthread implementation; to
    properly implement __CFTSDSetSpecific and __CFTSDGetSpecific with the
    same pthread implementation as Linux (and Mac)

  • CoreFoundation_Prefix.h: to exclude strlcpy and issetugid which is
    normally available in the system; to include sys/param.h for the
    MIN macro

  • CFData.c: for the __CFPageSize implementation

  • CFBundle_Resources.c: to include dirent.h symbols

  • CFXMLPreferencesDomain.c: to define sleep for the platform

  • CFBurstTrie.c: for mmap.

These particular uses should hopefully be uncontroversial.

* CFPlatform.c: to use the _CFIsMainThread pthread implementation; to
  properly implement __CFTSDSetSpecific and __CFTSDGetSpecific with the
  same pthread implementation as Linux (and Mac)

* CoreFoundation_Prefix.h: to exclude strlcpy and issetugid which is
  normally available in the system; to include sys/param.h for the
  MIN macro

* CFData.c: for the __CFPageSize implementation

* CFBundle_Resources.c: to include dirent.h symbols

* CFXMLPreferencesDomain.c: to define sleep for the platform

* CFBurstTrie.c: for mmap.
@parkera
Copy link
Contributor

parkera commented Dec 18, 2019

@swift-ci test and merge

@swift-ci swift-ci merged commit 86eec61 into swiftlang:master Dec 18, 2019
@3405691582 3405691582 deleted the CF_LinuxAlsoBSD branch January 3, 2020 16:01
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.

3 participants