[prev in list] [next in list] [prev in thread] [next in thread]
List: kstars-devel
Subject: Re: [Kstars-devel] KDE/kdeedu/kstars/kstars
From: Jasem Mutlaq <mutlaqja () ikarustech ! com>
Date: 2008-05-29 6:27:30
Message-ID: 200805290927.30781.mutlaqja () ikarustech ! com
[Download RAW message or body]
On Thursday 29 May 2008 09:06:32 am Akarsh Simha wrote:
> Hi Jasem,
>
> > As I said in my previous email, I already applied your patch and fixed a
> > few more issues. Now we have duplicate NULL checks as you can see below
> > :-) Please revert the changes if you have time.
> >
> > > -441,6 +441,9 @@
> > >
> > > if (m_CurrentLink == NULL) return;
> > >
> > > + if ( m_CurrentLink == NULL )
> > > + return;
> > > +
>
> Sorry I didn't understand you earlier. Thanks for pointing that
> out. I'll fix that when I find time.
>
> Before I proceed, is that the only change I need to revert, or were
> the other things applied as well?
No that's pretty much it.
--
Regards,
Jasem Mutlaq
Kuwait National Radio Observatory
Kuwait Science Club
http://www.astronomy.ksclub.org
_______________________________________________
Kstars-devel mailing list
Kstars-devel@kde.org
https://mail.kde.org/mailman/listinfo/kstars-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic