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

Bigarray doc mention deprecated function Data_bigarray_val(v) #6126

Closed
vicuna opened this issue Aug 7, 2013 · 3 comments
Closed

Bigarray doc mention deprecated function Data_bigarray_val(v) #6126

vicuna opened this issue Aug 7, 2013 · 3 comments
Assignees

Comments

@vicuna
Copy link

vicuna commented Aug 7, 2013

Original bug ID: 6126
Reporter: vbmithr
Assigned to: @gasche
Status: closed (set by @xavierleroy on 2015-12-11T18:23:46Z)
Resolution: fixed
Priority: normal
Severity: minor
Version: 4.00.1
Category: documentation
Related to: #8395 #5971
Monitored by: @gasche

Bug description

In section 29.2.2 Accessing an OCaml bigarray from C or Fortran, there is a mention of the function Data_bigarray_val(v).

This function is not in bigarray.h, but in compat.h (or whatever it is really), and is probably deprecated.

The manual should indicate Cam_ba_array_val(v) instead.

File attachments

@vicuna
Copy link
Author

vicuna commented Aug 7, 2013

Comment author: @gasche

Attached is a patch that fixes the occurrences of the old names I found in this chapter. A preview is available there: http://gallium.inria.fr/~scherer/tmp/manual/htmlman/libbigarray.html

vbmithr, could you confirm that the new code examples compile fine?

@vicuna
Copy link
Author

vicuna commented Aug 8, 2013

Comment author: vbmithr

Yeah, all the code compile fine. Thanks.

@vicuna
Copy link
Author

vicuna commented Aug 8, 2013

Comment author: @gasche

I committed the patch, thanks. This should be fixed in the manual released for the next OCaml version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants