Skip to content
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

Annotate more caml_stat_* functions #171

Open
wants to merge 1 commit into
base: caml_stat_strndup
Choose a base branch
from

Conversation

MisterDA
Copy link

The alloc_size attributes indicates a size in bytes and cannot be used with a parameter describing the number of wchar_t in a wide-character buffer.

@dra27 dra27 added the no-change-entry-needed Causes the check for a Changes entry to be skipped for PRs label Oct 24, 2024
CAMLextern char *caml_stat_char_array_of_utf16(const wchar_t *s, size_t size,
size_t *out_size);

/* [caml_copy_string_of_utf16(s)] returns an OCaml string containing a copy of
[s] re-encoded in UTF-8 if [caml_windows_unicode_runtime_enabled] is non-zero
or in the current code page otherwise.
*/
CAMLalloc(caml_stat_free, 1) CAMLreturns_nonnull()
Copy link
Owner

@dra27 dra27 Oct 24, 2024

Choose a reason for hiding this comment

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

This one simply shouldn't be here, right?

Copy link
Author

Choose a reason for hiding this comment

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

yes, my mistake

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-change-entry-needed Causes the check for a Changes entry to be skipped for PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants