[PATCH 1/4] Add support for importing o2 sensors from CSV file

Tomaz Canabrava tcanabrava at kde.org
Sat Jun 27 15:06:24 PDT 2015


The gtk / qt was costly but worth it. And I'm already reworking some stuff.

On Sat, Jun 27, 2015, 18:41 Dirk Hohndel <dirk at hohndel.org> wrote:

> On Sat, Jun 27, 2015 at 11:38:10PM +0200, Anton Lundin wrote:
> >
> > Dam, this code is clunky. We should really rework how that field-thingie
> > works.
> >
> > I want to pry my eyes out every time i touch that code.
> >
> > How many places do you need to touch to add a field to the csv-importer?
> >  - To many!
>
> It's pretty disgusting, that's for sure.
> I'm open to ideas for a redesign. But I always wonder about the
> cost/benefit analysis of things like that...
>
> _______________________________________________
> subsurface mailing list
> subsurface at subsurface-divelog.org
> http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150627/d4d7922f/attachment.html>


More information about the subsurface mailing list