Menu
▾
▴
[ cobertura-Patches-1255463 ] Small cleanup of SaveTimer.java
|
From: SourceForge.net <no...@so...> - 2012-08-18 07:14:05
|
Patches item #1255463, was opened at 2005-08-09 21:09 Message generated for change (Comment added) made by nobody You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=720017&aid=1255463&group_id=130558 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 3 Private: No Submitted By: James Seigel (cgul) Assigned to: Nobody/Anonymous (nobody) Summary: Small cleanup of SaveTimer.java Initial Comment: It looks like SaveTimer was changed a while back to use a shutdown hook instead of timed saves for the saving of the instrument file. I cleaned up some of the commented out code and changed the name of the class to better reflect its intent. SaveTimer.java removed Cleanup.zip has the diffs and the new file in it. Just doing some cleanup as I familiarize myself with more of the codebase. ---------------------------------------------------------------------- Comment By: Nobody/Anonymous (nobody) Date: 2012-08-18 00:14 Message: EwAMDa <a href="http://yvynttjcltwx.com/">yvynttjcltwx</a>, [url=http://ibichrnxemja.com/]ibichrnxemja[/url], [link=http://vpnrspjqlcle.com/]vpnrspjqlcle[/link], http://idswxlwolgfu.com/ ---------------------------------------------------------------------- Comment By: Nobody/Anonymous (nobody) Date: 2010-08-20 19:56 Message: uySBnI <a href="http://nlkwfjjjcijb.com/">nlkwfjjjcijb</a>, [url=http://yxrfbpccvkao.com/]yxrfbpccvkao[/url], [link=http://ysernlkszyrz.com/]ysernlkszyrz[/link], http://etrffhlovmul.com/ ---------------------------------------------------------------------- Comment By: James Seigel (cgul) Date: 2005-09-02 06:10 Message: Logged In: YES user_id=1282699 Will look into it. just getting back from holidays. ---------------------------------------------------------------------- Comment By: Grzegorz Lukasik (hauserx) Date: 2005-08-27 07:56 Message: Logged In: YES user_id=1216999 James, will you implement it? ---------------------------------------------------------------------- Comment By: James Seigel (cgul) Date: 2005-08-11 05:35 Message: Logged In: YES user_id=1282699 Sounds good. Cheers James ---------------------------------------------------------------------- Comment By: Mark Doliner (thekingant) Date: 2005-08-10 21:01 Message: Logged In: YES user_id=20979 Nope, no such document. I've been happy just using the various trackers at sourceforge. As for the SaveTimer... It DID originally have the ability to save after an interval, but I removed it at some point. I don't really have an opinion either way. On the one hand, I don't really like adding features that people haven't asked for/might not use, but on the other hand, it IS something that could be extremely useful in some circumstances. Probably better to add the ability to save every x number of seconds. ---------------------------------------------------------------------- Comment By: James Seigel (cgul) Date: 2005-08-10 08:22 Message: Logged In: YES user_id=1282699 Sounds good Grzegorz. Cheers James ---------------------------------------------------------------------- Comment By: Grzegorz Lukasik (hauserx) Date: 2005-08-10 08:18 Message: Logged In: YES user_id=1216999 There is probably no such a document. When Mark will be back maybe he will say something more. ---------------------------------------------------------------------- Comment By: James Seigel (cgul) Date: 2005-08-10 07:42 Message: Logged In: YES user_id=1282699 Sounds like an interesting approach. I will have a look. Is there a vision/todo document kicking around describing where we want to go with the development/features? Cheers James. ---------------------------------------------------------------------- Comment By: Grzegorz Lukasik (hauserx) Date: 2005-08-10 07:33 Message: Logged In: YES user_id=1216999 Maybe we can go the other way, and implement SaveTimer properly. Let's say that if property net.sourceforge.cobertura.flushtime is set, then new thread is created that flushes data periodically. It can be useful for server applications where people use hot deploy feature and server cannot be shutdowned (I know about one such case). Or reports could be generated during tests run. Just a proposition. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=720017&aid=1255463&group_id=130558 |
