[erlang-patches] Bug fixes for cover

José Valim jose.valim@REDACTED
Thu Jan 10 10:53:37 CET 2013


I have added ?line to keep consistent with the remaining of the test file.
Do you want me to go ahead and remove ?line from the whole *
lib/tools/test/cover_SUITE.erl* file or this is something you have already
done internally and I should remove just the ones in my patch?


*José Valim*
www.plataformatec.com.br
Skype: jv.ptec
Founder and Lead Developer


On Thu, Jan 10, 2013 at 10:45 AM, Fredrik <fredrik@REDACTED> wrote:

>  Hello,
> Your patch has been into review and the feedback you got back was that
> everything looks good but you have to remove the '?line' macros, because
> they are not used and thus has no effect anymore.
> Please fix this and give me a notice when it is done
>
> BR Fredrik Gustafsson
> Erlang OTP Team
>
> On 11/28/2012 02:11 PM, José Valim wrote:
>
> Hello,
>
>  I am sending a couple bug fixes for cover.
>
>  I have broken those fixes into three granular commits.
> The commit messages contains the rationale behind them.
>
>  One of the commits changes cover to get the source from
> "Module:module_info(compile)" if the current heuristic that
> traverses directories fails. In my opinion, we could rely
> solely on the source information and remove the heuristic
> completely but I have kept the current heuristic as the first
> mechanism in order to minimize the impact of the changes.
>
>  As such, I would appreciate if those changes could be merged
> into maint. :)
>
>  git fetch git://github.com/josevalim/otp.git cover-patches
>
>  https://github.com/josevalim/otp/compare/cover-patches
> https://github.com/josevalim/otp/compare/cover-patches.patch
>
>  Thank you,
>
>
>  *José Valim*
>  www.plataformatec.com.br
> Skype: jv.ptec
> Founder and Lead Developer
>
>
>
> _______________________________________________
> erlang-patches mailing listerlang-patches@REDACTED://erlang.org/mailman/listinfo/erlang-patches
>
>
>
> _______________________________________________
> erlang-patches mailing list
> erlang-patches@REDACTED
> http://erlang.org/mailman/listinfo/erlang-patches
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://erlang.org/pipermail/erlang-patches/attachments/20130110/344dcc57/attachment.htm>


More information about the erlang-patches mailing list