latest defaultfile branch

Dirk Hohndel dirk at hohndel.org
Fri Sep 14 08:06:38 PDT 2012


Henrik Brautaset Aronsen <subsurface at henrik.synth.no> writes:

> Den 14.09.12 08:13, skrev Henrik Brautaset Aronsen:
>> Den 12.09.12 01:28, skrev Dirk Hohndel:
>>> Can I please get some testing on Mac and Windows (and of course also
>>> Linux), and especially Henrik, can you verify that you can still see the
>>> bug that you reported (because either I don't understand the steps that
>>> you sent, or I'm seriously confused... it seems to work as expected
>>> here).
>>
>> I was finally able to test your changes.  (I've been busy!  I just did 
>> my 200th dive *and* I've finished my Divemaster *and* I was at a 
>> two-day conference with great food.  It was more like a food festival 
>> that somehow included presentations and workshops.)
>>
>> And I must say it all works great.  Thanks for implementing this!
>
> Too soon!  I found one edge case when starting without a log:
>
> 1) Open Subsurface, Import test1.xml, Quit --> Should ask about saving 
> default log (OK)
> 2) Open Subsurface, Import test1.xml, Save, Quit --> Should save default 
> log (OK)
> 3) Open Subsurface, Open test1.xml, Quit --> Shouldn't save default log (OK)
> 4) Open Subsurface, Open test1.xml, Save, Quit --> Shouldn't save 
> default log (FAIL)
> 5) Open Subsurface, Open test1.xml, Edit dive, Save, Quit --> Should 
> save test1.xml (FAIL), shouldn't save default log (FAIL)

These are all the same "feature". When Linus implemented the Open
function he implemented it as "add to the existing data file".

I think at this point this makes no sense and we should have "Open" and
"Import" as two distinct operations with different semantics.

Linus?

/D


More information about the subsurface mailing list