diff options
| author |  Linus Torvalds <torvalds@linux-foundation.org> | 2012-12-16 12:33:37 -0800 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2012-12-16 15:48:32 -1000 |
| commit | 7b457d1b5c44db5db1a2bace178fd9c3e447a65c (patch) | |
| tree | 78cbe4748a77665af33972da941070fcde470190 /save-xml.c | |
| parent | 459696c90c94834b7498a1b20cd6ccb85bfaa9e0 (diff) | |
| download | subsurface-7b457d1b5c44db5db1a2bace178fd9c3e447a65c.tar.gz | |
Properly remove redundant dive computer information
We had logic to remove duplicate dive computer information after merging
dives, but it didn't actually work.
Why? Because we had used the 'res' dive computer pointer to traverse the
list of dive computers, so it no longer actually pointed to the first
dive computer in the result list any more, and so the "remove redundant"
code only removed redundant dive computers from a limited and incomplete
list.
Oops.
Also, before checking the whole event and sample list, check if it's the
exact same dive computer using our new "match_one_dc()" helper function,
and don't even bother checking for sample details if it is.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'save-xml.c')
0 files changed, 0 insertions, 0 deletions