On 25/10/2022 16.28, Amarjargal Gundjalam wrote:
The TABs should be replaced with spaces, to make sure that we have a
consistent coding style with an indentation of 4 spaces everywhere.
Resolves: https://gitlab.com/qemu-project/qemu/-/issues/370
Reviewed-by: Daniel P. Berrangé <berra...@redhat.com>
Signed-off-by: Amarjargal Gundjalam <amarjarga...@gmail.com>
---
hw/audio/fmopl.c | 1664 ++++++++++++++++++-------------------
hw/audio/fmopl.h | 138 +--
hw/audio/intel-hda-defs.h | 990 +++++++++++-----------
hw/audio/wm8750.c | 270 +++---
4 files changed, 1531 insertions(+), 1531 deletions(-)
You're changes with regards to TAB clean up look fine to me here, so for
this patch:
Reviewed-by: Thomas Huth <th...@redhat.com>
... but when I looked through the fmopl.c part, it really looks like this
file is completely wrong with regards to the QEMU coding style. I wonder
whether we should rather use a tool like "astyle" or "indent" to get it into
proper shape? ... or do we rather want to keep it in its original style in
case somebody still wants to try to port patches from the original sources
(MAME)? In that latter case, we should maybe also keep the TABs here? Gerd,
what do you think?
Thomas
