kdesupport/decibel

Matt Rogers mattr at kde.org
Tue Nov 11 04:41:53 CET 2008


On Monday 10 November 2008 18:06:12 Matthew Woehlke wrote:
> 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)
>

I don't know if I count as a decibel dev, but I prefer lowercase myself, and I 
don't think Tobias has any preference, it's probably mostly just copy and 
paste. :)
-- 
Matt


More information about the Kde-buildsystem mailing list