Skip site navigation (1) Skip section navigation (2)

Re: [PATCH] Buildfailure on OSX (10.3.9) + fix

From: "Hiroshi Saito" <z-saito(at)guitar(dot)ocn(dot)ne(dot)jp>
To: "Florian G(dot) Pflug" <fgp(at)phlo(dot)org>,"pgadmin-hackers" <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH] Buildfailure on OSX (10.3.9) + fix
Date: 2007-03-31 09:52:15
Message-ID: 01ba01c7737a$434e7c90$0e01a8c0@hiroshi6e67244 (view raw or flat)
Thread:
Lists: pgadmin-hackers
Hi Florian-san.

It was applied. 
Is the screen of OSX comfortable?:-)
Thanks!

Regards,
Hiroshi Saito

----- Original Message ----- 
From: "Florian G. Pflug" 


> Hi
> 
> I've finally fixed my buildserver today, and discovered that
> the build of pgadmin3 trunk on OSX 10.3.9 (gcc 3.3) fails
> when compiling the debugger.
> 
> Attached is a patch that fixes the problem, though I must admit
> that I soley found the fix through trial and error. I have
> absolutly no idea why gcc complains about
> wxString( wxString( PQsomething(result), wxConvUTF8 ))
> but has no problem with
> wxString( PQsomething(result), wxConvUTF8 )
> 
> The error message is:
> wsDirectdbg.cpp: In member function `void
>    wsDirectDbg::OnResultReady(wxCommandEvent&)':
> wsDirectdbg.cpp:608: error: parse error before `,' token
> 
> Does someone have the slighest idea what gcc is unhappy about?
> Anyway, patch is attached.
> 
> greetings, Florian Pflug
>


--------------------------------------------------------------------------------


> Index: debugger/wsDirectdbg.cpp
> ===================================================================
> --- debugger/wsDirectdbg.cpp (revision 6146)
> +++ debugger/wsDirectdbg.cpp (working copy)
> @@ -605,7 +605,7 @@
> 
>  if(( PQresultStatus( result ) == PGRES_NONFATAL_ERROR ) || ( PQresultStatus( result ) == PGRES_FATAL_ERROR ))
>  {
> - wxString message( wxString(PQresultErrorMessage( result ), wxConvUTF8));
> + wxString message( PQresultErrorMessage( result ), wxConvUTF8 ) ;
> 
>  message.Replace( wxT( "\r" ), wxT( "" ));
>  message.Replace( wxT( "\n" ), wxT( " " ));
> @@ -615,7 +615,7 @@
>  }
>  else
>  {
> - wxString message( wxString( PQcmdStatus( result ), wxConvUTF8 ));
> + wxString message( PQcmdStatus( result ), wxConvUTF8 );
> 
>  message.Replace( wxT( "\r" ), wxT( "" ));
>  message.Replace( wxT( "\n" ), wxT( " " ));
> @@ -667,7 +667,7 @@
> 
>  if(( PQresultStatus( rawResult ) == PGRES_NONFATAL_ERROR ) || ( PQresultStatus( rawResult ) == PGRES_FATAL_ERROR ))
>  {
> - wxString message( wxString( PQresultErrorMessage( rawResult ), wxConvUTF8 ));
> + wxString message( PQresultErrorMessage( rawResult ), wxConvUTF8 );
> 
>  message.Replace( wxT( "\r" ), wxT( "" ));
>  message.Replace( wxT( "\n" ), wxT( " " ));
> @@ -677,7 +677,7 @@
>  }
>  else
>  {
> - wxString message( wxString( PQcmdStatus( rawResult ), wxConvUTF8 ));
> + wxString message( PQcmdStatus( rawResult ), wxConvUTF8 );
> 
>  message.Replace( wxT( "\r" ), wxT( "" ));
>  message.Replace( wxT( "\n" ), wxT( " " ));
> Index: debugger/wsQueryWindow.cpp
> ===================================================================
> --- debugger/wsQueryWindow.cpp (revision 6146)
> +++ debugger/wsQueryWindow.cpp (working copy)
> @@ -176,7 +176,7 @@
> 
>  if(( PQresultStatus( result ) == PGRES_NONFATAL_ERROR ) || ( PQresultStatus( result ) == PGRES_FATAL_ERROR ))
>  {
> - wxString message( wxString( PQresultErrorMessage( result ), wxConvUTF8 ));
> + wxString message( PQresultErrorMessage( result ), wxConvUTF8 );
> 
>  message.Replace( wxT( "\r" ), wxT( "" ));
>  message.Replace( wxT( "\n" ), wxT( " " ));
>


--------------------------------------------------------------------------------


> 
> ---------------------------(end of broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings
>

In response to

Responses

pgadmin-hackers by date

Next:From: Florian G. PflugDate: 2007-03-31 11:33:18
Subject: Re: [PATCH] Buildfailure on OSX (10.3.9) + fix
Previous:From: svnDate: 2007-03-31 09:47:05
Subject: SVN Commit by hiroshi: r6147 - trunk/pgadmin3/debugger

Privacy Policy | About PostgreSQL
Copyright © 1996-2014 The PostgreSQL Global Development Group