John Bandhauer wrote:
> 
> John Bandhauer wrote:
> >
> > See Syd's JS Profiler posting:
> > news://news.mozilla.org/3978C430.6AA77AA5%40netscape.com
> > It works on windows too, but you need to explicitly #define
> > XPC_TOOLS_SUPPORT (uncomment the line in xpcprivate.h). I have
> > not messed with this for a while, so there is potential for bit
> > rot. Feel free to improve it!
> 
> Also, the bit about setting a pref is no longer accurate. The
> prefs based config was removed to make the startup dependency
> ordering happy. You have to set the output file using the
> environment var. MOZILLA_JS_PROFILER_OUTPUT. See:
> http://lxr.mozilla.org/seamonkey/search?string=MOZILLA_JS_PROFILER_OUTPUT

Just tried it: it still works on windows. The attached diff makes it easier
to compile in (don't need to edit a header)

It doesn't count the time spent opening and compiling each .js file which
seems to be a major drawback (would we speed things up by combining .js
files? Can't tell). Any idea where a good place to add a hook for that would
be?

-Dan Veditz
? Debug
Index: makefile.win
===================================================================
RCS file: /cvsroot/mozilla/js/src/xpconnect/src/makefile.win,v
retrieving revision 1.20
diff -u -r1.20 makefile.win
--- makefile.win        2000/04/18 02:34:54     1.20
+++ makefile.win        2001/01/30 16:29:44
@@ -79,6 +79,9 @@
 LCFLAGS = \
         $(LCFLAGS) \
         $(DEFINES) \
+!ifdef XPC_TOOLS_SUPPORT
+        -DXPC_TOOLS_SUPPORT \
+!endif
         $(NULL)
 
 LLIBS= $(LIBNSPR)      \

Reply via email to