[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Spreadsheet Import Dialog
From: |
Ben Pfaff |
Subject: |
Re: Spreadsheet Import Dialog |
Date: |
Sun, 10 Mar 2013 09:12:59 -0700 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sun, Mar 10, 2013 at 09:03:09AM -0700, Ben Pfaff wrote:
> On Sat, Mar 09, 2013 at 07:44:10PM +0100, John Darrington wrote:
> > I'm posting this proposed change for review purposes. It enables import of
> > spreadsheet files using the gui.
>
> This sounds like a useful feature. I'm going to look over the changes
> today.
I see some "printf"s sprinkled into the code. I guess these are stray
debugging code that has not yet been removed.
It looks like most of the patch is moving code around. I see a some new
code, but it is hard to spot in the motion. Is there any particular
code that you'd like to draw my attention to? If not, then this seems
like a reasonable set of changes to me.
I didn't compile or test this.