[erlang-patches] Bug fixes for cover

Fredrik fredrik@REDACTED
Thu Jan 10 11:08:40 CET 2013


Great, I'll re-fetch this.
Thanks for your contribution!

BR Fredrik Gustafsson
Erlang OTP Team
On 01/10/2013 11:04 AM, José Valim wrote:
> Anyway, here are the patches without ?line.
>
> git fetch git://github.com/josevalim/otp.git 
> <http://github.com/josevalim/otp.git> cover-patches-no-line
>
> https://github.com/josevalim/otp/compare/cover-patches-no-line
> https://github.com/josevalim/otp/compare/cover-patches-no-line.patch
>
>
> *José Valim*
> www.plataformatec.com.br <http://www.plataformatec.com.br/>
> Skype: jv.ptec
> Founder and Lead Developer
>
>
> On Thu, Jan 10, 2013 at 10:53 AM, José Valim 
> <jose.valim@REDACTED 
> <mailto:jose.valim@REDACTED>> wrote:
>
>     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 <http://www.plataformatec.com.br/>
>     Skype: jv.ptec
>     Founder and Lead Developer
>
>
>     On Thu, Jan 10, 2013 at 10:45 AM, Fredrik <fredrik@REDACTED
>     <mailto: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
>>         <http://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 <http://www.plataformatec.com.br/>
>>         Skype: jv.ptec
>>         Founder and Lead Developer
>>
>>
>>
>>         _______________________________________________
>>         erlang-patches mailing list
>>         erlang-patches@REDACTED  <mailto:erlang-patches@REDACTED>
>>         http://erlang.org/mailman/listinfo/erlang-patches
>
>
>         _______________________________________________
>         erlang-patches mailing list
>         erlang-patches@REDACTED <mailto: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/094905ca/attachment.htm>


More information about the erlang-patches mailing list