Movatterモバイル変換


[0]ホーム

URL:


Quick Links

Re: Built-in CTYPE provider

From:Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To:Jeff Davis <pgsql(at)j-davis(dot)com>
Cc:Peter Eisentraut <peter(at)eisentraut(dot)org>, Daniel Verite <daniel(at)manitou-mail(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>, Jeremy Schneider <schneider(at)ardentperf(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject:Re: Built-in CTYPE provider
Date:2024-03-18 22:04:55
Message-ID:3703896.1710799495@sss.pgh.pa.us
Views:Whole Thread |Raw Message |Download mbox |Resend email
Thread:
Lists:pgsql-hackers

Jeff Davis <pgsql(at)j-davis(dot)com> writes:
> It may be moot soon, but I committed a fix now.

Thanks, but it looks like 846311051 introduced a fresh issue.
MSVC is complaining about

[21:37:15.349] c:\cirrus\src\backend\utils\adt\pg_locale.c(2515) : warning C4715: 'builtin_locale_encoding': not all control paths return a value

This is causing all CI jobs to fail the "compiler warnings" check.

Probably the best fix is the traditional

return <something>; /* keep compiler quiet */

but I'm not sure what the best default result is in this function.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

 FromDateSubject
Next MessageMaiquel Grassi2024-03-18 22:05:17RE: Psql meta-command conninfo+
Previous MessageLaurenz Albe2024-03-18 21:40:07Re: documentation structure

[8]ページ先頭

©2009-2026 Movatter.jp