Since your `split` function implements the (missing) `char_list_of_string` function in the particular case that `sep` is the empty string, it may be worth mentioning it in the `.mli` file.