Re: [PATCH x86] [5/16] Replace hard coded reservations in x86-64 early boot code with dynamic table II


In a minor way -- it is far less important than let's say good high level
comments or clear code flow.

e.g. this patch imho is a major improvement in code clarity and logic
and makes previously quite fragile code much simpler and more approachable
and easier changeable (I can say this because I was responsible for the
original mess -- only excuse is that it has grown over time). And then when
people don't see these advantages and just talk about tabs<->spaces that is
quite annoying.

It is great to have all the obvious style issues fixed no matter how much
the code clarity and logic is improved. For the simple reason that with
obvious coding style issues fixed you do not get distracted.

And I do not understand why people has any problem with that when we talk
about patches.

What is then considered "obvious style issues" are today implemented
as errors / warnings in checkpatch for the most part. Not everyone
agrees with what checkpatch says are errors/warnings but this is at
least an more or less unambigious way to express these obvious cases.
And this is far better than the maintainers choice of the day style.

Same for tabs versus spaces. That is something no human should
need to care about. And it is something a machine can handle fine too
anyways.
Things turn red in my vi if no-one cares - and I have no magic tool
to fix it here.
No-one should submit patches with white-space errors - thank you.

Sam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Relevant Pages