---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102799/
---
Review request for KDE Base Apps.
Description
---
Currently, the "Use
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102799/#review7185
---
As someone that works with NFS home directory systems with autof
> On Oct. 8, 2011, 10:48 p.m., Gary L. Greene Jr wrote:
> > As someone that works with NFS home directory systems with autofs much of
> > the time, I find the use of an icon in the users home directory (especially
> > on default configurations that tend to display many users on KDM) a very
> >
> On Oct. 8, 2011, 10:48 p.m., Gary L. Greene Jr wrote:
> > As someone that works with NFS home directory systems with autofs much of
> > the time, I find the use of an icon in the users home directory (especially
> > on default configurations that tend to display many users on KDM) a very
> >
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102799/#review7199
---
the change makes sense to me.
please remove the trailing whitesp
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102799/
---
(Updated Oct. 12, 2011, 4:55 p.m.)
Review request for KDE Base Apps.
Cha
No tabs, 4 spaces instead.
http://techbase.kde.org/Policies/Kdelibs_Coding_Style
On 12 October 2011 12:55, Ralf Jung wrote:
>This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102799/
> Review request for KDE Base Apps.
> By Ralf Jung.
>
> *Up
>On 13 October 2011 16:53, Ralf Jung wrote:
> Hi,
>
>> No tabs, 4 spaces instead.
>>
>> http://techbase.kde.org/Policies/Kdelibs_Coding_Style
> Almost the complete main.cpp is using tabs currently (except for
> KCMUserAccount::decodeImgDrop, which uses 2 spaces) so I used it for the two
> lines I
On Thu, Oct 13, 2011 at 06:27:45PM -0400, Steven Sroka wrote:
> >On 13 October 2011 16:53, Ralf Jung wrote:
> >> No tabs, 4 spaces instead.
> >>
> >> http://techbase.kde.org/Policies/Kdelibs_Coding_Style
> > Almost the complete main.cpp is using tabs currently (except for
> > KCMUserAccount::decod
Hi,
> No tabs, 4 spaces instead.
>
> http://techbase.kde.org/Policies/Kdelibs_Coding_Style
Almost the complete main.cpp is using tabs currently (except for
KCMUserAccount::decodeImgDrop, which uses 2 spaces) so I used it for the two
lines I added. Is that okay, or am I supposed to use 4 spaces
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102799/#review7558
---
Any news? Can I ship this (simplified) version of the original r
> On Oct. 23, 2011, 4:22 p.m., Ralf Jung wrote:
> > Any news? Can I ship this (simplified) version of the original request?
in qt review terms, you get a +1 from me. this code is pretty much
unmaintained, so it's unlikely that you'll get more feedback than that.
- Oswald
---
> On Oct. 23, 2011, 4:22 p.m., Ralf Jung wrote:
> > Any news? Can I ship this (simplified) version of the original request?
>
> Oswald Buddenhagen wrote:
> in qt review terms, you get a +1 from me. this code is pretty much
> unmaintained, so it's unlikely that you'll get more feedback than
---
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102799/#review8055
---
This review has been submitted with commit
f7ce3a513baef8479f76
14 matches
Mail list logo