[erlang-patches] Implement add_report_sup_handler which calls gen_event:add_sup_handler

Fredrik fredrik@REDACTED
Tue Mar 12 15:09:49 CET 2013


On 03/01/2013 08:15 PM, Pedram Nimreezi wrote:
> Hi,
>
> This patch adds add_report_sup_handler to the error_logger module
>
> To eliminate the need to go around the error_logger api, this patch
> allows for a report_handler to be added via add_sup_handler.
>
> https://github.com/DeadZen/otp/compare/error-logger-sup_handler.patch
> https://github.com/DeadZen/otp/compare/error-logger-sup_handler
>
> git fetch git://github.com/DeadZen/otp.git error-logger-sup_handler
>
Hello again,
I've got some feedback on your patch.
*The patch needs tests and documentation
*The spec for add_report_sup_handler/1 is not sufficient, should it be 
"any()" really?
*Your commit message is to short and needs further explanation.

-- 

BR Fredrik Gustafsson
Erlang OTP Team




More information about the erlang-patches mailing list