<div dir="ltr">The patch looks fine and it would be a good thing to do.<div><br></div><div style>Unfortunately, there is the pesky question of backward</div><div style>compatibility.</div><div style><br></div><div style>Someone can open a file on an R15 system, send the</div>
<div style>file descriptor (=a pid) to an R16B01 system with your</div><div style>patch and the R16B01 system will attempt to do file</div><div style>operations on the file.</div><div style><br></div><div style>The backward compatible way to do this is to first introduce</div>
<div style>a file_io_server module that can handle both the new</div><div style>and the old messages. Two releases later the file module</div><div style>can be changed to use the new message format.</div><div style><br></div>
<div style>Strictly speaking, two releases forward means R19</div><div style>to allow the original R16B release to communicate</div><div style>with an R18 release.</div><div style><br></div><div style>/Bjorn<br></div><div style>
<br></div><div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Apr 5, 2013 at 12:56 PM, Anthony Ramine <span dir="ltr"><<a href="mailto:n.oxyde@gmail.com" target="_blank">n.oxyde@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<br>
This patch adds the monitor references to the file io protocol to optimize the receive expressions in the 'file' module.<br>
<br>
        git fetch <a href="https://github.com/nox/otp.git" target="_blank">https://github.com/nox/otp.git</a> file-receive-optim<br>
<br>
        <a href="https://github.com/nox/otp/compare/erlang:maint...file-receive-optim" target="_blank">https://github.com/nox/otp/compare/erlang:maint...file-receive-optim</a><br>
        <a href="https://github.com/nox/otp/compare/erlang:maint...file-receive-optim.patch" target="_blank">https://github.com/nox/otp/compare/erlang:maint...file-receive-optim.patch</a><br>
<br>
Regards,<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Anthony Ramine<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>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div>