| From: | "Hiroshi Saito" <z-saito(at)guitar(dot)ocn(dot)ne(dot)jp> | 
|---|---|
| To: | "Magnus Hagander" <magnus(at)hagander(dot)net>, "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> | 
| Cc: | <pgsql-patches(at)postgresql(dot)org> | 
| Subject: | Re: Warning is adjusted of pgbench. | 
| Date: | 2007-09-28 01:51:36 | 
| Message-ID: | 0b7b01c80172$1aaa0a30$c601a8c0@HP22720319231 | 
| Views: | Whole Thread | Raw Message | Download mbox | Resend email | 
| Thread: | |
| Lists: | pgsql-patches | 
Hi.
From: "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> Magnus Hagander <magnus(at)hagander(dot)net> writes:
>> On Tue, Aug 07, 2007 at 04:58:19PM +0900, Hiroshi Saito wrote:
>>> pgbench.c: In function `main':
>>> pgbench.c:1257: warning: implicit declaration of function `getopt'
>>> 
>>> adjustment of some reference is required for this.
> 
>> Why do you need to #undef EXEC_BACKEND, and is there a specific reason for
>> removing the include of win32.h? 
> 
> I've applied the attached modified version of this patch; it keeps
> win32.h in there (correctly re-marked as a system header).  I think
> the fundamental issue must be that Hiroshi's system sets HAVE_GETOPT and
> HAVE_GETOPT_H.  The former causes port.h to not supply a declaration
> of getopt(), so we'd better include <getopt.h> to declare it.
> 
> Together with the changes for FRONTEND applied a little bit ago,
> I hope that this issue is fully resolved in CVS HEAD.  Let me know if
> there's still a problem.
Yeah, I think that it felt it very refreshed.!:-)
Though FRONTEND needs adjustment to consider carefully.
Thanks.
Regards,
Hiroshi Saito
| From | Date | Subject | |
|---|---|---|---|
| Next Message | ITAGAKI Takahiro | 2007-09-28 02:24:17 | Various ecpg cleanup | 
| Previous Message | Hiroshi Saito | 2007-09-28 01:42:44 | Re: Warning is adjusted of pgbench. |