[Libguestfs] Rust Bindings for Libnbd
Tage Johansson
tage.j.lists at posteo.net
Mon Jun 26 08:02:06 UTC 2023
>> +let hidden_handle_calls : NameSt.t = + NameSet.of_list + [
>> "get_debug"; "set_debug"; "set_debug_callback";
>> "clear_debug_callback" ]
>
> So, hmm. Isn't it better to track this in the API.ml file?
Perhaps, but it is only specific to Rust, so I thought it was best
placed in the Rust.ml file.
>> +(* Convert snake-case to upper camel-case. *)
>
> In this comment, an example would help. eg:
>
> (* Convert snake-case to upper camel-case. * For example: [get_debug
> => GetDebug] *)
>
> Doesn't golang have a similar function? It does:
>
> generator/Golang.ml: let camel_case name =
>
> It would be better to extract this function into common code (eg.
> into generator/utils.ml), as a separate patch.
Yes, I've made a new patch with the camel_case-function moved to
utils.ml.
I think I fixed most of your other comments as well.
Best regards,
Tage
More information about the Libguestfs
mailing list