<div dir="ltr">Eric, I'm sorry for the delay! I've looked at your patch again, and I think it looks good now. I would like you to amend your commits into one single commit since keeping them separate does not add any value imo.<div>
<br></div><div style>Also, you need to add tests for all changed code. There is an rb_SUITE in sasl/test which can be extended.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Regards</div><div class="gmail_extra">
/siri<br><br><div class="gmail_quote">2013/6/20 Fredrik <span dir="ltr"><<a href="mailto:fredrik@erlang.org" target="_blank">fredrik@erlang.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On 06/19/2013 11:23 PM, PAILLEAU Eric wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Le 19/06/2013 10:19, Siri Hansen a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Eric - I am looking at this patch, and I find the diff a bit strange.<br>
I see in the first commit that you updated the documentation (rb.xml)<br>
for e.g. rb:start:<br>
<br>
e.g.<br>
-<v>FileName = string() | standard_io</v><br>
+<v>FileName = string() | atom() | pid()</v><br>
<br>
but this is backed out again in your second commit, although is seems<br>
the code is still there (and, as far as I can understand, this<br>
functionality is also the main point of the patch). Could you please<br>
have a look at it? Did something go wrong in a merge or rebase?<br>
<br>
Regards<br>
</blockquote>
Hello Siri,<br>
yes you are true.<br>
I fixed this.<br>
Fredik can refetch.<br>
Regards<br>
</blockquote></div></div>
Re-fetched.<br>
Thanks,<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
<br>
BR Fredrik Gustafsson<br>
Erlang OTP Team<br>
<br>
</font></span></blockquote></div><br></div></div>