I can't run the test from Readme

[foul11]

On all configurations, the test failed.

Tested on:

• win10 x64
• and lua:
  • Lua 5.1.5
  • LuaJIT 2.1.1695653777
  • Lua 5.3.6 / due to the increased stack size, the number in the test was not 40000, but 900000

local memoize = require('memoize')

local function tri(x)
  if x == 0 then return 0 end
  return x+tri(x-1)
end

-- print(tri(40000)) -- here we get a stackoverflow, as expected

local memoized_tri = mem(tri)

for i=0, 40000 do
	if i % 1000 == 0 then print(i) end
	memoized_tri(i) -- here we also get a stackoverflow, but it was expected that there wouldn’t be one
end

print(memoized_tri(40000)) -- we'll never get here

[foul11]

I forgot to mention that I tested all the commits in the repository

[CMB]

Memoizing recursive functions is a tad fiddly, because essentially,
once you've memoized a recursive function, you need the function to
call the memoized version instead of the original during the recursion
step. The README does get this wrong. Here's how that (probably)
should look. Apologies in advance if the formatting is terrible.
I'm blind and using the gh CLI on Linux, and I can't attach files to comments.
The raw file can also be found at https://paste.xogium.me/WB.lua.

I made the max value passed to tri insanely large, because I don't
think I can predict what the max stack size will be.
Here on Linux with musl and Lua 5.3.6, it seems to be 499990.

local memoize = require 'memoize'

local tri
tri = function(x)
  if x == 0 then return 0 end
  return x+tri(x-1)
end

local success, errmsg = pcall(
  function()
    print(tri(9999999)) -- stack overflow: too much recursion
    end)

if success then error('Unexpected success')
end
print(errmsg)

tri = memoize(tri) -- now tri "remembers" previous results

print('Seeding cache, this will take a while:')
for i=0, 9999999 do tri(i) end -- seed cache
print('Cache seeded.')

print(tri(9999999)) -- 49999995000000, instantaneous result