Closed jkroonza closed 2 years ago
As a side note: this is already in production for us.
Hi, Jaco! Hah, I already created and merged a PR based on the patch you emailed me. So I'm going to close this one, but don't worry... it has been merged into master.
The value of 64 may seem reasonable, but ULS currently requires 77 and this is likely to grow in the short to medium future, instead of arbitrarily limiting allocate the memory dynamically.
Signed-off-by: Jaco Kroon jaco@uls.co.za