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

List:       kde-devel
Subject:    Re: [PATCH] KSnapshot bug
From:       "Aaron J. Seigo" <aseigo () kde ! org>
Date:       2010-03-30 20:58:51
Message-ID: 201003301358.52558.aseigo () kde ! org
[Download RAW message or body]

On March 30, 2010, Vishesh Handa wrote:
> Yes. I saw it. My solution was quite horrible (and buggy! :(, and Asegio
> seems to have a much better solution (committed today). I didn't want to
> add an extra member variable and in an attempt to circumvent that I seem
> to have produced shitty code.
> 
> I'll try to test my patches better from next time. Sorry!

don't worry too much about it: you made an effort and actually made a patch. 
most people don't get half that far! and it's always difficult when you dive 
into unfamiliar code for the first time. keep going and i'm sure your next 
patch will be even better! :)

-- 
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE core developer sponsored by Qt Development Frameworks
 
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]