Re: Fix for compiler warning triggered in WinGetFuncArgInPartition()

From: Nathan Bossart <nathandbossart(at)gmail(dot)com>
To: Greg Burd <greg(at)burd(dot)me>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Fix for compiler warning triggered in WinGetFuncArgInPartition()
Date: 2025-10-09 14:45:15
Message-ID: aOfKe98emuyeB9yi@nathan
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, Oct 09, 2025 at 09:41:15AM -0500, Nathan Bossart wrote:
> On Thu, Oct 09, 2025 at 10:21:23AM -0400, Greg Burd wrote:
>> ‘WinGetFuncArgInPartition’:../src/backend/executor/nodeWindowAgg.c:3809:33:
>> warning:‘notnull_relpos’ may be used uninitialized
>> [-Wmaybe-uninitialized] 3809 | } while (notnull_offset <
>> notnull_relpos); |
>> ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~../src/backend/executor/nodeWindowAgg.c:3723:33:
>> note: ‘notnull_relpos’was declared here 3723 | int
>> notnull_relpos; | ^~~~~~~~~~~~~~
>
> Just curious, what compiler/version is this? I skimmed the buildfarm a bit
> but didn't find any such warnings.

Looks like Tom just fixed this: https://postgr.es/c/71540dc

--
nathan

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2025-10-09 14:47:03 Re: Should we update the random_page_cost default value?
Previous Message Melanie Plageman 2025-10-09 14:43:52 Re: Fix overflow of nbatch