[MLton-commit] [MLton/mlton] f64d01: Fix bug in `ImperativeIO.inputAll` failing to upda...

['Matthew F. v. MLton-c. <mlt...@ml...>]

  Branch: refs/heads/master
  Home:   https://github.com/MLton/mlton
  Commit: f64d01ff602aed1cddb2e74961bbd24771a4317a
      https://github.com/MLton/mlton/commit/f64d01ff602aed1cddb2e74961bbd24771a4317a
  Author: Matthew Fluet <mat...@gm...>
  Date:   2026-09-18 (Fri, 18 Sep 2026)

  Changed paths:
    M CHANGELOG.adoc
    M basis-library/io/imperative-io.fun
    A regression/textio.4.ok
    A regression/textio.4.sml

  Log Message:
  -----------
  Fix bug in `ImperativeIO.inputAll` failing to update buf first pos

`inputAll` drains the current buffer, but failed update the `first`
position.  This leads to incorrectly answering `false` to
`endOfStream`.

Closes MLton/mlton#649


  Commit: 720e316195ff465af25821b142292489fe76eac7
      https://github.com/MLton/mlton/commit/720e316195ff465af25821b142292489fe76eac7
  Author: Matthew Fluet <Mat...@gm...>
  Date:   2026-09-18 (Fri, 18 Sep 2026)

  Changed paths:
    M CHANGELOG.adoc
    M basis-library/io/imperative-io.fun
    A regression/textio.4.ok
    A regression/textio.4.sml

  Log Message:
  -----------
  Merge pull request #655 from MatthewFluet/input-all-endofstream-bug

Fix bug in `ImperativeIO.inputAll` failing to update buf first pos

Closes #649


Compare: https://github.com/MLton/mlton/compare/585259a93e30...720e316195ff

To unsubscribe from these emails, change your notification settings at https://github.com/MLton/mlton/settings/notifications

To unsubscribe from this group and stop receiving emails from it, send an email to mlt...@ml....