Menu

#2142 xplus4 broken samples

v3.x
closed-fixed
nobody
None
GTK3
xplus4
2025-08-02
2025-06-01
Durandal
No

TED samples are broken on xplus4, noticed on 45633 but probably started on 45283 according with the commit history.

Samples sound extremely low and distorted.
Setting the experimental SOUND_SYSTEM_FLOAT define fixes the distortion not the low volume, and normal TED sound is silent.
Attached a program with samples, press 1-8 for sample playback, program is compatible with PAL and NTSC

Video of the same program running on an NTSC Plus/4
https://youtube.com/shorts/eZLtTbUllUc

1 Attachments

Discussion

  • Querino

    Querino - 2025-06-01

    after some simple testing with older versions, i think the problem starts here:

    https://sourceforge.net/p/vice-emu/code/45283/

     
  • Durandal

    Durandal - 2025-06-01

    Tested with YapeSDL 0.80.1 (July 2024), and samples work fine there.

     
  • Querino

    Querino - 2025-06-05

    Well, the last one that sounded properly in my short test me was 3.8.45282, while 3.8.45298 sounds as it does up to today.

     
  • gpz

    gpz - 2025-06-05

    Really hard to tell what the problem could be.... the new sound logic is basically copied 1:1 from SDL-Yape for that matter (and it did fix some other problems too)

     
  • gpz

    gpz - 2025-06-05

    If someone wants to help, this is the code to stare at and find the difference to what VICE does:

    https://github.com/calmopyrin/yapesdl/blob/master/tedsound.cpp

     
  • gpz

    gpz - 2025-06-05

    Looks like i was using an older version of that code as a reference.

    Good news: that program uses "digital mode" so it should be rather easy to fix (no LFSR trickery or sth like that)

     
  • gpz

    gpz - 2025-08-02

    should be fixed in r45723 - please test

     
  • Querino

    Querino - 2025-08-02

    seems to be fine.

     
  • Durandal

    Durandal - 2025-08-02

    Yes, confirmed fixed

     
  • Durandal

    Durandal - 2025-08-02
    • status: open --> closed-fixed
     

Log in to post a comment.

MongoDB Logo MongoDB