| From: | Nicolai Tufar <ntufar(at)gmail(dot)com> | 
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | pgsql-hackers(at)postgresql(dot)org, pgsql-hackers-win32(at)postgresql(dot)org | 
| Subject: | Re: [HACKERS] %2$, %1$ gettext placeholder replacement is not working under Win32 | 
| Date: | 2005-01-17 20:16:31 | 
| Message-ID: | d809293905011712162e0d34fc@mail.gmail.com | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-hackers pgsql-hackers-win32 | 
On Mon, 17 Jan 2005 14:54:44 -0500, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Also, it looks like src/port/snprintf.c is not %n$ capable either.
> I'm not sure which platforms that affects.
> 
> A possible route to a solution is to upgrade snprintf.c and then use
> it on platforms that don't have this support.  This only fixes those
> cases where we go through snprintf, which is probably not all of the
> affected messages, but it might be enough.  It's not happening for
> 8.0.0 though.
Bad news for the Turks. Turkish language syntax
is lake Latin: the verb is always at the end. Because
of that almost all messages that have two or more 
placeholders are broken under Windows.
The solution you propose seem to be a trivial exercise:
1. write a replacement function, say, pg_snprintf() that 
    would shuffle input arguments, replace %x$ with % 
    and then call libc's snprintf();
2. replace snprintf() in all source files with pg_snprintf();
3. Update Makefiles to use our custom function on broken
    platforms or default to snprinf(). (Or maybe pg_snprintf()
    should be a #define);
4. Tell everyone to use pg_snprintf() instead of snprintf()
     from now on.
I would like volunteer to implement this for 8.1 and
would be honoured if I get a chance to contribute.
> regards, tom lane
Best regards,
Nicolai Tufar
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Andrew Sullivan | 2005-01-17 20:16:49 | Re: ARC patent | 
| Previous Message | Devrim GUNDUZ | 2005-01-17 20:16:07 | Re: [pgsql-hackers-win32] %2$, %1$ gettext placeholder | 
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Magnus Hagander | 2005-01-17 20:18:34 | Re: [HACKERS] %2$, %1$ gettext placeholder replacement is not working under Win32 | 
| Previous Message | Devrim GUNDUZ | 2005-01-17 20:16:07 | Re: [pgsql-hackers-win32] %2$, %1$ gettext placeholder |