[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-buildsystem
Subject: Re: kdesupport/decibel
From: Matthew Woehlke <mw_triad () users ! sourceforge ! net>
Date: 2008-11-11 0:06:12
Message-ID: 4918CC74.90706 () users ! sourceforge ! net
[Download RAW message or body]
Alexander Neundorf wrote:
> On Tuesday 11 November 2008, Matthew Woehlke wrote:
>> Alexander Neundorf wrote:
> ...
>>> I just saw the patch. So yes, try to stay consistent within the file,
>>> lowercase is preferred.
>> Ok. But I'm still curious what "consistency" you're thinking of:
>
> Yes, this looks like mostly uppercase.
> So either stay with all-uppercase (and make the few lowercase commands also
> uppercase) or convert everything to lower case.
Too late :-), I already checked it in sans case change. But it's better
anyway to do such things in a separate commit.
Alex: do you speak for decibel devs? Else, any decibel devs have a
preference here? Otherwise I'll plan to upper-case* everything tomorrow
or Wednesday, if I can remember that long ;-), and if no one beats me to
it. (And please don't think you'd be stepping on my toes if anyone wants
to do it themselves :-).)
(*...just because that seems to be predominant in decibel)
--
Matthew
Please do not quote my e-mail address unobfuscated in message bodies.
--
We are Microsoft. What you are experiencing is not a problem; it is an
undocumented feature. -- Unknown (from fortune's bofh-excuses)
_______________________________________________
Kde-buildsystem mailing list
Kde-buildsystem@kde.org
https://mail.kde.org/mailman/listinfo/kde-buildsystem
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic