Re: [PATCH] emacs: Avoid regexp overflow when tidying citations.

Subject: Re: [PATCH] emacs: Avoid regexp overflow when tidying citations.

Date: Tue, 16 Nov 2010 11:57:22 -0800

To: David Edmondson, notmuch@notmuchmail.org

Cc:

From: Carl Worth


On Fri, 12 Nov 2010 12:50:02 +0000, David Edmondson <dme@dme.org> wrote:
> Declare `notmuch-wash-tidy-citations-max', which is the largest region
> that `notmuch-wash-tidy-citations' will attempt to improve.

Hi David,

Could you add a test case for whatever bug is being fixed here?

I'm a little concerned about giving the user a tuning knob which appears
to have a somewhat arbitrary default, and also without providing much
guidance to the user on how to set that particular knob.

I'd feel better if the code could somehow "know" when it needs to stop
tidying well enough that we would feel fine just not offering the knob
at all.

Thanks,

-Carl
part-000.sig (application/pgp-signature)

Thread: