Hi,
As consistent formatting not always translates into readable formatting
(although in many/most cases it does) I’m not in favour of forced automatic
formatting.
And that is where great freedom comes with great responsibility. I’m already
pleased if we can have a consistent style _per file_
EditorConfig has been in use with Wireshark for seven years at this
point and it's still hit or miss to configure the indentation correctly
for C.
It's not a good tool to manage inconsistent styles. That's not what it
was designed for. The only advantage EditorConfig has over modelines in
tha
Always difficult to comment on such things as I'm a hard-line centrist, but
I would be in favour of enforcing both indents and style\formatting.
If we had enforced style, we wouldn't be having this discussion, nor would
we need to even think about it when writing code, it would just happen
automag
The policy that exists for new files is a mere recommendation to use 4
space indentation, and no one is enforcing it, nor really paying much
attention to it.
On 01/03/22 19:00, Roland Knall wrote:
Policy always was and has been, that we try to achieve consistent guidelines
for new files and i
Policy always was and has been, that we try to achieve consistent guidelines
for new files and in general the guidelines for each file should be reflecting
that files style.
Although I do appreciate applying consistent styles, I acknowledge the fact
that we have a really old code base in some
On 01/03/22 17:45, David Perry wrote:
Hi all,
Bottom line up front: how much do people care about the formatting of
Wireshark's source code?
I would like to have indentation harmonized (and enforced consistently)
across the entire C code base. Preferably 4-space.
Don't care so much about oth
Hi all,
Bottom line up front: how much do people care about the formatting of
Wireshark's source code?
Background: I'm looking into [#17253][1]. It's chiefly about removing
editor modelines from the footer of each source file in favour of just
using `.editorconfig` files. But by extension it
