[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.e
From: |
Theodor Thornhill |
Subject: |
bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el |
Date: |
Tue, 2 Mar 2021 22:45:43 +0100 |
> 2. mar. 2021 kl. 22:37 skrev Dmitry Gutov <dgutov@yandex.ru>:
>
> Hi Theodor,
>
>> On 02.03.2021 23:13, Theodor Thornhill via Bug reports for GNU Emacs, the
>> Swiss army knife of text editors wrote:
>> Yeah, maybe. However, without benchmarking, it is quite clear that
>> adding your option is faster than my patch, since ripgrep has to search
>> the whole minified file. I assume it short circuits, so that results
>> are delivered quicker to emacs. Maybe this bug can be closed.
>
> Could you try benchmarking both approaches?
>
Absolutely, I will see what I can come up with.
> If the performance improvement from yours is at all comparable with Juri's,
> I'm inclined to prefer that direction for reasons described in
> https://debbugs.gnu.org/cgi/bugreport.cgi?bug=44983#71.
>
Yeah, that was my initial motivation for this change as well.
> In both cases Ripgrep (or Grep) will search the whole file. The -M flag just
> affects its output.
Oh, ok!
Ill get back to you.
—
Theo
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Theodor Thornhill, 2021/03/01
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Theodor Thornhill, 2021/03/01
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Juri Linkov, 2021/03/02
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Theodor Thornhill, 2021/03/02
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Dmitry Gutov, 2021/03/02
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el,
Theodor Thornhill <=
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Theodor Thornhill, 2021/03/02
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Dmitry Gutov, 2021/03/02
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Theodor Thornhill, 2021/03/03
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Dmitry Gutov, 2021/03/03
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Theodor Thornhill, 2021/03/03
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Dmitry Gutov, 2021/03/06
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Dmitry Gutov, 2021/03/06
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Dmitry Gutov, 2021/03/06
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Theodor Thornhill, 2021/03/07
- bug#46859: 28.0.50; [PATCH]: Add option to truncate long lines in xref.el, Dmitry Gutov, 2021/03/07