You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@avalon.apache.org by le...@apache.org on 2002/03/05 17:36:00 UTC
cvs commit: jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/altprofile/profiler DefaultProfilerManager.java ProfilableProxy.java
leif 02/03/05 08:36:00
Modified: src/scratchpad/org/apache/avalon/excalibur/altprofile/profiler
DefaultProfilerManager.java ProfilableProxy.java
Log:
Non configured profiler elements should not be saved in the state file.
Revision Changes Path
1.5 +7 -2 jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/altprofile/profiler/DefaultProfilerManager.java
Index: DefaultProfilerManager.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/altprofile/profiler/DefaultProfilerManager.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- DefaultProfilerManager.java 5 Mar 2002 13:04:11 -0000 1.4
+++ DefaultProfilerManager.java 5 Mar 2002 16:35:59 -0000 1.5
@@ -36,7 +36,7 @@
/**
*
* @author <a href="mailto:leif@silveregg.co.jp">Leif Mortenson</a>
- * @version CVS $Revision: 1.4 $ $Date: 2002/03/05 13:04:11 $
+ * @version CVS $Revision: 1.5 $ $Date: 2002/03/05 16:35:59 $
* @since 4.1
*/
public class DefaultProfilerManager
@@ -400,7 +400,12 @@
for ( int i = 0; i < profilableProxies.length; i++ )
{
- state.addChild( profilableProxies[i].saveState() );
+ // Only save configured profilables as they are the only ones
+ // that will contain profile samples.
+ if ( profilableProxies[i].isConfigured() )
+ {
+ state.addChild( profilableProxies[i].saveState() );
+ }
}
return state;
1.4 +7 -2 jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/altprofile/profiler/ProfilableProxy.java
Index: ProfilableProxy.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-excalibur/src/scratchpad/org/apache/avalon/excalibur/altprofile/profiler/ProfilableProxy.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- ProfilableProxy.java 5 Mar 2002 12:34:32 -0000 1.3
+++ ProfilableProxy.java 5 Mar 2002 16:35:59 -0000 1.4
@@ -22,7 +22,7 @@
* Not Synchronized.
*
* @author <a href="mailto:leif@silveregg.co.jp">Leif Mortenson</a>
- * @version CVS $Revision: 1.3 $ $Date: 2002/03/05 12:34:32 $
+ * @version CVS $Revision: 1.4 $ $Date: 2002/03/05 16:35:59 $
* @since 4.1
*/
class ProfilableProxy
@@ -318,7 +318,12 @@
ProfilePointProxy[] proxies = getProfilePointProxies();
for ( int i = 0; i < proxies.length; i++ )
{
- state.addChild( proxies[i].saveState() );
+ // Only save configured profilables as they are the only ones
+ // that will contain profile samples.
+ if ( proxies[i].isConfigured() )
+ {
+ state.addChild( proxies[i].saveState() );
+ }
}
return state;
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>