> -----Original Message-----
> From: Miha Radej [mailto:[EMAIL PROTECTED] 
> Sent: 29 May 2006 21:27
> To: Dave Page
> Cc: pgadmin-hackers
> Subject: Re: [pgadmin-hackers] Compiling pgAdmin on SuSE 10.1
> 
> Hi!
> 
> Dave Page pravi:
> >> ./frm/frmReport.cpp: In member function 'virtual void
> >> reportObjectDataDictionaryFactory::GenerateReport(frmReport*, 
> >> pgObject*)':
> >> ./frm/frmReport.cpp:1335: warning: cannot pass objects of 
> non-POD type
> >> 'class wxString' through '...'; call will abort at runtime
> > 
> > Yeah, these ones only fail on GCC - it's complaining about passing a
> > wxString as a parameter to a variadic function. The fix is 
> to use the
> > c_str() member, eg.
> 
> The above warnings are now gone, but I noticed these now:
> 
> ./frm/frmEditGrid.cpp: In member function 'void sqlTable::Paste()':
> ./frm/frmEditGrid.cpp:1840: warning: comparison between signed and
> unsigned integer expressions
> ./frm/frmEditGrid.cpp:1841: warning: comparison between signed and
> unsigned integer expressions
> ./frm/frmEditGrid.cpp:1856: warning: comparison between signed and
> unsigned integer expressions
> ./frm/frmEditGrid.cpp:1857: warning: comparison between signed and
> unsigned integer expressions
> 
> The attached patch is yet another "works for me" kind of 
> patch, I don't
> know what good/standard C++ practice is in such situations, 
> considering
> I don't know C++ at all :)

:-)

Modified patch applied. I blind-fixed that one earlier and put the casts
on the wrong operands :-(

Thanks, Dave.

---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faq

Reply via email to