[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [Libguestfs] [PATCH 5/5] mllib: Replace various ad hoc string_* functions with String.*

On Tuesday 06 October 2015 13:30:50 Richard W.M. Jones wrote:
> This is just a straight refactoring.  Various ad hoc string_*
> functions that appeared in Common_utils have been renamed and placed
> in the String.* namespace.  The old vs "new" functions are:
> string_prefix  -> String.is_prefix
> string_suffix  -> String.is_suffix
> string_find    -> String.find
> replace_str    -> String.replace
> string_nsplit  -> String.nsplit
> string_split   -> String.split
> string_lines_split -> String.lines_split
> string_random8 -> String.random8
> ---

As mentioned yesterday on IRC, I'm torn about this one: the functions
would automatically alias functions with the same name, whenever
available in OCaml' stdlib.  Also (but this is more personal), reading
String.foo would make me think that it is part of the String module,
and go reading its help on the documentation of the String module.

Pino Toscano

Attachment: signature.asc
Description: This is a digitally signed message part.

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]