subsurface.git/tests, branch v4.8.0

Add planner infra structure for bailout

2018-05-14T20:47:00Z

Add a divemode column to the planner model and a corresponding field to struct divepoint and fill it in the corresponding functions. Signed-off-by: Robert C. Helling

Core: remove dive.h from files that don't need it

2018-05-14T17:13:39Z

Of course, quite a few of them indirectly get it through other header files. Signed-off-by: Dirk Hohndel

Core: introduce new subsurface-string header

2018-05-14T17:13:39Z

First small step to shrinking dive.h. Signed-off-by: Dirk Hohndel

CSV import: fix setpoint (typo)

2018-04-19T12:09:02Z

Signed-off-by: Miika Turkia

Change taglist_get_tagstring to support 'unlimited' tag list size

2018-04-09T14:59:51Z

Previous taglist_get_tagstring signature/implementation did not allow handling of cases where inputted buffer could not contain all tags. New implementation allocates buffer based on pre-computed size allowing to insert all tags in the returned string. Added get_taglist_string in qthelper to handle conversion to QString Added TestTagList with tests for taglist_get_tagstring Signed-off-by: Jeremie Guichard

CSV import/export: Sort parameter list and add comment

2018-03-19T18:59:58Z

Signed-off-by: Stefan Fuchs

Cleanup: introduce copy_qstring() function

2018-03-14T11:55:36Z

strdup(qPrintable(s)) and copy_string(qPrintable(s)) were such common occurrences that they seem worthy of a short helper-function. Signed-off-by: Berthold Stoeger

Cleanup: consistently use qPrintable()

2018-03-14T11:55:36Z

Replace constructs of the kind s.toUtf8().data(), s.toUtf8().constData(), s.toLocal8Bit().data(), s.toLocal8Bit.constData() or qUtf8Printable(s) by qPrintable(s). This is concise, consistent and - in principle - more performant than the .data() versions. Sadly, owing to a suboptimal implementation, qPrintable(s) currently is a pessimization compared to s.toUtf8().data(). A fix is scheduled for new Qt versions: https://codereview.qt-project.org/#/c/221331/ Signed-off-by: Berthold Stoeger

Cleanup: Remove hash field from picture-structure

2018-03-05T16:04:57Z

The hash field in the picture-structure was in principle non-operational. It was set on loading, but never actually changed. The authoritative hash comes from the filename->hash map. Therefore, make this explicit by removing the hash field from the picture structure. Instead of filling the picture structure on loading, add the hash directly to the filename->hash map. This is done in the register_hash() function, which does not overwrite old entries. I.e. the local hash has priority over the save-file. This policy might be refined in the future. Signed-off-by: Berthold Stoeger

Cleanup: Fold add_hash() call into learnHash() function

2018-03-05T16:04:57Z

learnHash() was always called in conjunction with add_hash(). The pattern was that a local filename and a hash were connected in the hash-to-filename and the filename-to-hash maps. Then, the original picture-filename or url were registered in the filename-to-hash map. This commit changes learnHash() to take three parameters (original-filename, local-filename and hash) and do all of the above. The new code is simpler because no dummy picture struct has to be generated in DiveListView::loadImageFromURL(). The tests were extended to check for all hash<->filename associations. Signed-off-by: Berthold Stoeger