Re: Add RESPECT/IGNORE NULLS and FROM FIRST/LAST options

On 2025-Oct-03, Tatsuo Ishii wrote:

> I have just pushed the patch (plus patches for syntax.sgml and
> sql_features.txt. They were missued after I splitted the patch).
> Thank you for your effort!

I just noticed this compiler warning in a CI run,

[16:06:29.920] ../src/backend/executor/nodeWindowAgg.c:3820:16: warning: ‘datum’ may be used uninitialized
[-Wmaybe-uninitialized]
[16:06:29.920]  3820 |         return datum;
[16:06:29.920]       |                ^~~~~
[16:06:29.920] ../src/backend/executor/nodeWindowAgg.c:3719:25: note: ‘datum’ was declared here
[16:06:29.920]  3719 |         Datum           datum;
[16:06:29.920]       |                         ^~~~~

The logic in this function looks somewhat wicked.

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/