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

Omission in caml_copy_int64 documentation #5971

Closed
vicuna opened this Issue Apr 1, 2013 · 1 comment

Comments

Projects
None yet
1 participant
@vicuna
Copy link
Collaborator

vicuna commented Apr 1, 2013

Original bug ID: 5971
Reporter: @Leonidas-from-XIV
Assigned to: meyer
Status: closed (set by @xavierleroy on 2015-12-11T18:21:29Z)
Resolution: fixed
Priority: normal
Severity: text
Version: 4.00.1
Fixed in version: 4.00.2+dev
Category: documentation
Related to: #6126

Bug description

Hi,

Section 19.4.4 - Allocating blocks, says:

caml_copy_int32(i), copy_int64(i) and caml_copy_nativeint(i) return a value of OCaml type int32, int64 and nativeint, respectively, initialized with the integer i.

But in the int64 type, it forgets to add the "caml_" prefix. Now I know that this is still valid because there are compatibility wrappers, it would be great if someone could just change "copy_int64(i)" to "caml_copy_int64(i)" to promote the more modern versions.

regards,
Marek

@vicuna

This comment has been minimized.

Copy link
Collaborator Author

vicuna commented Apr 1, 2013

Comment author: meyer

Fix in commit 13471. Thanks for the report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.