[erlang-patches] Fix bug in cth_log_redirect:terminate/2

Fredrik fredrik@REDACTED
Mon Sep 16 16:39:31 CEST 2013


On 09/16/2013 11:13 AM, Roberto Aloi wrote:
> Hi all,
>
> The gen_event callback module expects terminate/2, not terminate/1.
> Try to run a ct suite using ct:run_test/1 enabling tracing for error_logger and you should get something like:
>
> (<0.1206.0>) call error_logger:delete_report_handler(cth_log_redirect)
> (<0.1206.0>) returned from error_logger:delete_report_handler/1 ->  {'EXIT',
>                                                                      {undef,
>                                                                       [{cth_log_redirect,
>                                                                         terminate,
>                                                                         [[],
>                                                                          tc_log_async],
>                                                                         []},
>                                                                        {gen_event,
>                                                                         do_terminate,
>                                                                         7,
>                                                                         [{file,
>                                                                           "gen_event.erl"},
>                                                                          {line,
>                                                                           636}]},
>                                                                        {gen_event,
>                                                                         server_delete_handler,
>                                                                         4,
>                                                                         [{file,
>                                                                           "gen_event.erl"},
>                                                                          {line,
>                                                                           431}]},
>                                                                        {gen_event,
>                                                                         handle_msg,
>                                                                         5,
>                                                                         [{file,
>                                                                           "gen_event.erl"},
>                                                                          {line,
>                                                                           270}]},
>                                                                        {proc_lib,
>                                                                         init_p_do_apply,
>                                                                         3,
>                                                                         [{file,
>                                                                           "proc_lib.erl"},
>                                                                          {line,
>                                                                           227}]}]}}
>
> To fetch the patch:
>
> git fetch git://github.com/robertoaloi/otp.git cth_fix_terminate
>
> https://github.com/robertoaloi/otp/compare/erlang:maint...cth_fix_terminate
> https://github.com/robertoaloi/otp/compare/erlang:maint...cth_fix_terminate.patch
>
> Kind regards,
>
> Roberto Aloi
> ---
> Erlang Solutions Ltd.
> www.erlang-solutions.com
> _______________________________________________
> erlang-patches mailing list
> erlang-patches@REDACTED
> http://erlang.org/mailman/listinfo/erlang-patches
Hello Roberto,
I've fetched your patch and assigned it to be reviewed by responsible 
developer.
Thanks,

-- 

BR Fredrik Gustafsson
Erlang OTP Team




More information about the erlang-patches mailing list