qemu-ppc
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 3/3] hw/ide: Include 'ide_internal.h' from current path


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v2 3/3] hw/ide: Include 'ide_internal.h' from current path
Date: Mon, 26 Feb 2024 09:00:03 +0100
User-agent: Mozilla Thunderbird

On 26/2/24 08:40, Markus Armbruster wrote:
BALATON Zoltan <balaton@eik.bme.hu> writes:

On Sun, 25 Feb 2024, Philippe Mathieu-Daudé wrote:
Rename "internal.h" as "ide_internal.h", and include

Is there a convention about using underscore or dash in file names? The headers 
Thomas added are using - as well as ahci-allwinner.c, only ahci_internal.h has 
_ (but there are others elsewhere such as pci_device.h). Maybe we should be 
consistent at least within IDE and this series is now a good opportunity for 
renaming these headers to match. But it's just a small nit, thanks for picking 
this up.

This is one of the many unnecessary inconsistencies we're inflicting on
ourselves.

We have more than 3600 file names containing '-', and more almost 2700
containing '_'.  Bizarrely, 68 of them contain both.

I strongly prefer '_' myself.

Zoltan is making a local consistency argument for '-'.

Let's use '-' here.

Fine, patch updated.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]