Results 397 comments of Florian Angeletti

Note that I am planning to review this PR, but I will not have time before mid-September.

There is already a simple solution for grouping together the printer and the printed argument: the `%t` specifier, for instance ```ocaml let () pp x ppf = pp ppf x...

I have some comments left on the documentation and tests, but the implementation looks fine to me.

Thanks for the update to the documentation ! I hope that I will have the time for a last round of review this week, but overall everything looks fine to...

Another option would be to export a type abbreviation ```ocaml type 'a args = 'a Arg_list.t ``` which would be enough to use `Format.args` for disambiguation purpose without breaking the...

`Format.Args.t` is also a good and concise name, and it would be one less type name in the name space.

As far as I can see, this is missing only a Change entry before we can merge.

Merged with an additional Changes entry in 8d697505a4d8 . @zazedd , thanks again for the work and don't hesitate to tell if you want to update the Changes entry that...

Thanks for the request ! This PR does sound like a good fit for committee consideration, I would keep you updated once the committee has chosen a shepherd.

@yallop has accepted to shepherd the pull request, in other words, to summarize it to the language committee and propose a recommendation for the committee to debate.