<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
On 04/09/2013 02:27 PM, Pedram Nimreezi wrote:
<blockquote
cite="mid:CAAaFgHs0zKDTjOY_NAiy9WwOg1RFipjm9BYqYL=7-1POODmyvA@mail.gmail.com"
type="cite">
<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 moz-do-not-send="true"
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
moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
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 moz-do-not-send="true"
href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
> <a moz-do-not-send="true"
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>
</blockquote>
Great,<br>
Re-fetched and assigned it for review again.<br>
<br>
<pre class="moz-signature" cols="72">--
BR Fredrik Gustafsson
Erlang OTP Team</pre>
</body>
</html>