Florian Angeletti
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.