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

Dirk Hohndel dirk at hohndel.org
Sat Jun 27 14:41:25 PDT 2015


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...



More information about the subsurface mailing list