<div dir="ltr">On Tue, May 14, 2013 at 3:53 PM, Bengt Kleberg <span dir="ltr"><<a href="mailto:bengt.kleberg@ericsson.com" target="_blank">bengt.kleberg@ericsson.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<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">New name for the patch since I had a momentary lapse of reason with git.<br>

This should include all the current suggestions.<br>
<br>
git fetch git://<a href="http://github.com/ebengt/otp.git" target="_blank">github.com/ebengt/otp.git</a> correct_ls_file_v2<br>
<br>
<a href="https://github.com/ebengt/otp/compare/erlang:master...correct_ls_file_v2" target="_blank">https://github.com/ebengt/otp/compare/erlang:master...correct_ls_file_v2</a><br>
<a href="https://github.com/ebengt/otp/compare/erlang:master...correct_ls_file_v2.patch" target="_blank">https://github.com/ebengt/otp/compare/erlang:master...correct_ls_file_v2.patch</a><br>
<div class="im"><br></div></blockquote><div><br></div><div style>Thanks!</div><div style><br></div><div style>Your new version looks better, but I still see room for improvement.</div><div style><br></div><div style>If I try to list a non-existing file I get:</div>
<div style><br></div><div style><div>1> ls("non-existing").</div><div>no such file or directoryok</div><div>2></div></div><div style><br></div><div style>It does not look pretty. There should be a new line</div>
<div style>after the error message.</div><div style><br></div><div style>Also, attempting to list a non-existing file is not</div><div style>tested in the test suite. I no that you cannot test</div><div style>for the exact error message (it may be differently</div>
<div style>worded on different platforms), but you can at least</div><div style>make a smoke test so that the code will be covered.</div><div style><br></div><div style>Next issue:</div><div style><br></div><div style><div>
2> ls("/dev/null").</div><div>Invalid directory or file</div><div>ok</div><div>3></div><div><br></div><div style>I find the error message confusing. What are</div><div style>you trying to tell the user?</div>
<div style><br></div><div style>Why not simply print the file if it is not a</div><div style>directory?</div></div><div style><br></div><div style>/Bjorn</div><div style> </div></div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div></div>