Re: import: source code is pushed to git repository

Le 04/01/2010 10:18, Dave Page a écrit :
> On Mon, Jan 4, 2010 at 4:03 AM, Quan Zongliang <quanzongliang@gmail.com> wrote:
>>> If you don't have a Linux box, I'd suggest grabbing a copy of
>>> VirtualBox for Windows, and installing Ubuntu or Fedora in it.
>>
>> It can work now. CSV ZIP and TAR files can be imported.
>
> Cool - I'll give it a try as soon as I can. I'm away next week, and
> catching up/preparing this week though, so it may not be for a little
> while.
>
> I'd encourage others to give Quan's code a try.
>

I tried it. I fixed a few warnings I got with gcc. I fixed also the
installation of frmImport.xrc. The patch is attached.

Unfortunately, I didn't fix all warnings. Here are the ones still there:

./frm/frmImport.cpp: In member function ‘void
frmImport::OnArchFilePathKillFocus(wxFocusEvent&)’:
./frm/frmImport.cpp:841: warning: taking address of temporary
./frm/frmImport.cpp: In member function ‘void frmImport::ImportData()’:
./frm/frmImport.cpp:1458: warning: unused variable ‘logwarn’
./frm/frmImport.cpp: In member function ‘void frmImport::GenColsPreview()’:
./frm/frmImport.cpp:3779: warning: unused variable ‘skipemptyline’
./frm/frmImport.cpp:3421: warning: unused variable ‘errcount’
./frm/frmImport.cpp: In member function ‘bool
frmImport::GenCSVSQLData(wxString&, wxString&, wxString**, size_t,
wxDateTime&, int, int)’:
./frm/frmImport.cpp:4199: warning: unused variable ‘pvgridrowidx’
./frm/frmImport.cpp: In member function ‘void
frmImport::ResetColSettingSrcColEditor(bool)’:
./frm/frmImport.cpp:5516: warning: unused variable ‘firstish’
./frm/frmImport.cpp: In member function ‘frmImport::exprSnippet*
frmImport::ParseImportExpr(const wxString&)’:
./frm/frmImport.cpp:6826: warning: unused variable ‘incoldef’

The vast majority are unused variables. Probably because of a WIP. The
only one that really bugs me is:

./frm/frmImport.cpp:841: warning: taking address of temporary

I don't see how to fix this. Certainly because I don't understand the
warning :/

With all my fixes, I'm able to launch the frmImport window. I load a csv
file. When I click Next, it crashes. See the screenshots attached. I
don't know why yet. I also attached the csv file, as it could be the
culprit. FYI, here is the table definition:

CREATE TABLE fbb_bans
(
  id serial NOT NULL,
  username character varying(200),
  ip character varying(255),
  email character varying(50),
  message character varying(255),
  expire integer,
  CONSTRAINT fbb_bans_pkey PRIMARY KEY (id)
)

Any idea on what could be wrong? and what about this strange warning
message?


--
Guillaume.
 http://www.postgresqlfr.org
 http://dalibo.com