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

  • src/xpdev/gen_defs.h

    From Rob Swindell@VERT to Git commit to main/sbbs/master on Tue Jan 24 12:44:00 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/c0e857c930feb9833d40617f
    Modified Files:
    src/xpdev/gen_defs.h
    Log Message:
    Make SAFECOPY() even safer by not passing NULL destination to strncpy()

    This should resolve (new) CID 434884 and 434885.

    Also, remove the SAFECOPY_USES_SPRINTF check/block - nobody enables this.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on ChromeOS)@VERT to Git commit to main/sbbs/master on Thu Jan 26 23:30:10 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/1e3a6b84e7bc1485b2a96828
    Modified Files:
    src/xpdev/gen_defs.h
    Log Message:
    Revert "Make SAFECOPY() even safer by not passing NULL destination to strncpy()"

    This reverts commit c0e857c930feb9833d40617f00ff5d11be9f7772.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell (on ChromeOS)@VERT to Git commit to main/sbbs/master on Thu Jan 26 23:45:00 2023
    https://gitlab.synchro.net/main/sbbs/-/commit/ea99ca801e63268479f91e94
    Modified Files:
    src/xpdev/gen_defs.h
    Log Message:
    Add XSAFECOPY (extra safe copy) that won't pass NULL to strncpy()

    The previous (reverted) change to SAFECOPY would double-resolve the (src) argument and caused weird side effects.

    Again, removes SAFECOPY_USES_SNPRINTF since it was unused.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Tue Sep 30 10:03:32 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/59a695825a13d6898fb804a0
    Modified Files:
    src/xpdev/gen_defs.h
    Log Message:
    Don't rely on locale for isspace()

    With truncsp() and SKIP_WHITESPACE() dominating runtime for INI
    parsing now, the locale lookups needed for every isspace() call
    have become significant. Since we likely don't actually want isspace()
    to use the local, expand the C locale whitespace table directly.

    This provides a 20% performance boost to INI parsing when using
    the FastParse calls (much smaller using other parsing functions)
    and is likely to improve various other whitespace-heavy things.

    ---
    ï¿­ Synchronet ï¿­ Vertrauen ï¿­ Home of Synchronet ï¿­ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Thu Oct 9 16:02:28 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/a4a69d20760f5f5608658fe5
    Modified Files:
    src/xpdev/gen_defs.h
    Log Message:
    inttypes.h and stdint.h are part of C99 and C++17

    So we can assume the exist if we're using a C99 or newer compiler.
    stdint.h is in C++11 (though inttypes.h isn't)

    This has been the most painful bit of xpdev for a while now.

    ---
    ï¿­ Synchronet ï¿­ Vertrauen ï¿­ Home of Synchronet ï¿­ [vert/cvs/bbs].synchro.net
  • From Deucе@VERT to Git commit to main/sbbs/master on Fri Oct 10 12:45:56 2025
    https://gitlab.synchro.net/main/sbbs/-/commit/a136e0d08cd0da404a519b01
    Modified Files:
    src/xpdev/gen_defs.h
    Log Message:
    Both inttypes.h and stdint.h were in C++11

    ---
    ï¿­ Synchronet ï¿­ Vertrauen ï¿­ Home of Synchronet ï¿­ [vert/cvs/bbs].synchro.net