• DDMsgReader: As a loadable module, now make use of user# parameter for

    From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 14:24:53
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3203

    Just curious: what does +(string) accomplish, differently than say parseInt()?

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 14:25:30
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3204

    Did you mean to leave this "Temporary" bit in there?

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 14:26:08
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3205

    What's the purpose of this new comment?

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 14:32:44
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3206

    Another temporary code block left in?

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 15:43:20
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3207

    It's just a different style of ensuring it's a number, I suppose. I saw this style in an example years ago and have been doing it this way for a while. I don't know if there's any either advantage either way.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 15:46:11
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3208

    Nope. I'll remove that.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 15:51:27
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3214

    It must have been something I forgot to remove.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Rob Swindell@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 15:56:11
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3215

    It doesn't ensure it's a number however:
    ```
    ;eval
    Parameter(s): +("blah")
    Result (number): NaN
    ```
    Just as an FYI.

    ---
    þ Synchronet þ Vertrauen þ Home of Synchronet þ [vert/cvs/bbs].synchro.net
  • From Eric Oulashin@VERT to GitLab note in main/sbbs on Thursday, February 09, 2023 16:04:45
    https://gitlab.synchro.net/main/sbbs/-/merge_requests/254#note_3219

    Yep. In this block of code, I did a regex test to make sure argv[1] is all digits, so that I'd be able to expect it could be cast to a number successfully.

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