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

  • src/conio/sdl_con.c

    From Deuc¿@VERT to Git commit to main/sbbs/master on Sunday, December 25, 2022 21:45:27
    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 Monday, December 26, 2022 00:38:10
    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 Friday, February 10, 2023 19:12:31
    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 Thursday, April 20, 2023 15:12:48
    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 Thursday, April 20, 2023 16:00:27
    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 Friday, April 21, 2023 10:26:55
    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 Friday, April 21, 2023 10:26:55
    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 Tuesday, April 25, 2023 09:41:15
    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 Wednesday, April 26, 2023 10:39:20
    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 Saturday, April 29, 2023 10:42:01
    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 Thursday, May 04, 2023 07:21:50
    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 Friday, May 05, 2023 21:29:01
    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 Sunday, May 14, 2023 20:26:56
    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 Thursday, June 01, 2023 13:58:12
    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 Saturday, June 03, 2023 18:14:04
    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 Saturday, February 17, 2024 10:36:15
    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 Sunday, March 17, 2024 12:29:59
    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 Sunday, March 17, 2024 14:30:07
    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 Sunday, March 17, 2024 14:31:06
    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 Monday, September 23, 2024 02:25:05
    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 Monday, September 23, 2024 09:08:42
    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 Thursday, October 10, 2024 20:34:24
    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 Friday, October 18, 2024 19:18:55
    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 Saturday, October 19, 2024 08:10:30
    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 Saturday, October 19, 2024 08:10:30
    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 Saturday, October 19, 2024 11:17:33
    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 Sunday, November 03, 2024 11:14:25
    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 Sunday, November 03, 2024 11:14:25
    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 Friday, November 22, 2024 18:05:08
    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 Saturday, November 23, 2024 16:57:28
    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 Saturday, November 23, 2024 17:02:05
    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 Saturday, November 23, 2024 17:08:07
    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
  • From Deuc¿@VERT to Git commit to main/sbbs/master on Sunday, November 24, 2024 11:02:24
    https://gitlab.synchro.net/main/sbbs/-/commit/446c6785d406604b39c646f6
    Modified Files:
    src/conio/sdl_con.c
    Log Message:
    Fix up win_lock mutex usage

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