[Mono-dev] Monocov patch

[Zoltan Varga]

                             Hey,

  As I said before, feel free to check in stuff to monocov. That
particular function
is there to deal with the sometimes strange line number info emitted to mcs.
Because of this, the function is a bit of a hack/mess.
Try checking whenever the output of that function is identical before and after
your patch for some large body of code like mcs or mscorlib. If it is, then the
loops were really redundant.

                Zoltan

On 4/4/06, Jacob Ilsø Christensen <jacobilsoe at gmail.com> wrote:
> Hi.
>
> I am currently looking a bit at the monocov code as I would like to
> make a coverage add-in for MonoDevelop. To better understand what is
> going on, I am trying to simplify things a bit. The attached patch
> removes two (seemingly) redundant for loops.
>
> Does this look good to commit? Should I even ask on this list as it
> seems monocov is more or less unmaintained? I would also like to add
> some unit tests. Anyone did some for monocov?
>
> /Jacob
>
>
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>
>
>
>