From: | Michael Meskes <meskes(at)postgresql(dot)org> |
---|---|
To: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Boszormenyi Zoltan <zb(at)cybertec(dot)at> |
Subject: | Re: Windows 64 bit warnings |
Date: | 2011-04-19 13:44:14 |
Message-ID: | 20110419134414.GA31777@feivel.credativ.lan |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sat, Apr 16, 2011 at 08:46:44PM -0400, Andrew Dunstan wrote:
> One is at src/interfaces/ecpg/ecpglib/sqlda.c:231, which is this line:
>
> sqlda->sqlvar[i].sqlformat = (char *) (long) PQfformat(res, i);
>
> I'm not clear about the purpose of this anyway. It doesn't seem to
> be used anywhere, and the comment on the field says it for future
By used you mean inside the postgres right? If so this is not surprising, sqlda
is used to give data to the program using the library.
> use. If we're going to do it, I think it should be cast to a long
> long on Win64, since a char * is 8 bytes there, while a long is only
> 4. But if we really want to store the result from PQfformat() in it,
> why is it a char * at all?
Zoltan, did you see this? Given that you wrote the code it might be good if you
could comment on this.
Michael
--
Michael Meskes
Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
Michael at BorussiaFan dot De, Meskes at (Debian|Postgresql) dot Org
Jabber: michael.meskes at googlemail dot com
VfL Borussia! Força Barça! Go SF 49ers! Use Debian GNU/Linux, PostgreSQL
From | Date | Subject | |
---|---|---|---|
Next Message | David Fetter | 2011-04-19 14:41:46 | Re: pgbench \for or similar loop |
Previous Message | Heikki Linnakangas | 2011-04-19 11:59:19 | Re: Warning during PostgreSQL 9.0.4 libpq.dll build on WinXP+MinGW |