GitHub: Discord (official chat):
bozhidar 2021-02-13T16:49:49.003500Z

@audiolabs If I recall correctly otherwise you'd get several ghost prompts from comint-mode.

bozhidar 2021-02-13T16:50:14.003800Z

(each newline would result in some blank prompt being displayed)


@bozhidar if I remove that code (`inf-clojure--make-single-line`), I do not get that behaviour. I haven't tried anything but clj/cljs. Notably, clj works totally fine with inferior-lisp as-is without this code. So I'm a bit confused by its existence.


I thought perhaps it was a workaround for joker, lumo or planck. I have not tested (nor do I care to)

dpsutton 2021-02-13T17:00:23.006600Z

@audiolabs if you create an issue and assign it to dpsutton i can dig a bit through these


will do

dpsutton 2021-02-13T17:01:08.007400Z



@dpsutton unable to assign, but issue is above ^

bozhidar 2021-02-13T17:18:08.008300Z

> Notably, clj works totally fine with inferior-lisp as-is without this code. So I'm a bit confused by its existence. (edited)

bozhidar 2021-02-13T17:18:39.008900Z

It's possible that something has changed, but I'm pretty sure this didn't work properly in the past.

bozhidar 2021-02-13T17:23:18.009200Z

This was the original problem this was supposed to address


It does not appear to be an issue with latest clojure, macOS 11.1 and emacs 27.1, at least.


I did quickly check in emacs 26.3 and macOS catalina, with no issues noted. No clojure <1.10 tested.


I suspect maybe there was a change to a comint-input-filter somewhere between now and 2015?

bozhidar 2021-02-13T18:09:36.011500Z

Perhaps comint handles such cases differently today. I'd be happy to remove this if it's no longer needed.


well, I have about 90 hours of using inf-clojure with that code removed under the conditions listed above 😉


I don't know how well that may translate to other environments or versions

bozhidar 2021-02-13T18:51:09.013400Z

When in doubt - delete first and check if someone complains later. 😄 This has historically worked pretty well for me.

richiardiandrea 2021-02-13T20:38:24.015300Z

Hey just answered to the above if - I like the ☝️ take - my patch was working for everything I tried except lumo