From kde-devel Tue Mar 30 20:58:51 2010 From: "Aaron J. Seigo" Date: Tue, 30 Mar 2010 20:58:51 +0000 To: kde-devel Subject: Re: [PATCH] KSnapshot bug Message-Id: <201003301358.52558.aseigo () kde ! org> X-MARC-Message: https://marc.info/?l=kde-devel&m=126998278815682 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 <<