1. Forum
  2. DOVE-Net
  3. Synchronet Programming

  • src/conio/sdl_con.c

    From Deuc¿@VERT to Git commit to main/sbbs/master on Sun Dec 25 21:45:27 2022
    https://gitlab.synchro.net/main/sbbs/-/commit/a194a15736de98517f48a9b3
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Various SDL fixes:

    Pass window size in SDL_USEREVENT_SETVIDMODE
    - Fixes race condition where SDL_WINDOWEVENT_RESIZED or
    SDL_WINDOWEVENT_SIZE_CHANGED arraves while
    SDL_USEREVENT_SETVIDMODE is in the queue

    Set new scaling based on old scaling multiplier
    - Will resize window on mode changes, but tries to keep integer
    multiplier similar. May break fullscreen modes, dunno.

    Have aspect_fix() return higher rather than lower
    - This may make it the same as aspect_correct(), but I'm too lazy
    to figure that out.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Mon Dec 26 00:38:10 2022
    https://gitlab.synchro.net/main/sbbs/-/commit/2fa700696c34fb298fd24c7b
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Now that SDL_USEREVENT_SETVIDMODE takes a size, we can clean up sdl_setwinsize() and eliminate another race.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Fri Feb 10 19:12:31 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/f47c60792c13c8bd9c4e67c4
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Fix a couple issues found by Coverity

    1) Unreachable return without a lock in sdl_useR_func_ret() (harmless)
    2) LOR in handling SDL_USEREVENT
    - vstatlock was obtained inside of win_mutex which should never
    be done... this could (and likely does) result in a deadlock
    when using SDL mode.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Thu Apr 20 15:12:48 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/6369925c868a8a9485e6627f
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Use a local vstat copy instead of cvstat to avoid holding the lock
    for an extended time. Should fix SDL mode slowness.

    Only call RenderPresent() if we called RenderCopy(). Should fix
    SDL mode flickering.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Thu Apr 20 16:00:27 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/f06029a3889734491909ebe9
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Add TODO comment for cvstat.

    When originally created, it was intended to be accessed only by a
    single thread, and therefore not need mutex protection, allowing
    less code to run with vstatlock held. Unfortunately, this is not
    how it turned out, so cvstat should either be removed, or it should
    actually be cleaned up to work as intended.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Fri Apr 21 10:26:55 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/fdce00a456a33dc858067aef
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Push cvstat into sdl_video_event_thread() stack

    This makes cvstat work as intended, so we no longer need to lock
    vstatlock when accessing it since it's only accessed from the
    video event thread.

    There's still an irritating dependency on vstat for ALT-Enter handling.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Fri Apr 21 10:26:55 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/22c8b09f824496b7c68e599e
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Get vstat out of sdl_add_key() path

    This should almost completely remove vstatlock contention in SyncTERM,
    which should bring SDL output performance close to X11 performance.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Tue Apr 25 09:41:15 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/832871707fc3f2d37ff1891d
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    On a window resize event, only set the window size if it's different

    Fixes issue on Linux where the window size is constantly updated,
    and window rabidly steals focus.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Wed Apr 26 10:39:20 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/defc638c9f1cc95ca234a780
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Check SDL window size when checking if size changed.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Apr 29 10:42:01 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/47977472de4bfce66f1bd4d1
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Don't treat numpad /*-+ differently based on NumLock in SDL

    This caused double-keys on those when numlock was off. Only the
    number keys and period need special NumLock handling.

    Fixes bug 108 on SourceForge, thanks nelgin!

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Thu May 4 07:21:50 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/9137e82c9afa91f0c0647c70
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Fix warnings reported by Digitalman

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Debian Linux)@VERT to Git commit to main/sbbs/master on Fri May 5 21:29:01 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/875c77ea0bc774be2c2a4f67
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Resolve GCC warnings: cast from pointer to integer of different size

    Pretty sure this is what intptr_t was created for.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sun May 14 20:26:56 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/4af4effb3148c0c695e56ad1
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Fix deadlock in SDL mode.

    sdl_getscaling() does not require vstatlock. Move it out.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Thu Jun 1 13:58:12 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/978d4d0b7b64a39a30092e57
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Use cast instead of pointer math to fix Win32 build

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Jun 3 18:14:04 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/268893cc9b77f7e96343bb11
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    When switching from fullscreen mode, recalculate win size.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Feb 17 10:36:15 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/7c44187868e59355f598180a
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    For SDL mode, ensure rect matches vstat as well.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sun Mar 17 12:29:59 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/960cee2703cd4650f2805d4f
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Do not attempt to clean up SDL unless it was initialized.

    Should fix js test suite on darwin.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sun Mar 17 14:30:07 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/d281862b290f8ee309892e63
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Call exit() from exit_sdl_con if ciolib_reaper is enabled and SDL is not initialized.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sun Mar 17 14:31:06 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/ea900928153cef278cf404bc
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Revert last change reaper is explicitly disabled here.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Mon Sep 23 02:25:05 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/02dce6574620f4ce357e8f3f
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Update fullscreen and mode on Darwin to match window

    This should normalize everything, with the possible exception of
    "zoom" (the thing other OSs call maximized). Ideally, we wouldn't
    allow window size changes when "zoomed" if we could detect the state,
    but it doesn't look like we can.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Mon Sep 23 09:08:42 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/5d04a76d7d03ec4a64934d05
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Fix build.

    Whoops.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Thu Oct 10 20:34:24 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/f8044342788577da85ca26a1
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Store window size on size change event.

    Allows arbitrary sized windows on Darwin... the OS is still scaling
    the window contents though rather than allowing us to dynamically
    update them.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Fri Oct 18 19:18:55 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/a5438fc8bc465938a29d0af3
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Basically a revert of 02dce657

    This appears to break the mouse position stuff. :(

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Oct 19 08:10:30 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/b493e7b3ea4d4d9e2d351403
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Query current fullscreen state on ALT-Enter

    Because things get weird sometimes.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Oct 19 08:10:30 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/ae97eae2e8b3fc61d1fbddb1
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Spell SDL_WINDOW_FULLSCREEN correctly.

    Also, there's SDL_WINDOW_FULLSCREEN_DESKTOP, so or that in even
    though we don't need to due to how it's defined.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Oct 19 11:17:33 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/f6f20a635e8a693c511fe43d
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Add call to update_cvstat() in setup_surfaces()

    This should ensure that cvstat has the right window size at the start.
    I hope.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sun Nov 3 11:14:25 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/099cfb35ff70d4074329280d
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Fix LOR

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sun Nov 3 11:14:25 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/20165689e200d526fb30269b
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Add locking around win access

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on Windows 11)@VERT to Git commit to main/sbbs/master on Fri Nov 22 18:05:08 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/eef585abf35368fd3168a877
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Deuce said I remove this error message

    It was always displayed (sent to stderr) when running scfg, echocfg, uedit, etc. in auto-curses mode, so suppress that noise.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Nov 23 16:57:28 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/76ebd193e01d11374440871f
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Scale SDL_GetDisplayUsableBounds() like SDL_GetWindowSize()

    For retina lies.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Nov 23 17:02:05 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/16bf43e5b425f472d1fbdab2
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Try this the other way...

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sat Nov 23 17:08:07 2024
    https://gitlab.synchro.net/main/sbbs/-/commit/b38127ee8ce07bb3172c45ca
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Remove unused variable

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net