[m-rev.] for post-commit review: reorder the interface of io.m
Zoltan Somogyi
zoltan.somogyi at runbox.com
Wed Oct 30 12:13:51 AEDT 2019
On Wed, 30 Oct 2019 12:01:44 +1100, Peter Wang <novalazy at gmail.com> wrote:
> On Wed, 30 Oct 2019 09:10:52 +1100 (AEDT), "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
> > I am not attaching the diff, because it is a mess. Instead, I am asking
> > any volunteer reviewer to read the updated interface section of io.m.
> >
> > I am committing the change because while the new order may not be
> > perfect, it is definitely much better than what went before.
>
> It looks much better.
Thanks.
> format doesn't quite belong in the "Writing out several values" section.
> Perhaps add a "Formatted output" section before that section, even if it
> contains just the two predicates.
Agreed, but I have question: should write_many, which also uses poly_types,
go in that section as well? I can see arguments both for and against.
> The "Whole file text input predicates" section contains two predicates
> that work on binary input streams. You could remove "text" from the
> section title.
Done.
Zoltan.
More information about the reviews
mailing list