These are chat archives for kshyatt/julia-coverage

26th
Aug 2015
David P. Sanders
@dpsanders
Aug 26 2015 11:09
Feel free to take over int.jl
But most of the uncovered lines are due to the 32 bit stuff that it not currently coverage checked
And I don't know what to do about that...
You may as well remove my name from the list for the moment
Scott P. Jones
@ScottPJones
Aug 26 2015 12:42
I think that code can be changed so that the 32/64 isn't different lines
David P. Sanders
@dpsanders
Aug 26 2015 13:32
There was a discussion about changing the code and IIUC the opinion was that we shouldn't be massaging the code to get around the fact that the coverage tools don't work correctly
Scott P. Jones
@ScottPJones
Aug 26 2015 13:59
Not just to massage for coverage tools, to make it simpler
David P. Sanders
@dpsanders
Aug 26 2015 14:17
OK then go for it! What did you have in mind?
Scott P. Jones
@ScottPJones
Aug 26 2015 14:32
Never mind, I had only looked at the beginning, where it would be pretty easy to fix the promote_rule ones, however the widemul ones would not be possible.
Scott P. Jones
@ScottPJones
Aug 26 2015 14:46
Only thing might be to put those size dependent widemul functions in two files, where one or the other is included into base.
David P. Sanders
@dpsanders
Aug 26 2015 18:34
Yep, the promotion should be easy to fix but I haven't got round to it yet. Jameson suggested splitting it into two files, which seems like a nice solution to me, but Tony was against it IIRC