<div dir="ltr">On Mon, May 6, 2013 at 7:01 AM, 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">Greetings,<br>
<br>
        When I use c:ls/1 it reminds me so much of Unix "ls" that I<br>
expect c:ls("filename") to work. The resulting error surprises me every<br>
time (not the same day). Here is a patch that corrects c:ls/1 to be more<br>
like Unix "ls".<br>
<br></blockquote><div><br></div><div style>Thanks for your patch.</div><div style><br></div><div style>The original code was written a long time ago, before the</div><div style>functions in the file module returned documented error</div>
<div style>reasons and before file:format_error/1 was introduced.</div><div style><br></div><div style>Therefore, if the code is to be updated, it should be</div><div style>updated properly, by matching for enotdir and generating</div>
<div style>the error messages for other errors using file:format_error/1.</div><div style><br></div><div style>Also, a test case is needed to ensure that the code works</div><div style>and that it continues to work.</div>
<div style><br></div><div style><br></div><div style>Your current version of the patch has two issues.</div><div style><br></div><div style>First, there is the style issue. Quoting</div><div style><a href="https://github.com/erlang/otp/wiki/Submitting-patches">https://github.com/erlang/otp/wiki/Submitting-patches</a>:</div>
<div style><br></div><div style>  "Make sure your changes follow the coding and indentation style of the code surrounding your changes."</div><div style><br></div><div style>(There should not be any extra spaces inside parenthesis.)</div>
<div style><br></div><div style>Second, filelib:is_file/1 returns true for both files and directories.</div><div style>You probably want to use filelib:is_regular/1.</div><div style><br></div><div style>/Bjorn</div><div> </div>
</div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div></div>