martin-saurer / jkernel

Jupyter Notebook / J Integration
GNU General Public License v3.0
44 stars 15 forks source link

J902 does not NULL terminate strings in C interface #16

Closed tingtron closed 2 years ago

tingtron commented 3 years ago

One issue is at #235 in jinter.py

Reproduce / Test

load'plot'
plot ? 10$0

Error because adCata is extraneous part from non-null terminated string.

JKernel: Internal Error.
[Errno 2] No such file or directory: 'c:\\tools\\j902-useradCata\\temp\\plot.html'

Background

Current J maintainers don't care about breaking existing C interfaces.

martin-saurer commented 2 years ago

Fixed in latest release. Using shape to get string size.