Re: [PATCH 2/3] lib: s/free-page-reporting/free_page_reporting/

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

 



On Wed, Oct 14, 2020 at 05:29:33PM +0200, Michal Privoznik wrote:
> In fee8a61d29 a new attribute to <memballoon/> was introduced:
> free-page-reporting. We don't really like hyphens in attribute
> names. Use underscores instead.

Disclaimer: I just noticed this by accident, I'm sure I've missed several other
cases in the past half a year at least, nevertheless, I'll comment here.

Back when we added support for SEV I remember being asked to rename the
underscored name to a camel case one as apparently at that time we were trying
to standardize on it and so I did change it. Back to the present, we're still
not consistent with this, so I'd like to propose we either let this in
 or we switch to camelcase and update the code-style page with a new paragraph
under the naming convention dealing with XML attributes regardless.

Regards,
Erik




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux