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

List:       kde-devel
Subject:    Re: Patch for KTeaTime
From:       Thomas Baumgart <thb () net-bembel ! de>
Date:       2007-08-28 4:37:12
Message-ID: 200708280637.12524 () net-bembel ! de
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


Hi all,

on Monday 27 August 2007 23:02, Daniel Teske wrote:

> > > Hmm, this is actually wrong. The correct code is:
> > > if(parent.isValid())
> > >    return 0;
> > > else
> > >    return m_tealist.size();
> > >
> > > (Pretty clear, right?)
> >
> > Not for me :-(
> > This code returns something if the index is _not_ valid? Is this the
> > intention?
>
> Yes. :)
> The (invisible) root of a model is a invalid QModelIndex. That root has
> one child per tea. So for the root the code must return tealist.siez().
> Each and every child, i.e. every tea has no child, so if the QModelIndex
> is valid, and is thus not the invisible root, the code musr return 0.

An idea might be to drop just that as comment in front of the resp. code 
section.


-- 

Regards

Thomas Baumgart

GPG-FP: E55E D592 F45F 116B 8429   4F99 9C59 DB40 B75D D3BA
-------------------------------------------------------------
The amount of experience is directly proportional to
the amount of destroyed equipment.
-------------------------------------------------------------

[Attachment #5 (application/pgp-signature)]

>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<


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