Menu

fuse Merge Request #30: improve the teletext datastream code (merged)

Merging...

Merged

Something went wrong. Please, merge manually

Checking if merge is possible...

Something went wrong. Please, merge manually

Guesser wants to merge 2 commits from /u/guesser/fuse-emulator/ to master, 2024-07-06

This code checks that an entire field of packet data is available in the socket read buffer, to avoid desynchronization through partial reads. It also discards data from the buffer in half-second chunks if it falls more than a second behind. This prevents buffered data accumulating whenever emulation is paused due to menu interaction etc. until the server eventually blocks and disconnects us.

Commit Date  
[52da91] (ttx2000s-socket-improvements) by ZXGuesser ZXGuesser

move FIONREAD probe to compat function

2024-06-30 09:46:31 Tree
[256bd7] by ZXGuesser ZXGuesser

Protect against desynchronized packet streams, and fast forward buffer after emulation is paused for menus etc.

2024-04-06 23:25:25 Tree

Discussion

  • Sergio Baldoví

    Sergio Baldoví - 2024-06-30

    Thank you Guesser. When I try to compile on linux I get this error:

    peripherals/ttx2000s.c: In function ttx2000s_field_event:
    peripherals/ttx2000s.c:357:9: warning: implicit declaration of function ioctlsocket [-Wimplicit-function-declaration]
      357 |     if( ioctlsocket( teletext_socket, FIONREAD, &n ) == -1) {
          |         ^~~~~~~~~~~
    peripherals/ttx2000s.c:357:39: error: FIONREAD undeclared (first use in this function)
      357 |     if( ioctlsocket( teletext_socket, FIONREAD, &n ) == -1) {
          |                                       ^~~~~~~~
    

    Can you fix it?

     
  • Guesser

    Guesser - 2024-06-30

    whoops, that's the windows specific function. I'll move it to a compat function

     
  • Sergio Baldoví

    Sergio Baldoví - 2024-07-06

    Thank you. Merged in [8fd925].

     

    Related

    Commit: [8fd925]

  • Sergio Baldoví

    Sergio Baldoví - 2024-07-06
    • Status: open --> merged
     

Log in to post a comment.

MongoDB Logo MongoDB