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

Re: [Xen-devel] A few EFI code questions



>>> On 05.12.14 at 10:33, <Ian.Campbell@xxxxxxxxxx> wrote:
> On Fri, 2014-12-05 at 07:37 +0000, Jan Beulich wrote:
>> >>> On 04.12.14 at 22:22, <roy.franz@xxxxxxxxxx> wrote:
>> > On Thu, Dec 4, 2014 at 1:35 AM, Jan Beulich <JBeulich@xxxxxxxx> wrote:
>> >>>>> On 03.12.14 at 22:02, <daniel.kiper@xxxxxxxxxx> wrote:
>> >>> 3) Should not we change xen/arch/*/efi/efi-boot.h to
>> >>>    xen/arch/*/efi/efi-boot.c? efi-boot.h contains more
>> >>>    code than definitions, declarations and short static
>> >>>    functions. So, I think that it is more regular *.c file
>> >>>    than header file.
>> >>
>> >> That's a matter of taste - I'd probably have made it .c too, but
>> >> didn't mind it being .h as done by Roy (presumably on the basis
>> >> that #include directives are preferred to have .h files as their
>> >> operands). The only thing I regret is that I didn't ask for the
>> >> pointless efi- prefix to be dropped.
>> > 
>> > I don't mind a change here, and I agree that it is more like a .c file
>> > than a .h.  If a name change is done, is it worth dropping the "efi-" at
>> > the same time?
>> 
>> If we indeed want to change the name (post 4.5), making both
>> adjustments at once would be kind of a requirement of mine.
> 
> Random thought: *.inc for .c files which happen to be embedded into
> another using #include?

That may conflict with certain editors' language detection, as .inc
may have other meanings (in the x86 Windows world I'd expect this
to be an assembler include file for example).

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.