[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: [PATCH] drag to panel
From:       "Christopher Blauvelt" <cblauvelt () gmail ! com>
Date:       2008-02-02 13:56:04
Message-ID: ffa898c90802020556r240d392dr3fadd55e31cece24 () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Can we avoid one letter parameter/variable names and use something more
descriptive?  Otherwise it will be much more difficult later to maintain.
Also please avoid putting foul language in comments.  While the word doesn't
offend me, it de-values the codebase to have it laced with foul language.
Otherwise it looks great! :)

Chris

[Attachment #5 (text/html)]

Can we avoid one letter parameter/variable names and use something more \
descriptive?&nbsp; Otherwise it will be much more difficult later to maintain.&nbsp; \
Also please avoid putting foul language in comments.&nbsp; While the word doesn&#39;t \
offend me, it de-values the codebase to have it laced with foul language.<br> \
Otherwise it looks great! :)<br><br>Chris<br>



_______________________________________________
Panel-devel mailing list
Panel-devel@kde.org
https://mail.kde.org/mailman/listinfo/panel-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic