<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    -----BEGIN PGP SIGNED MESSAGE-----<br>
    Hash: SHA1<br>
    <br>
    On 11/07/2011 15:22, Baptiste Daroussin wrote:<br>
    <span style="white-space: pre;">> Hi,<br>
      ><br>
      > here is a small patch to use realpath(3) instead of ls -l |
      awk<br>
      ><br>
      > I think the best would be to directly use realpath instead of
      get_abspath but I<br>
      > didn't want to be too much intrusive.<br>
      ><br>
      > Before sending more patches, I see you often use:<br>
      > if [ something ]; then return, fi<br>
      ><br>
      > in general I prefer<br>
      > [ something ] && return<br>
      ><br>
      > but that's is a matter of choice, I find the second more
      readable, but before<br>
      > sending you patches you won't accept, I prefer asking, would
      you accept that<br>
      > kind of patches?</span><br>
    I will accept those kind of patches since they both make sense to me
    :)<br>
    <br>
    <span style="white-space: pre;">><br>
      > You also often use expr(1) you sh provide some native way to
      do the mathematical<br>
      > stuff, would you accept patches to use sh builting things
      when possible?</span><br>
    <br>
    That's fine! Send them over.<br>
    <span style="white-space: pre;">><br>
      > last you often to <br>
      > cmd ...<br>
      > if [ $? -eq ]; then<br>
      > return<br>
      > fi<br>
      ><br>
      > where you would simple <br>
      > if cmd; then<br>
      > ...<br>
      > fi<br>
      ><br>
      > of cmd && ...<br>
      > depending of the cases<br>
      ><br>
      > Would those one be accepted to or would that be against your
      style.</span><br>
    Again, I'm pretty style-agnostic, so feel free to send adjustments
    if they improve speed / readability.<br>
    <br>
    <span style="white-space: pre;">><br>
      > Sorry for the boring mail, but I prefer to understand your
      coding rules before<br>
      > sending useless patches.<br>
      ><br>
      > regards,<br>
      > Bapt<br>
      ><br>
      ><br>
      > _______________________________________________<br>
      > Pbi-dev mailing list<br>
      > <a class="moz-txt-link-abbreviated" href="mailto:Pbi-dev@lists.pcbsd.org">Pbi-dev@lists.pcbsd.org</a><br>
      > <a class="moz-txt-link-freetext" href="http://lists.pcbsd.org/mailman/listinfo/pbi-dev">http://lists.pcbsd.org/mailman/listinfo/pbi-dev</a></span><br>
    <br>
    I've also just commited the realpath fix, thanks!<br>
    <br>
    - -- <br>
    Kris Moore<br>
    PC-BSD Software<br>
    iXsystems<br>
    -----BEGIN PGP SIGNATURE-----<br>
    Version: GnuPG v2.0.18 (FreeBSD)<br>
    Comment: Using GnuPG with Mozilla - <a class="moz-txt-link-freetext" href="http://enigmail.mozdev.org/">http://enigmail.mozdev.org/</a><br>
    <br>
    iQEcBAEBAgAGBQJOvUySAAoJEH/cIgwwV3zXLRQH+wXGqQJz7vygmgbpAd8Vc2la<br>
    CVvnxYrdKw/MOmKCfoGutwnOAjQgX6I6/Ed4uZpPEQj5pcbvF2qg7aeEsfIYPKE6<br>
    OG/mmoapH2x6GO6TIWemwu6ifzNZEnnP4o3q58rTpXnJNBU0uJCCS67mOQod6EUy<br>
    Eeiq1k6kZuzrfir3+Nh8J6s2gLGIoSt2QfgJhe+F9gO51Z900xXa8RO4H7PUoD4B<br>
    twHn1dLj8WOYy/FSaHxPvFHwfFrT7UBZFPdHGMtfU6HceZJk7cE29ihizaU+nQba<br>
    CUW2mEkKEap8YEsZmhvxmjTu/HTadj9Cm0IyBbzs1HQRtrbgcLfCCMyFap1imlY=<br>
    =CkAx<br>
    -----END PGP SIGNATURE-----<br>
    <br>
  </body>
</html>