(Comment)Bug in CteScanNext - Mailing list pgsql-hackers

In CteScanNext():
/*      * If we are not at the end of the tuplestore, or are going 
backwards, try      * to fetch a tuple from tuplestore.      */     eof_tuplestore =
tuplestore_ateof(tuplestorestate);
     if (!forward && eof_tuplestore)

For the comment to be correct, wouldn't the condition need to be 
(!forward || !eof_tuplestore)?
-- 
Jim Nasby, Data Architect, Blue Treble Consulting, Austin TX
Experts in Analytics, Data Architecture and PostgreSQL
Data in Trouble? Get it in Treble! http://BlueTreble.com
855-TREBLE2 (855-873-2532)   mobile: 512-569-9461