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

Jump to bottom

Unified API to grab a file descriptor in unixsupport.h #5249

Closed
vicuna opened this issue Apr 7, 2011 · 4 comments
Closed

Unified API to grab a file descriptor in unixsupport.h #5249

vicuna opened this issue Apr 7, 2011 · 4 comments
Labels
feature-wish otherlibs

Comments

@vicuna
Copy link

vicuna commented Apr 7, 2011

Original bug ID: 5249
Reporter: toots
Status: acknowledged (set by @damiendoligez on 2011-05-02T10:55:43Z)
Resolution: open
Priority: normal
Severity: feature
Version: 3.12.0
Category: otherlibs
Tags: patch
Monitored by: @ygrek

Bug description

It would be nice to have a unified API to get an integer representing a file descriptor in both win32 and unix cases.

Please consider the attached patch that (trivially) add this feature.

File attachments
@mshinwell
Copy link
Contributor

I don't think this feature is likely to be accepted in its current form (although I think external libraries may end up using Obj.magic to get access to these descriptors, so there is something to potentially improve here).

@mshinwell
Copy link
Contributor

Actually, I am mistaken, this is only a C binding change. However the lack of interest does not excite. @ygrek Do you still want this?

@gasche
Copy link
Member

gasche commented Apr 17, 2020

Note: The OCaml side of this question was discussed (and rejected) in #1990.

@ygrek
Copy link
Contributor

ygrek commented Apr 17, 2020

I am ok with current macros avaialble

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature-wish otherlibs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ygrek @gasche @mshinwell @vicuna