24 Nov
2014
24 Nov
'14
5:19 p.m.
On Mon, Nov 24, 2014 at 5:40 AM, Jef Driesen jef@libdivecomputer.org wrote:
You're right, I'm only looking at the basic patch without the new string stuff. So I left the code, and just fixed the memory leak.
The eonsteel backend is now on the master branch.
.. and here are the updated patches to support the extended information. Things still seem to work, although subsurface now has a somewhat misleading debug message:
string field "FW Version": 1.0.3 -- overwriting the existing firware of (null) string field "Serial": #p1437019 -- overwriting the existing serial of (null)
which might even SIGSEGV on non-Linux targets due to that "%s" with NULL. But that's purely a subsurface debug message issue.
Linus