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.
