diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2020-04-19 16:48:53 +0200 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2020-05-06 13:58:09 -0700 |
commit | 74f03e3537ee530000d4b11013bb12de91cb21dd (patch) | |
tree | d534e7ee2718318b9ae0bba9578ec84001a5dff9 | |
parent | 093551363602e723864467af0812f13f4525855f (diff) | |
download | subsurface-74f03e3537ee530000d4b11013bb12de91cb21dd.tar.gz |
media: move addition of pictures out of create_picture()
If we want to make addition of pictures undoable, then create_picture()
must not add directly to the dive. Instead, return the dive to which the
picture should be added and let the caller perform the addition.
This means that the picture-test has to be adapted.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
-rw-r--r-- | core/dive.c | 36 | ||||
-rw-r--r-- | core/dive.h | 3 | ||||
-rw-r--r-- | desktop-widgets/divelistview.cpp | 13 | ||||
-rw-r--r-- | tests/testpicture.cpp | 18 |
4 files changed, 45 insertions, 25 deletions
diff --git a/core/dive.c b/core/dive.c index 8ed9995b8..5ac008e8f 100644 --- a/core/dive.c +++ b/core/dive.c @@ -3556,7 +3556,7 @@ bool picture_check_valid_time(timestamp_t timestamp, int shift_time) return false; } -static void dive_set_geodata_from_picture(struct dive *dive, struct picture *picture, struct dive_site_table *table) +void dive_set_geodata_from_picture(struct dive *dive, struct picture *picture, struct dive_site_table *table) { struct dive_site *ds = dive->dive_site; if (!dive_site_has_gps_location(ds) && has_location(&picture->location)) { @@ -3570,31 +3570,31 @@ static void dive_set_geodata_from_picture(struct dive *dive, struct picture *pic } } -void create_picture(const char *filename, int shift_time, bool match_all) +/* Creates a picture and indicates the dive to which this picture should be added. + * The caller is responsible for actually adding the picture to the dive. + * If no appropriate dive was found, no picture is created and NULL is returned. + */ +struct picture *create_picture(const char *filename, int shift_time, bool match_all, struct dive **dive) { struct metadata metadata; - struct dive *dive; timestamp_t timestamp; get_metadata(filename, &metadata); timestamp = metadata.timestamp + shift_time; - dive = nearest_selected_dive(timestamp); - - if (!dive) - return; - if (get_picture_idx(&dive->pictures, filename) >= 0) - return; - if (!match_all && !dive_check_picture_time(dive, timestamp)) - return; + *dive = nearest_selected_dive(timestamp); - struct picture picture; - picture.filename = strdup(filename); - picture.offset.seconds = metadata.timestamp - dive->when + shift_time; - picture.location = metadata.location; + if (!*dive) + return NULL; + if (get_picture_idx(&(*dive)->pictures, filename) >= 0) + return NULL; + if (!match_all && !dive_check_picture_time(*dive, timestamp)) + return NULL; - add_picture(&dive->pictures, picture); - dive_set_geodata_from_picture(dive, &picture, &dive_site_table); - invalidate_dive_cache(dive); + struct picture *picture = malloc(sizeof(struct picture)); + picture->filename = strdup(filename); + picture->offset.seconds = metadata.timestamp - (*dive)->when + shift_time; + picture->location = metadata.location; + return picture; } /* clones a dive and moves given dive computer to front */ diff --git a/core/dive.h b/core/dive.h index 33c7c8b0f..8e9ee280a 100644 --- a/core/dive.h +++ b/core/dive.h @@ -210,8 +210,9 @@ extern enum divemode_t get_divemode_at_time(const struct divecomputer *dc, int d for (struct picture *picture = (_dive)->pictures.pictures; \ picture < (_dive)->pictures.pictures + (_dive)->pictures.nr; \ picture++) -extern void create_picture(const char *filename, int shift_time, bool match_all); +extern struct picture *create_picture(const char *filename, int shift_time, bool match_all, struct dive **dive); extern bool picture_check_valid_time(timestamp_t timestamp, int shift_time); +extern void dive_set_geodata_from_picture(struct dive *dive, struct picture *picture, struct dive_site_table *table); extern bool has_gaschange_event(const struct dive *dive, const struct divecomputer *dc, int idx); extern int explicit_first_cylinder(const struct dive *dive, const struct divecomputer *dc); diff --git a/desktop-widgets/divelistview.cpp b/desktop-widgets/divelistview.cpp index 403165759..fc7d34047 100644 --- a/desktop-widgets/divelistview.cpp +++ b/desktop-widgets/divelistview.cpp @@ -11,6 +11,7 @@ #include "desktop-widgets/divepicturewidget.h" #include "core/selection.h" #include "core/divefilter.h" +#include "core/divesite.h" // for dive_site_table. TODO: remove once adding pictures is undoified #include <unistd.h> #include <QSettings> #include <QKeyEvent> @@ -890,8 +891,16 @@ void DiveListView::matchImagesToDives(QStringList fileNames) return; updateLastImageTimeOffset(shiftDialog.amount()); - for (const QString &fileName: fileNames) - create_picture(qPrintable(fileName), shiftDialog.amount(), shiftDialog.matchAll()); + for (const QString &fileName: fileNames) { + struct dive *d; + picture *pic = create_picture(qPrintable(fileName), shiftDialog.amount(), shiftDialog.matchAll(), &d); + if (!pic) + continue; + add_picture(&d->pictures, *pic); + dive_set_geodata_from_picture(d, pic, &dive_site_table); + invalidate_dive_cache(d); + free(pic); + } mark_divelist_changed(true); copy_dive(current_dive, &displayed_dive); diff --git a/tests/testpicture.cpp b/tests/testpicture.cpp index 7bd683523..b5c6ae22a 100644 --- a/tests/testpicture.cpp +++ b/tests/testpicture.cpp @@ -22,7 +22,7 @@ void TestPicture::initTestCase() void TestPicture::addPicture() { - struct dive *dive; + struct dive *dive, *dive1, *dive2; struct picture *pic1, *pic2; verbose = 1; @@ -34,9 +34,19 @@ void TestPicture::addPicture() // So far no picture in dive QVERIFY(dive->pictures.nr == 0); - create_picture(SUBSURFACE_TEST_DATA "/dives/images/wreck.jpg", 0, false); - create_picture(SUBSURFACE_TEST_DATA "/dives/images/data_after_EOI.jpg", 0, false); - // Now there are two picture2 + pic1 = create_picture(SUBSURFACE_TEST_DATA "/dives/images/wreck.jpg", 0, false, &dive1); + pic2 = create_picture(SUBSURFACE_TEST_DATA "/dives/images/data_after_EOI.jpg", 0, false, &dive2); + QVERIFY(pic1 != NULL); + QVERIFY(pic2 != NULL); + QVERIFY(dive1 == dive); + QVERIFY(dive2 == dive); + + add_picture(&dive->pictures, *pic1); + add_picture(&dive->pictures, *pic2); + free(pic1); + free(pic2); + + // Now there are two pictures QVERIFY(dive->pictures.nr == 2); pic1 = &dive->pictures.pictures[0]; pic2 = &dive->pictures.pictures[1]; |