akkartik/template-carousel-mobile - Change AIHGJ4BTQNEUC22KVGVL6J7QNS6HVUSJJJWBTWFMSRTKOM6H64EQC

bugfix: infinite loop inside a very narrow window

I'm not sure this can trigger everywhere (I've only been able to exercise it in Lua Carousel), but it seems like a safety net worth having against future modifications by anybody.

Created by akkartik on November 24, 2023

AIHGJ4BTQNEUC22KVGVL6J7QNS6HVUSJJJWBTWFMSRTKOM6H64EQC

Dependencies

In channels

main

Change contents

Replacement in text.lua at line 106 [4.1]

B:BD[2.1892] → [2.1892:1962]

        -- everything works if bpos == 0, but is a little inefficient

[2.1892]

[2.1962]

        if x == 0 and bpos == 0 then
          assert(false, ("Infinite loop while line-wrapping. Editor is %dpx wide; window is %dpx wide"):format(State.width, App.screen.width))
        end

Replacement in source_text.lua at line 131 [5.83723]

B:BD[3.2886] → [3.2886:2956]

        -- everything works if bpos == 0, but is a little inefficient

[3.2886]

[3.2956]

        if x == 0 and bpos == 0 then
          assert(false, ("Infinite loop while line-wrapping. Editor is %dpx wide; window is %dpx wide"):format(State.width, App.screen.width))
        end