[cmake-developers] Support for coverage.py coverage on the topic stage
Patrick Reynolds
patrick.reynolds at kitware.com
Tue Oct 1 09:44:16 EDT 2013
There is definitely a use case in which we want to aggregate coverage between C++, Python, and Javascript in a web application.
By the way, sorry for the warning on the nightlies. That's fixed in next.
--
Patrick Reynolds
Technical Leader
Kitware, Inc.
919 869 8848
On Monday, September 30, 2013 at 2:44 PM, Bill Hoffman wrote:
> On 9/30/2013 11:24 AM, Rolf Eike Beer wrote:
> >
> > For all other handlers it may happen that more than one coverage db is
> > found in the build tree and they would both be used. This is really
> > useful, e.g. if you want to have coverage for both C and Python code.
> > This will not work with the BullsEye handler involved. I think it would
> > be best if that would be refactored to have the same return values as
> > the other handlers. The file_count isn't really needed as file_count in
> > cmCTestCoverageHandler::ProcessHandler(), at least not in the first
> > half. It's just a marker "has anything been processed at all" there, so
> > it doesn't matter if the bullseye method returns 1 or the correct file
> > count there.
> >
>
>
> OK, I took a look at the code and the problem is in how ProcessHandler
> works. For the other handlers that keep track of lines covered and not
> branches, it does some processing at the end. That processing makes no
> sense for the branch based coverage of Bullseye. So, when Bullseye
> works it does not do any of the other summary stuff at the bottom nor do
> we want it too. It returns in that case. I had not considered the
> possibility of a project with more than one coverage thing going on at
> the same time. Certainly it would not make sense to do two different
> c++ coverages at the same time. However, you might want python or some
> other language.
>
> -Bill
>
>
> --
>
> Powered by www.kitware.com (http://www.kitware.com)
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/cgi-bin/mailman/listinfo/cmake-developers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/cmake-developers/attachments/20131001/308a90ec/attachment.html>
More information about the cmake-developers
mailing list