On Monday 28 Jan 2008 3:14:39 pm Jarosław Staniek wrote:

> > Index: cursor.cpp
> > ===================================================================
> > --- cursor.cpp      (revision 766421)
> > +++ cursor.cpp      (working copy)
> > @@ -406,10 +406,10 @@
> >                                     m_afterLast = true;
> >  //js                                       m_at = m_records_in_buf;
> >                                     m_at = -1; //position is invalid now 
> > and will not be used
> > -                                   if ((FetchResult) m_result == FetchEnd) 
> > {
> > -                                           return false;
> > -                                   }
>
> [**]
>
> > -                                   setError(ERR_CURSOR_RECORD_FETCHING, 
> > i18n("Cannot fetch next
> > record.")); +//                                     if ((FetchResult) 
> > m_result == FetchEnd) {
> > +//                                         return false;
> > +//                                 }
> > +//sr                                       
> > setError(ERR_CURSOR_RECORD_FETCHING, i18n("Cannot fetch next
> > record.")); return false;
>
> Do we need the above? m_error == FetchError at [**] so I guess that
> setError() is expected.
>

Hmm, you mean to say, I should add a 

 if( (FetchResult) m_result == FetchError ) {
     setError( ERR_CURSOR_FETCHING, i18n("Cannot fetch next record") )  ;
 }

 at  [**]
 ? 

Cheers!
 Sharan Rao
_______________________________________________
Kexi mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kexi

Reply via email to