pgSet MoveNext bug ?

Поиск
Список
Период
Сортировка
От Robins Tharakan
Тема pgSet MoveNext bug ?
Дата
Msg-id 36af4bed0803031724s681693e2pf392db9dfac4b5b4@mail.gmail.com
обсуждение исходный текст
Ответы Re: pgSet MoveNext bug ?  ("Dave Page" <dpage@pgadmin.org>)
Список pgadmin-hackers
Дерево обсуждения
Hi,

While reading the code, the PGSet->MoveNext() definition seems to have a small bug.

Since the PGSet->EOF() is defined as
bool Eof() const { return (!nRows || pos > nRows); }

I think it doesn't make sense to define PGSet->MoveNext() as
void MoveNext() { if (pos <= nRows) pos++; }

It should rather be
void MoveNext() { if (pos < nRows) pos++; }

The attached patch does the same.

Regards,
Robins Tharakan
Вложения

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: svn@pgadmin.org
Дата:
Сообщение: SVN Commit by dpage: r7111 - trunk/pgadmin3/plugins
Следующее
От: "Dave Page"
Дата:
Сообщение: Re: Display line numbers in SQL view