<div dir="ltr"><div style><font face="arial, sans-serif">Following changes have been made.. </font></div><div style><font face="arial, sans-serif">error_logger_SUITE tests added.</font></div><div style><font face="arial, sans-serif">error_logger module specs updated.</font></div>

<div style><font face="arial, sans-serif">better explanation added to commit message.</font></div><div style><br></div><div style>Branch rebased and updated. Thanks.</div><div class="gmail_extra"><br><br><div class="gmail_quote">

On Tue, Mar 12, 2013 at 11:03 AM, Pedram Nimreezi <span dir="ltr"><<a href="mailto:mc@majorcomputing.com" target="_blank">mc@majorcomputing.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Thank you for the update, I'll make the proper changes.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Mar 12, 2013 at 10:09 AM, Fredrik <<a href="mailto:fredrik@erlang.org">fredrik@erlang.org</a>> wrote:<br>
> On 03/01/2013 08:15 PM, Pedram Nimreezi wrote:<br>
>><br>
>> Hi,<br>
>><br>
>> This patch adds add_report_sup_handler to the error_logger module<br>
>><br>
>> To eliminate the need to go around the error_logger api, this patch<br>
>> allows for a report_handler to be added via add_sup_handler.<br>
>><br>
>> <a href="https://github.com/DeadZen/otp/compare/error-logger-sup_handler.patch" target="_blank">https://github.com/DeadZen/otp/compare/error-logger-sup_handler.patch</a><br>
>> <a href="https://github.com/DeadZen/otp/compare/error-logger-sup_handler" target="_blank">https://github.com/DeadZen/otp/compare/error-logger-sup_handler</a><br>
>><br>
>> git fetch git://<a href="http://github.com/DeadZen/otp.git" target="_blank">github.com/DeadZen/otp.git</a> error-logger-sup_handler<br>
>><br>
> Hello again,<br>
> I've got some feedback on your patch.<br>
> *The patch needs tests and documentation<br>
> *The spec for add_report_sup_handler/1 is not sufficient, should it be<br>
> "any()" really?<br>
> *Your commit message is to short and needs further explanation.<br>
><br>
><br>
> --<br>
><br>
> BR Fredrik Gustafsson<br>
> Erlang OTP Team<br>
><br>
> _______________________________________________<br>
> erlang-patches mailing list<br>
> <a href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
> <a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
<br>
<br>
<br>
</div></div><div class="HOEnZb"><div class="h5">--<br>
/* Sincerely<br>
--------------------------------------------------------------<br>
Pedram Nimreezi - Chief Technology Officer  */<br>
<br>
// The hardest part of design … is keeping features out. - Donald Norman<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>/* Sincerely<br>-------------------------------------------------------------- <br>Pedram Nimreezi - Chief Technology Officer  */<div><br></div><div>

// The hardest part of design … is keeping features out. - Donald Norman</div><div><br></div><div><br></div>
</div></div>