[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