Re: Indentation style

2019-07-28 Thread David Bremner
Thomas Schneider  writes:

> David Bremner  writes:
>
>> We're using more or less the default for Emacs (you can see the details
>> in .dir-locals.el).
>
> Maybe I misunderstood, but this doesn’t really look like Emacs’ defaults
> for C.

Right, I should have been more precise, using tabs at the beginning of
the line, with tab-width 8, followed by less than 8 spaces is the emacs
default.

>
> Yes, if you let Vim parse a per-directory .vimrc (similar to forbidding
> Emacs to parse .dir-locals.el).
>
> While we’re at it, how about a generic .editorconfig?  The world isn’t
> split into good people and dea… uh, Emacs and Vim.

Sure, if someone is willing to provide .vimrc / .editorconfig, and
support it at least till we shake the bugs out, then I'm happy to
include them.

d
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch


Re: Indentation style

2019-07-28 Thread Thomas Schneider
David Bremner  writes:

> We're using more or less the default for Emacs (you can see the details
> in .dir-locals.el).

Maybe I misunderstood, but this doesn’t really look like Emacs’ defaults
for C.

> I'm a bit dubious that we want to re-whitespace everything, but I'd
> certainly be receptive to matching configuration for other
> editors. Does anyone know if vim indentation can be configured per
> project / directory?

Yes, if you let Vim parse a per-directory .vimrc (similar to forbidding
Emacs to parse .dir-locals.el).

While we’re at it, how about a generic .editorconfig?  The world isn’t
split into good people and dea… uh, Emacs and Vim.

--qsx
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch


Re: Indentation style (was: [PATCH] notmuch-dump.c: Fix output file being closed twice)

2019-07-28 Thread David Bremner
Ralph Seichter  writes:

> * David Bremner:
>
>> I did have to revert one unrelated whitespace change.
>
> I knew I might run afoul of that the minute I read "Indent is 4 spaces
> with mixed tab/spaces and a tab width of 8" in the code style notes. ;-)
>
> In my experience, it is error-prone to mix tabs and spaces. Perhaps it
> would be useful to change to all spaces? Also, with an added editor mode
> line, supporting that style would be more convenient (for Emacs and Vim
> at least).

We're using more or less the default for Emacs (you can see the details
in .dir-locals.el).  I'm a bit dubious that we want to re-whitespace
everything, but I'd certainly be receptive to matching configuration for
other editors. Does anyone know if vim indentation can be configured per
project / directory?

d
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch


Indentation style (was: [PATCH] notmuch-dump.c: Fix output file being closed twice)

2019-07-28 Thread Ralph Seichter
* David Bremner:

> I did have to revert one unrelated whitespace change.

I knew I might run afoul of that the minute I read "Indent is 4 spaces
with mixed tab/spaces and a tab width of 8" in the code style notes. ;-)

In my experience, it is error-prone to mix tabs and spaces. Perhaps it
would be useful to change to all spaces? Also, with an added editor mode
line, supporting that style would be more convenient (for Emacs and Vim
at least).

-Ralph
___
notmuch mailing list
notmuch@notmuchmail.org
https://notmuchmail.org/mailman/listinfo/notmuch