Skip to content

Alias uuid_generate_random as uuid_generate - #12430

Open
vadimkantorov wants to merge 1 commit into
emscripten-core:mainfrom
vadimkantorov:patch-1
Open

Alias uuid_generate_random as uuid_generate#12430
vadimkantorov wants to merge 1 commit into
emscripten-core:mainfrom
vadimkantorov:patch-1

Conversation

@vadimkantorov

Copy link
Copy Markdown

Fixes #12093

@welcome

welcome Bot commented Oct 4, 2020

Copy link
Copy Markdown

Thank you for submitting a pull request! If this is your first PR, make sure to add yourself to AUTHORS.

@vadimkantorov

Copy link
Copy Markdown
Author

The failing test seems to be about atomics / harfbuzz, not related to this PR

@kripken

kripken commented Oct 5, 2020

Copy link
Copy Markdown
Member

Thanks @vadimkantorov !

Code looks good. Please add some testing in tests/uuid/test.c, and add yourself to AUTHORS.

The CI errors may have been a temporary network problem earlier. A new commit with those changes could fix it.

Base automatically changed from master to main March 8, 2021 23:49
@stale

stale Bot commented Apr 16, 2022

Copy link
Copy Markdown

This issue has been automatically marked as stale because there has been no activity in the past year. It will be closed automatically if no further activity occurs in the next 30 days. Feel free to re-open at any time if this issue is still relevant.

@sbc100

sbc100 commented Aug 1, 2025

Copy link
Copy Markdown
Collaborator

Would you have time to write a test for this so we can land it?

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.

[feature request] Support uuid_generate_random (in uuid/uuid.h)

3 participants