Standardize on indentation?

[rgrinberg]

It would be nice if you could give us your ocp-indent config to use for this project because the indentation scheme is a little non-standard. Would make contributing a lot easier.

[vincenthz]

I don't use ocp-indent, so i don't have a config. I could have a look at using a standard indentation scheme if that's more practical for everyone. Out of curiosity, what's odd in the current indentation scheme ?

[rgrinberg]

To me 4 spaces for indentation is a little odd. 2 is standard everywhere I've seen either way I don't care as long as it's easy to setup. The alignment of in is also peculiar for example in https://github.com/vincenthz/obuild/blob/master/src/main.ml line 88. Anyway your indentation is perfectly fine, just hard to replicate.

To me the standard indentation scheme is what ocp-indent does by default as it's the most common method I've seen (2 spaces, let on the same column as in if in is on its own, etc.) but I'm good with anything that can be done automatically.

This is what I'd consider as "standard" btw: https://github.com/OCamlPro/opam/blob/master/src/core/opamFile.ml

[vincenthz]

i'll see if i can play with ocp-indent to reindent the whole thing to a more standard thing.
One thing that need to stay (for me at least) if the 4 spaces, as i find 2 spaces harder to "notice".

[UnixJunkie]

Long lines should be forbidden also ...

[UnixJunkie]

I think indenting all files with M-x indent-buffer using emacs' tuareg-mode would satisfy most developpers