Re: [PATCH 3/8] tools/kvm_stat: mark private methods as such

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

 



On 13.02.2018 15:48, Paolo Bonzini wrote:
> On 05/02/2018 13:59, Stefan Raspl wrote:
>> From: Stefan Raspl <stefan.raspl@xxxxxxxxxx>
>>
>> Helps quite a bit reading the code when it's obvious when a method is
>> intended for internal use only.
>>
>> Signed-off-by: Stefan Raspl <raspl@xxxxxxxxxxxxxxxxxx>
> 
> Since no subclassing is involved, should these be _single_underscore
> methods rather than __double_underscore?

Yup - one could argue that this way it's more future proof. But the reality is
that I thought I had read that this is the way to go. Probably listened to the
wrong voices in my head...
Will fix.

Ciao,
Stefan




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux