Menu
▾
▴
Re: [Cobertura-devel] Major 'instrumentation code' refactoring
|
From: Steven C. <ste...@re...> - 2010-10-07 16:00:02
|
This is great! I ran all my tests, and it looks like this fix is going to be a nice one. I was just wondering though if you could provide another jar without the calls to System.out. Also I was wondering is ThreadSafeRigorous set to OFF by default? Thanks again! On 10/4/2010 3:01 AM, Piotr Tabor wrote: > If you want to test the RC version from the branch, I've prepared the > ready jar for it: > http://www.mimuw.edu.pl/~ptab/Cobertura/cobertura-1.10.0-2010.10-ptab.jar > <http://www.mimuw.edu.pl/%7Eptab/Cobertura/cobertura-1.10.0-2010.10-ptab.jar>. > > Piotr Tabor > > On Sun, Oct 3, 2010 at 11:14 PM, Piotr Tabor <pi...@ta... > <mailto:pi...@ta...>> wrote: > > The last publicly available version 1.9.4.1 does not support this > options (--ignoreTrivial, --ignoreMethodAnnotation). Trunk (with > content planned of 1.9.5) and ptab_v2_0 branch (with proposition > probably for 1.10.0) > contains support for this options (description here: > https://cobertura.svn.sourceforge.net/svnroot/cobertura/trunk/cobertura/ChangeLog). > Documentation has not been updated because this changes has not > been released yet. > > End - yes: all this options are supported by ant-task. > > Regards, > Piotr Tabor > > > 210/9/29 Steven Christou <ste...@re... > <mailto:ste...@re...>> > > I was wondering if there was going to be any documentation > added to the cobertura documentation page about these new > arguments. Also I was wondering if there was any support for > these arguments through the ant task? > Thanks! > > > On 9/26/2010 4:46 PM, Piotr Tabor wrote: >> I've worked on version in the brunch. The code is AFAIK >> test-before-release ready. >> I've migrated support for --ignoreTrivial and >> --ignoreAnnotations and >> added a new switch --threadsafeRigorous. In the mode >> cobertura is less faster then without >> the switch, but the counts of hits per line are precise in >> multithreaded situations. The switch should not affect the >> coverage results (% of hit lines and branches). >> >> My spare time is going to be over, so I don't plan any other >> changes then bugfixes to the reported problems with the new >> solution. >> >> Piotr Tabor >> >> On Mon, Sep 13, 2010 at 8:14 PM, John W. Lewis >> <Joh...@sa... <mailto:Joh...@sa...>> wrote: >> >> >> I’ll take a look – hopefully soon. >> >> *From:* Piotr Tabor [mailto:pio...@gm... >> <mailto:pio...@gm...>] >> *Sent:* Sunday, September 12, 2010 3:59 AM >> *To:* cob...@li... >> <mailto:cob...@li...> >> *Subject:* [Cobertura-devel] Major 'instrumentation code' >> refactoring >> >> Hello Cobertura masters, >> >> Long time ago (March) I promised to provide a really >> faster version of Cobertura, but I started to be really >> busy then. Finally I found some time and prepared it. >> It is stored here: >> https://cobertura.svn.sourceforge.net/svnroot/cobertura/branches/ptab_v2_0 >> (should be probably 1.10) and the >> net.sourceforge.cobertura.instrument package >> is totally rewritten. >> >> The current status is: >> >> Additional benefits: >> * Speed, speed, speed >> * It is extremly close to supress requirement that >> *.sar file created during instrumentation is needed >> during runtime. In fact the file is now created >> only for backward compability. >> * SwitchFunctionalTest works now. Some simillar >> fixes to coverage in other areas. >> * I think that the new code is cleaner. >> >> Drawbacks: >> * To count the hits I'm using in this implementation >> just: int[] hits; >> and to register hit I use: hits[...]++; >> It is extremly fast, but not exactly thread safe. >> There will be no 'exception' thrown or other problems, >> but we can >> miss some hits (tests shows that it is <5% of >> hits). But it is guaranteed that if there is at least one >> hit for a single touch-point, it will be >> registered. So it does not affect coverage, and >> seams to be acceptable for 99.9% usage scenarios. >> If required an exact-thread-safe mode can be >> implemented (see AtomicArrayCodeProvider) >> (but usege of AtomicIntegerArray causes >> significant (5-10 times) performance decrease - but still >> is faster then cobertura 1.9.4+). >> >> TODO: >> * ignore and ignore trival functionality has not >> been merged yet >> * exact-thread-safe mode swotch >> * The code need more tests >> >> I would appreciate if you (John ?) could have a look at >> the code and scribe opinion about it. >> >> Regards, >> Piotr Tabor >> >> >> >> >> -- >> Pozdrawiam, >> Piotr Tabor >> >> >> ------------------------------------------------------------------------------ >> Start uncovering the many advantages of virtual appliances >> and start using them to simplify application deployment and >> accelerate your shift to cloud computing. >> http://p.sf.net/sfu/novell-sfdev2dev >> >> >> _______________________________________________ >> Cobertura-devel mailing list >> Cob...@li... <mailto:Cob...@li...> >> https://lists.sourceforge.net/lists/listinfo/cobertura-devel > Email Disclaimer: > http://www.redprairie.com/emaildisclaimer/ > > > ------------------------------------------------------------------------------ > Start uncovering the many advantages of virtual appliances > and start using them to simplify application deployment and > accelerate your shift to cloud computing. > http://p.sf.net/sfu/novell-sfdev2dev > _______________________________________________ > Cobertura-devel mailing list > Cob...@li... > <mailto:Cob...@li...> > https://lists.sourceforge.net/lists/listinfo/cobertura-devel > > > > > -- > Pozdrawiam, > Piotr Tabor > > > > > -- > Pozdrawiam, > Piotr Tabor > > > ------------------------------------------------------------------------------ > Virtualization is moving to the mainstream and overtaking non-virtualized > environment for deploying applications. Does it make network security > easier or more difficult to achieve? Read this whitepaper to separate the > two and get a better understanding. > http://p.sf.net/sfu/hp-phase2-d2d > > > _______________________________________________ > Cobertura-devel mailing list > Cob...@li... > https://lists.sourceforge.net/lists/listinfo/cobertura-devel Email Disclaimer: http://www.redprairie.com/emaildisclaimer/ |
