Please test the pylint command in the ekr-hang5 branch

19 views
Skip to first unread message

Edward K. Ream

unread,
Apr 9, 2022, 5:14:43 PM4/9/22
to leo-editor
This branch contains PR #2592, that is, PR 3 for issue #2586.

The new code seems like the last best hope of having background processing work consistently without hanging Leo.

The (supposed) Aha is this: we can wait until after the process completes to call proc.communicate.  I don't see any reason why there would be any more chance of jamming a pipe by waiting, but we shall see.

Edward

tbp1...@gmail.com

unread,
Apr 9, 2022, 6:30:05 PM4/9/22
to leo-editor
I just tried this branch, and get the same result - no output.  Here's the problem. In check_process, we have (with my added comments_:

    if self.pid:   # tbp: Will never be non-None if the pipe fills up ...
        # The process has completed. Wait for the output!
        outs, errs = self.pid.communicate()  # tbp: ... so we will will never get here.


As before, when I manually killed the pylint task with task manager, I got some output from pylint, but not nearly all of it.
Reply all
Reply to author
Forward
0 new messages