[Emc-commit] [LinuxCNC/linuxcnc] 5a1776: hal: a stream of depth N holds N samples

[BsAtHome <no...@gi...>]

  Branch: refs/heads/master
  Home:   https://github.com/LinuxCNC/linuxcnc
  Commit: 5a1776f832141dca919f8732ee052d315dbdb131
      https://github.com/LinuxCNC/linuxcnc/commit/5a1776f832141dca919f8732ee052d315dbdb131
  Author: Luca Toniolo <107...@us...>
  Date:   2026-10-14 (Wed, 14 Oct 2026)

  Changed paths:
    M src/hal/components/histobinstream.comp
    M src/hal/components/latencybinstream.comp
    M src/hal/hal_lib.c
    M tests/halmodule/stream/stream_reader.py
    M tests/halmodule/stream/stream_writer.py

  Log Message:
  -----------
  hal: a stream of depth N holds N samples

The ring keeps one slot free to tell full from empty, so a stream
created with depth N took only N-1 samples. Allocate N+1 slots and keep
reporting N as maxdepth. Drop the +1 that histobinstream and
latencybinstream added for it, and fill the halmodule stream test to
10. Found by Bertho Stultiens in #4674.


  Commit: 912313954ee13070c3a97075bc1434648f89351f
      https://github.com/LinuxCNC/linuxcnc/commit/912313954ee13070c3a97075bc1434648f89351f
  Author: BsAtHome <be...@va...>
  Date:   2026-10-14 (Wed, 14 Oct 2026)

  Changed paths:
    M src/hal/components/histobinstream.comp
    M src/hal/components/latencybinstream.comp
    M src/hal/hal_lib.c
    M tests/halmodule/stream/stream_reader.py
    M tests/halmodule/stream/stream_writer.py

  Log Message:
  -----------
  Merge pull request #4678 from grandixximo/hal-stream-depth

hal: a stream of depth N holds N samples


Compare: https://github.com/LinuxCNC/linuxcnc/compare/2655e62fe6aa...912313954ee1

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