Ernie Rael wrote:

> Poking around...
> 
> In debugger.c::debuggy_find():976 it looks like there's a missing `break;`.

Hmm, it needs to loop if "lnum" can be lower in another entry, but for
profiling that doesn't matter.  I suppose we could break when
"gap == &prof_ga".  Do you want to make a PR for that?

> With my corrected code, has profiling() still looks like ~11% of the the 
> time (it's got those two vim_regexec...()); wonder if has_profiling()
> could/should cache the results of debuggy_find().

Caching can be tricky.  In this case it can be difficult to clear cached
information when anything changes (new function defined, profiling entry
added/deleted, etc.).

-- 
You were lucky to have a LAKE! There were a hundred and sixty of
us living in a small shoebox in the middle of the road.

 /// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net   \\\
///                                                                      \\\
\\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups 
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/vim_dev/20230322220232.AF6811C0932%40moolenaar.net.

Raspunde prin e-mail lui