[PATCH] parser: add DC_FIELD_DEVINFO field type for parse-time device information

Linus Torvalds torvalds at linux-foundation.org
Wed Oct 22 15:00:36 PDT 2014


On Wed, Oct 22, 2014 at 12:40 PM, Linus Torvalds
<torvalds at linux-foundation.org> wrote:
>
>
> So I'm sending this out not to be applied by Dirk, but to show how simple
> the new DC_FIELD_DEVINFO model is to use, and how subsurface can trivially
> use it.

Oh, and I realized that I need to add another

  #ifdef DC_FIELD_DEVINFO
  ..
  #endif

around the whole parse_devinfo() function too, since otherwise the
compile will fail with any libdivecomputer version that doesn't define
the 'dc_field_devinfo_t' type.

But other than that, I'm actually using this all now, and the EON
Steel profiles are now part of the dive logs I maintain..

I will have to add some event parsing (and turn the ceiling and tts
data into events too, for that matter), but the basics all work.
Supporting things like synchronizing the time or even changing the
configurable screens is a whole different matter. That's going to take
a lot more effort.

                      Linus


More information about the subsurface mailing list