[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#895: slow processing of process output
From: |
Lars Magne Ingebrigtsen |
Subject: |
bug#895: slow processing of process output |
Date: |
Wed, 06 Jul 2011 20:50:00 +0200 |
User-agent: |
Gnus/5.110018 (No Gnus v0.18) Emacs/24.0.50 (gnu/linux) |
"Drew Adams" <drew.adams@oracle.com> writes:
> Absolutely. But there is no reason to conflate "[having] the `M-x
> grep' command be less...shell-script-ey" with doing the highlighting
> in Lisp. Why do you want less shell-script-ey?
For the very simple use cases, having "grep -nH -e " seems like a
distraction. But I usually want it more shell-script-ey than less --
that `M-x grep' adds "/dev/null" at the end has often annoyed me
greatly, because you can't then say "grep foo *.el | grep -v bar".
> That is what I did, IIUYC. The relevant bit of the compile+20.el code:
>
> ;; Remember `grep-pattern' for highlighting, if highlighting is possible.
> (cond (;; Quoted pattern (either "..." or '...')
Yeah, That's what I was thinking of. :-)
> FWIW, I think the Emacs Dev approach of using external `grep'
> highlighting is generally not bad (but there have been a few bugs that
> needed to be fixed along the way).
Yes, it's a good idea, but there are these corner cases where it doesn't
work quite well. Like when you have some (implicit) commands in the
pipeline that reacts badly to being run in a terminal, like the guy who
did a "git log" (or something?), and git then started "less", because it
thought it was in a terminal.
So to me it looks like the best long-term general solution is to bind
`process-connection-type' to nil in compile.el, and fix the highlighting
in Emacs Lisp, like your code does.
Perhaps. :-)
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog http://lars.ingebrigtsen.no/