iainctduncan / scheme-for-pd

Pure Data port of the Scheme For Max external, providing an embedded S7 Scheme/Lisp interpreter for scripting and live-coding Pure Data in Scheme Lisp.
BSD 3-Clause "New" or "Revised" License
64 stars 5 forks source link

load-from-path with strings causes errors if used without parens #19

Open iainctduncan opened 2 years ago

iainctduncan commented 2 years ago

using load-from-path is working fine on windows when in parens, but trying to use it with a quoted filename without parens is doing all kinds of weird crap. A better solution would be to implement read as I have done in s4m so one can do

|read file-no-quotes.scm|

iainctduncan commented 2 years ago

hmm, seems like the reader was just borked. but still, the read command would be better

Lucarda commented 2 years ago

using load-from-path is working fine on windows when in parens, but trying to use it with a quoted filename without parens is doing all kinds of weird crap

This is also happening on Linux.

iainctduncan commented 2 years ago

good to know, I'll make it a priority. I've had this happen on s4m too, where it's possible to bork the reader by not having input close properly, but have not been able to consistently replicate the bug.

iainctduncan commented 2 years ago

It would be good to know if it only happens to you from the no-parens version.

Lucarda commented 2 years ago

Yep, that's correct. Only happening to "no-parens version" both on Linux and Windows (both tested only on 64bit arch).

Lucarda commented 2 years ago

Something that can help for later debugging:

Only on Linux:

On Windows it does not behave the same. It always gives errors when clicking [load-from-path "help-1.scm"(

iainctduncan commented 2 years ago

going to rename this ticket as I would still like to figure out what is going wrong, but have now added a read function, so using load-from-path in patchers is no longer necessary. If anyone gets these errors using the read message, please let me know, but as it doesn't use strings, I can't make the errors so far.

iainctduncan commented 2 years ago

there is now a read message that seems to work reliably.