Patch-review changes

Aleix Pol aleixpol at kde.org
Mon Feb 22 15:20:03 UTC 2010


On Sun, Feb 21, 2010 at 5:50 PM, David Nolden <zwabel at googlemail.com> wrote:

> Am Sonntag 21 Februar 2010 17:07:34 schrieb Andreas Pakulat:
> > > I think there is something to be learned: If you don't know how
> something
> > > works, and especially if you don't even use it, then simply don't touch
> > > its code! You can only break it then.
> >
> > That would just mean nobody would work on anything except the code he
> > wrote and thats just stupid. Just because somebody is not as fast as you
> > understanding somebody else's codebase (and frankly, the code you write
> > is not always the easiest to understand) doesn't mean he cannot fix the
> > issues he introduces. After all, we still have plenty of time to fix
> > things up.
>
> Well, I'm not saying that you must know the code, but you should at least
> be
> an active user of the feature that you're working in, as else you simply
> won't
> notice all the new issues you create.
>
> --
> KDevelop-devel mailing list
> KDevelop-devel at kdevelop.org
> https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel
>

Hi,
I'm sorry I haven't been able to address that properly. I'm really
disconnected these days and it's not easy for me.

About the lib change, I think it made sense from the moment we saw that with
the new version there was more stuff working.

I'll try to adapt better next time.

Sorry again,
Aleix
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20100222/623097a3/attachment.html>


More information about the KDevelop-devel mailing list