Skip to content

Conversation

@hattne
Copy link
Collaborator

@hattne hattne commented Jul 5, 2025

Inspired by Microsoft's _mktemp example.

@hattne hattne marked this pull request as ready for review July 7, 2025 01:40
@yayahjb yayahjb self-assigned this Sep 4, 2025
Copy link
Collaborator

@yayahjb yayahjb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reasonable changes

The replacement implementation somewhat arbitrarily fails with EEXIST
after 65536 attempts.  Assume mkstemp(3) is now everywhere available and
use it in favor of mktemp(3).  Inspired by Microsoft's _mktemp example
and an implementation derived from the wcecompat library.
@yayahjb yayahjb merged commit feeac19 into dials:main Sep 5, 2025
2 checks passed
yayahjb added a commit that referenced this pull request Sep 6, 2025
This reverts commit feeac19, reversing
changes made to 2cb404e.
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