• April commit breaks ansiview

    From Nigel Reed@1:103/705 to GitLab issue in main/sbbs on Wednesday, May 01, 2024 22:59:39
    open https://gitlab.synchro.net/main/sbbs/-/issues/752

    By using git bisect I was able to determine the following commit broke the lightbar in ansiview, but only on the first page.

    ```
    commit 8eef2dc8294a551009f2ee63b92aa14983b0b811 (HEAD, refs/bisect/bad)
    Author: Rob Swindell (on Debian Linux) <rob@synchro.net>
    Date: Thu Apr 25 23:00:46 2024 -0700
    ```

    There is no highlighted bar after this commit.

    Either that is it 'fixes' something that is now showing as broken within ansiview.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Thursday, May 02, 2024 12:38:02
    https://gitlab.synchro.net/main/sbbs/-/issues/752#note_5316

    Thank you! Good find (and the bisection report saves me a lot of time!).

    It appears to be the result of the change to `attrstr` (now called `strtoattr`) which stops parsing when reaching an invalid attribute character (but before would continue parsing, just ignoring invalid attribute characters).

    So something in `tree.js` or one its dependent scripts must be setting an invalid attribute character. :-( I'll have to dig a little more later to find out why/what.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Nigel Reed@1:103/705 to GitLab note in main/sbbs on Thursday, May 02, 2024 13:18:54
    https://gitlab.synchro.net/main/sbbs/-/issues/752#note_5317

    Full credit to Dan_C for spotting the issue. I did the laborious job of hunting down the commit.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Nigel Reed@1:103/705 to GitLab note in main/sbbs on Thursday, May 02, 2024 14:58:20
    https://gitlab.synchro.net/main/sbbs/-/issues/752#note_5318

    What sort is thing would we be looking for?
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Rob Swindell@1:103/705 to GitLab note in main/sbbs on Thursday, May 02, 2024 15:06:42
    https://gitlab.synchro.net/main/sbbs/-/issues/752#note_5319

    Assignments of strings (that contain invalid attribute characters) to `console.attribute` most likely.
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From Nigel Reed@1:103/705 to GitLab note in main/sbbs on Thursday, May 02, 2024 16:14:35
    https://gitlab.synchro.net/main/sbbs/-/issues/752#note_5320

    tree.js only includes funclib.js and that itself only includes sbbsdefs.js upon certain conditions.
    Maybe it's another of the included files in ansiview.js ?
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)
  • From echicken@1:103/705 to GitLab issue in main/sbbs on Thursday, May 02, 2024 20:25:46
    close https://gitlab.synchro.net/main/sbbs/-/issues/752
    --- SBBSecho 3.20-Linux
    * Origin: Vertrauen - [vert/cvs/bbs].synchro.net (1:103/705)