On Mon, Apr 14, 2025 at 04:21:18PM +0200, Jean-Marc Lasgouttes wrote:
> Le 14/04/2025 à 15:35, Scott Kostyshak a écrit :
> > 
> > JMarc, can you confirm this removal is OK? If it is in there, then I
> > get the following:
> > 
> > In file included from BufferView.cpp:82:
> > ./support/pmprof.h:121:2: error: Profiling is not usable when run-time 
> > debugging is in effect
> >    121 | #error Profiling is not usable when run-time debugging is in effect
> >        |  ^
> 
> Yes, sure. It seems you are the only one using stdlib-debug, it is good that
> you do it.

I remember it found an issue a while ago. I don't know how useful it is
but it doesn't seem to affect performance much for my use cases so I
think I'll keep it, and just disable when I want to use PMPROF.

> > By the way, that error is very informative. Thanks for having it in there.
> 
> I should catch stdlib assertions too (and lic++ too).
> 
> > There is still the following in GuiFontMetrics.cpp:
> > 
> > #define DISABLE_PMPROF
> > #include "support/pmprof.h"
> > 
> > I don't know why this doesn't trigger the error.
> 
> Because of the DISABLE above. This allows to keep the instrumentation
> (PROFILE_THIS_BLOCK) in the code, but make it inert.

Ah that is very nice.

Scott

Attachment: signature.asc
Description: PGP signature

-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
https://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to