yallop / ocaml-ctypes

Library for binding to C libraries using pure OCaml
MIT License
363 stars 95 forks source link

Add CArray.iteri #775

Closed Ant1-Provot closed 6 days ago

Ant1-Provot commented 6 days ago

Description

Hi !

Working with Ctypes I realized I needed a CArray.iteri method in order to iter over a carray passing the index to the argument function.

As it doesn't exist I felt it was time for me to do my first open source PR :)

Type of change

val iteri : (int -> 'a -> unit) -> 'a t -> unit

This is a new feature, non breaking that just adds a functionality.

How Has This Been Tested?

I tried to stick with testing and coding style of the already existing CArray.iterfunction:

If you believe this change is futile for the code base

No worry, I just wanted to learn how to do an opensource PR.

yallop commented 6 days ago

Thank you for the contribution, @Ant1-Provot! This looks good to me.