Re: [patch] SELinux: trivial, unify iterator variable naming

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

 



Hello,

On Sun, 20 Jul 2008, Paul Moore wrote:

> Normally I like janitorial fixes such as whitespace cleanups, spelling
> corrections, etc.  However, these three patches seem a bit silly to me
> simply changing a variable name from "idx" to "i" and not adding any
> real value to the code.  If it were up to me alone, I would say "NO",
> but if everyone else agrees they are worthwhile I won't to stand in the
> way.

I understand, but my preference is that all code should always look as
similar as possible style-wise. To my eyes, it is quite annoying to see,
e.g. "iter" and "idx" used mixed in the same source file, and it distracts
my code reading.

Best regards,
vmk
--
************************************************************************
               Tietotekniikkaosasto / Helsingin yliopisto
                 IT Department / University of Helsinki
************************************************************************

--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with
the words "unsubscribe selinux" without quotes as the message.

[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux