pgSet MoveNext bug ?

Поиск
Список
Период
Сортировка
От Robins Tharakan
Тема pgSet MoveNext bug ?
Дата
Msg-id 36af4bed0803031724s681693e2pf392db9dfac4b5b4@mail.gmail.com
обсуждение исходный текст
Ответы Re: pgSet MoveNext bug ?
Список 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 по дате отправления: