<div dir="ltr"><div><br></div>Hi Eric <div><br></div><div>Some formality, can you please redo the branch to one commit or change the commit message</div><div>for each commit, describing the changes in each commit.</div><div class="gmail_extra">
<br></div><div class="gmail_extra">The coding style must follow what is there previously, </div><div class="gmail_extra">i.e. %% for line comments</div><div class="gmail_extra">and no space before ';' </div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Either the page should be created dynamically when the state information is available, </div><div class="gmail_extra">or a proper error message should be displayed, not a blank page.</div>
<div class="gmail_extra" style>Maybe {"Information", "Not available"}.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Also the io_lib:format("~-20.s ~p~n"... should maybe use ~tp to be able to display <br>
</div><div class="gmail_extra" style>unicode strings in the state.</div><div class="gmail_extra"><br></div><div class="gmail_extra">/Dan<br><br><div class="gmail_quote">On Mon, May 13, 2013 at 10:17 AM, Fredrik <span dir="ltr"><<a href="mailto:fredrik@erlang.org" target="_blank">fredrik@erlang.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>On 05/10/2013 08:29 AM, PAILLEAU Eric wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hello,<br>
I didn't see it before, but the indentation was messed up with tabulations.<br>
Please ask to gustehn to fetch again the last commit.<br>
Sorry for the inconvenience.<br>
Regards<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
Hello Eric,<br>
Your patch has been sent to a review process. You will get updated on<br>
the outcome of this.<br>
Thanks for your contribution!<br>
<br>
</blockquote></blockquote></div>
Re-fetched.<br>
Thanks,<div><br>
<br>
-- <br>
<br>
BR Fredrik Gustafsson<br>
Erlang OTP Team<br>
<br></div><div><div>
______________________________<u></u>_________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org" target="_blank">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/<u></u>listinfo/erlang-patches</a><br>
</div></div></blockquote></div><br></div></div>