[m-rev.] for review: io.stream_db.m

Zoltan Somogyi zoltan.somogyi at runbox.com
Fri Mar 11 18:30:51 AEDT 2022


2022-03-11 18:05 GMT+11:00 "Julien Fischer" <jfischer at opturion.com>:
> Ok, I'm still inclined to place it in the submodule however.

That is what I committed.

> Related to this: why is there a declaration for the type stream_id/0
> in the publicly documented interface of io.m?  There's nothing in that
> part of the interface that would allows users to do anything with a
> stream_id value.

Agreed. The attached diff, most of which addresses one of Peter's
earlier reviews, also deletes this redundant type declaration.

Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.review
Type: application/octet-stream
Size: 3881 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20220311/7347b1e1/attachment.obj>


More information about the reviews mailing list