🚀 go-pugleaf

On July 23 at 10:12, Kyle Jones wrote:

>  Mosur Mohan writes:
>   > I was able to pin down a work-around:
>   >    (setq vm-generic-header-regexp "^[^: \t\n]+[ \t]*:.*\n\\([ \t].*\n\\)*")
>   > The original regexps in both 4.42 and 5.33 (below) contain nested
>   > sub-regexp's that weren't really necesesary, 
>  
>  The \(..\) groupings are necessary and the code expects to be able
>  to use match-beginning/match-end to grab parts of matches.  The
>  above regexp will break header highlighting and the method VM
>  uses to quickly move to the first visible header.

My apologies for the error.  However, I think THIS one works, and
satisfies the groupings that VM expects:

(setq vm-generic-header-regexp
  "^\\([^: \t\n]+\\)[\t ]*:\\(.*\n\\([ \t].*\n\\)*\\)")
                                         |
                                         +----------------+
                                                          |
This differs from the released defconst in that a single "+" has been
removed after the [ \t].  This time, the regexp should really be
unaffected.  I also checked by timing the execution of looking-at on
the two regexp's, and found that removing the single + brought run
time down from about 17 seconds to under a second for a matching
string that spanned 8 lines.

-- Mohan.   {mosur_mohan@wv.mentorg.com}
            {Mosur Mohan, Mentor Graphics, Wilsonville, OR}
----------------------------------------------------------------------
"Cogito ergo sum"  ==>  "I think, therefore I add."