On Sat, Apr 11, 2026 at 10:03:47AM +0800, Qian Yun wrote:
> I'm combining all interp files into one and use ECL to
> bytecode compile it, and it finds several errors:
>
> 1. "systemError" takes 1 argument, but in several places
> it is called without argument: "systemError()"
Other places pass empty list as an argument. It does not look
entirely right, but did not cause a problem yet.
> 2. "throwKeyedMsg" in definition of "upLoopIters" is called
> with 1 argument instead of 2.
The second argument is a list which may be empty, so this
should be
throwKeyedMsg('"Malformed iterator", [])
Eventually 'throwKeyedMsg' should be changed to 'throw_msg', but
currently 'throw_msg' needs indentifier string which we do not
have here. So, changes beyond adding missing argument should wait
for later cleanup.
> 3. "bcNotReady" in definition of "bcSeriesByFormulaGen"
> is called with 0 argument instead of 1.
AFAICS bcSeriesByFormulaGen and bcNotReady should be removed.
bcSeriesByFormulaGen is unused and bcNotReady has only one
user, that is bcSeriesByFormulaGen.
It is probably better if you do such changes as I would have
to redo your work to make sure that I did not miss some
"systemError()".
--
Waldek Hebisch