janestreet / async_unix

Jane Street Capital's asynchronous execution library (unix)
MIT License
33 stars 21 forks source link

Document semantics of fsync and fdatasync #20

Closed Cjen1 closed 3 years ago

Cjen1 commented 4 years ago

With the current documentation it is unclear whether fsync t just calls fsync on the underlying file descriptor, or waits until all previous writes have occurred before calling fsync.

This PR hopefully clears that up.

I'm not 100% clear on the syntax for odoc so it is possible I haven't done it quite right.

seliopou commented 4 years ago

Thanks, I've imported the patch. This should get automatically merged after going through internal process.