Infinite event loop in plot

[tikkanz]

For example providing an incorrect label to plot causes a wdhandler dialog to appear. Closing the dialog causes another to appear. AFAICS it is not possible to dismiss the dialog and return to Term without force closing the full QtIDE session (e.g. using Task Manager on Windows). The following code will cause the issue (xlabel is invalid & should be xcaption).

load 'plot'
'line;xlabel year' plot i. 4 3

[cdburke]

I think we had a problem like this before, but I don't see it on my system. What is your version?

JVERSION
Engine: j903/j64avx2/linux
Beta-u: commercial/2021-10-01T12:13:10
Library: 9.03.06
Qt IDE: 1.9.4/5.12.8(5.12.8)
Platform: Linux 64

[tikkanz]

I get the same behaviour on Windows in both J9.02 and J9.03 beta:

Engine: j902/j64avx2/windows
Release-b: commercial/2021-02-18T12:07:41
Library: 9.02.08
Qt IDE: 1.9.2s/5.12.10(5.12.10)
Platform: Win 64
Installer: J902 install
InstallPath: c:/users/me/appdata/local/programs/j902
Contact: www.jsoftware.com

Engine: j903/j64avx2/windows
Beta-u: commercial/2021-10-01T12:08:24
Library: 9.03.06
Qt IDE: 1.9.4/5.15.2(5.15.2)
Platform: Win 64
Installer: J903 install
InstallPath: c:/users/me/appdata/local/programs/j903
Contact: www.jsoftware.com

[bilam]

You can avoid infinite loop and investigate the real reason if plot failure by setting wdinfo to echo, eg
wdinfo_z_ =: echo_z_
and then run to test case

[tikkanz]

Thanks yes that can help with tracking issues down. I suppose I'm thinking plot should fail gracefully if there is a problem.
Currently if the user gives an option that isn't valid (e.g. xbadoption) then plot does fail gracefully with a sensible error message.

In this case I think the issue is that xcaption (label the axis) and xlabel (label data points on the axis) are both valid options and I misremembered which one was which and tried to use xlabel to label the xaxis.
I'm guessing there needs to be a check to see that the values provided for the options given are valid before executing?

[bilam]

I can reproduce failure on jandroid. I said something about length error. I'll check.