From: Louis Gesbert <louis.gesbert@ocamlpro.com>
To: Gabriel Kerneis <kerneis@pps.jussieu.fr>
Cc: caml-list@inria.fr
Subject: Re: [Caml-list] ocp-indent beta release
Date: Fri, 8 Feb 2013 17:09:20 +0100 [thread overview]
Message-ID: <201302081709.20352.louis.gesbert@ocamlpro.com> (raw)
In-Reply-To: <20130208155520.GD11513@kerneis.info>
Le vendredi 8 février 2013 16:55:20, Gabriel Kerneis a écrit :
> Hi,
>
> On Fri, Feb 08, 2013 at 04:25:29PM +0100, Louis Gesbert wrote:
> > Some comparison with tuareg on a few big OCaml projects can be seen at
> > http://htmlpreview.github.com/?https://github.com/AltGr/ocp-indent-tests/
> > blob/master/status.html
>
> How do you evaluate which line is "correctly" indented on this page?
>
> Best,
Well, the assumption, admittedly quite wrong, is that the existing source code
is already "well" indented (ie, any change made by the indenter is wrong). That
is why we should'nt aim for a 100% score in this table.
I take the opportunity to point out that most of these projects has probably
been written using tuareg, which gives a certain bias. On the other hand, our
unit tests (of which most have been written to show tuareg's weaknesses) have an
opposite bias, and tuareg fails the vast majority of them.
--
Louis Gesbert, OCamlPro
next prev parent reply other threads:[~2013-02-08 16:09 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-02-08 15:25 Louis Gesbert
2013-02-08 15:55 ` Gabriel Kerneis
2013-02-08 16:09 ` Louis Gesbert [this message]
2013-02-08 16:02 ` Daniel Bünzli
2013-02-08 16:22 ` Louis Gesbert
2013-02-08 17:50 ` Romain Bardou
2013-02-12 11:33 ` Louis Gesbert
2013-02-08 18:10 ` Ashish Agarwal
2013-02-08 18:19 ` Ashish Agarwal
2013-02-12 10:42 ` Didier Remy
2013-02-12 11:20 ` Louis Gesbert
2013-02-14 16:18 ` Daniel Bünzli
2013-02-15 0:00 ` Daniel Bünzli
2013-02-15 11:01 ` Louis Gesbert
2013-02-15 11:30 ` Daniel Bünzli
2013-02-13 3:48 ` Ivan Gotovchits
2013-02-13 9:00 ` Louis Gesbert
2013-02-15 13:57 ` [Caml-list] ocp-indent 1.0.0 release Louis Gesbert
2013-02-16 20:34 ` Niki Yoshiuchi
2013-02-17 18:05 ` Raphael Proust
2013-02-17 18:30 ` Török Edwin
2013-02-18 9:50 ` Louis Gesbert
2013-02-17 18:37 ` David Powers
2013-02-17 23:51 ` Niki Yoshiuchi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201302081709.20352.louis.gesbert@ocamlpro.com \
--to=louis.gesbert@ocamlpro.com \
--cc=caml-list@inria.fr \
--cc=kerneis@pps.jussieu.fr \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).