On Sun, Jul 15 2012, Mark Walters <markwalters1009@gmail.com> wrote: > However, there are some problems with multiline search results (see > below) so I think we should either fix these or just downplay this new > functionality by, for example, removing the comments on newlines from > the defcustom and saying in NEWS that the feature is experimental/not > complete or similar. (NEWS could say how to enter newlines in the > defcustom) > > With this minor comment on the documentation my criticisms should not > hold up this excellent series. I agree that multi-line support needs a little bit more work before it's ready for prime time. I'll keep experimenting with it as well and see if I can uncover any other issues. But I definitely also agree that that work should *not* hold up this patch series, as it adds plenty of other benefits. > Examples of "incompleteness": these are rather more personal but it > seems odd to me to highlight one line rather than one result in the > buffer. Similarly I would expect <up> <down> to scroll up or down by > one result rather than one line. Highlighting the entire entry should definitely be fixed, since otherwise it can be hard to see which of the other lines is associated with the current entry. I also notice that the author field munging causes some weird behaviors. Certain formatter strings can cause it to break. If we can get that fixed it might be nice to have a similar functionality for the subject field, which can also be really long. jamie.