get rid of some legacy events;

update playlist grouping on metadata changes

9 files changed, 32 insertions, 18 deletions

diff --git a/plugins/gtkui/ddblistview.c b/plugins/gtkui/ddblistview.c
index 996e66cd..a2c4bd16 100644
--- a/plugins/gtkui/ddblistview.c
+++ b/plugins/gtkui/ddblistview.c
@@ -530,8 +530,8 @@ ddb_listview_list_configure_event            (GtkWidget       *widget,
 
 static void
 ddb_listview_groupcheck (DdbListview *listview) {
-    time_t tm = listview->binding->modification_time ();
-    if (tm != listview->groups_build_time) {
+    int idx = listview->binding->modification_idx ();
+    if (idx != listview->groups_build_idx) {
         ddb_listview_build_groups (listview);
     }
 }
@@ -2882,7 +2882,7 @@ void
 ddb_listview_build_groups (DdbListview *listview) {
     deadbeef->pl_lock ();
     int old_height = listview->fullheight;
-    listview->groups_build_time = listview->binding->modification_time ();
+    listview->groups_build_idx = listview->binding->modification_idx ();
     ddb_listview_free_groups (listview);
     listview->fullheight = 0;
 
diff --git a/plugins/gtkui/ddblistview.h b/plugins/gtkui/ddblistview.h
index c7476607..ed1334e8 100644
--- a/plugins/gtkui/ddblistview.h
+++ b/plugins/gtkui/ddblistview.h
@@ -85,7 +85,7 @@ typedef struct {
     void (*col_free_user_data) (void *user_data);
     void (*vscroll_changed) (int pos);
     void (*cursor_changed) (int pos);
-    time_t (*modification_time) (void);
+    int (*modification_idx) (void);
 } DdbListviewBinding;
 
 struct _DdbListviewColumn;
@@ -140,7 +140,7 @@ struct _DdbListview {
 
     struct _DdbListviewColumn *columns;
     struct _DdbListviewGroup *groups;
-    time_t groups_build_time;
+    int groups_build_idx; // must be the same as playlist modification idx
     int fullheight;
     int block_redraw_on_scroll;
     int grouptitle_height;
diff --git a/plugins/gtkui/gtkui.c b/plugins/gtkui/gtkui.c
index e1ee403a..7e3a0ee0 100644
--- a/plugins/gtkui/gtkui.c
+++ b/plugins/gtkui/gtkui.c
@@ -573,11 +573,11 @@ gtkui_hide_status_icon () {
     }
 }
 
-time_t
-gtkui_get_curr_playlist_modtime (void) {
+int
+gtkui_get_curr_playlist_mod (void) {
     deadbeef->pl_lock ();
     void *plt = deadbeef->plt_get_handle (deadbeef->plt_get_curr ());
-    time_t res = plt ? deadbeef->plt_get_modification_time (plt) : 0;
+    int res = plt ? deadbeef->plt_get_modification_idx (plt) : 0;
     deadbeef->pl_unlock ();
     return res;
 }
diff --git a/plugins/gtkui/gtkui.h b/plugins/gtkui/gtkui.h
index e940bcfa..dfc73455 100644
--- a/plugins/gtkui/gtkui.h
+++ b/plugins/gtkui/gtkui.h
@@ -166,7 +166,7 @@ gtkui_playlist_set_curr (int playlist);
 void
 gtkui_setup_gui_refresh ();
 
-time_t
-gtkui_get_curr_playlist_modtime (void);
+int
+gtkui_get_curr_playlist_mod (void);
 
 #endif
diff --git a/plugins/gtkui/mainplaylist.c b/plugins/gtkui/mainplaylist.c
index 9c4c5877..e1537ea0 100644
--- a/plugins/gtkui/mainplaylist.c
+++ b/plugins/gtkui/mainplaylist.c
@@ -284,7 +284,7 @@ DdbListviewBinding main_binding = {
     .list_context_menu = list_context_menu,
     .delete_selected = main_delete_selected,
     .vscroll_changed = main_vscroll_changed,
-    .modification_time = gtkui_get_curr_playlist_modtime,
+    .modification_idx = gtkui_get_curr_playlist_mod,
 };
 
 void
diff --git a/plugins/gtkui/search.c b/plugins/gtkui/search.c
index 4f0eeb46..471d3620 100644
--- a/plugins/gtkui/search.c
+++ b/plugins/gtkui/search.c
@@ -369,7 +369,7 @@ DdbListviewBinding search_binding = {
     .header_context_menu = header_context_menu,
     .list_context_menu = list_context_menu,
     .delete_selected = search_delete_selected,
-    .modification_time = gtkui_get_curr_playlist_modtime,
+    .modification_idx = gtkui_get_curr_playlist_mod,
 };
 
 void
diff --git a/plugins/gtkui/trkproperties.c b/plugins/gtkui/trkproperties.c
index 5b58890c..12e51924 100644
--- a/plugins/gtkui/trkproperties.c
+++ b/plugins/gtkui/trkproperties.c
@@ -462,6 +462,7 @@ static gboolean
 write_finished_cb (void *ctx) {
     gtk_widget_destroy (progressdlg);
     progressdlg = NULL;
+    deadbeef->plt_modified (deadbeef->plt_get_handle (deadbeef->plt_get_curr ()));
     main_refresh ();
     search_refresh ();
     trkproperties_modified = 0;
diff --git a/plugins/vfs_curl/vfs_curl.c b/plugins/vfs_curl/vfs_curl.c
index 528b3809..0ef5cc6a 100644
--- a/plugins/vfs_curl/vfs_curl.c
+++ b/plugins/vfs_curl/vfs_curl.c
@@ -170,7 +170,12 @@ vfs_curl_set_meta (DB_playItem_t *it, const char *meta, const char *value) {
     uint32_t f = deadbeef->pl_get_item_flags (it);
     f |= DDB_TAG_ICY;
     deadbeef->pl_set_item_flags (it, f);
-    deadbeef->plug_trigger_event_trackinfochanged (it);
+    ddb_event_track_t *ev = (ddb_event_track_t *)deadbeef->event_alloc (DB_EV_TRACKINFOCHANGED);
+    ev->track = it;
+    if (ev->track) {
+        deadbeef->pl_item_ref (ev->track);
+    }
+    deadbeef->event_send ((ddb_event_t *)ev, 0, 0);
 }
 
 int
@@ -207,7 +212,8 @@ http_parse_shoutcast_meta (HTTP_FILE *fp, const char *meta, int size) {
                 else {
                     vfs_curl_set_meta (fp->track, "title", title);
                 }
-                deadbeef->plug_trigger_event_playlistchanged ();
+                deadbeef->plt_modified (deadbeef->plt_get_handle (deadbeef->plt_get_curr ()));
+                deadbeef->sendmessage (DB_EV_PLAYLISTCHANGED, 0, 0, 0);
             }
             return 0;
         }
@@ -462,8 +468,9 @@ http_content_header_handler (void *ptr, size_t size, size_t nmemb, void *stream)
             fp->wait_meta = fp->icy_metaint; 
         }
     }
+    deadbeef->plt_modified (deadbeef->plt_get_handle (deadbeef->plt_get_curr ()));
     if (refresh_playlist) {
-        deadbeef->plug_trigger_event_playlistchanged ();
+        deadbeef->sendmessage (DB_EV_PLAYLISTCHANGED, 0, 0, 0);
     }
     if (!fp->icyheader) {
         fp->gotsomeheader = 1;
diff --git a/plugins/vorbis/vorbis.c b/plugins/vorbis/vorbis.c
index 691e40f3..27275eec 100644
--- a/plugins/vorbis/vorbis.c
+++ b/plugins/vorbis/vorbis.c
@@ -94,7 +94,7 @@ static const char *metainfo[] = {
     NULL
 };
 
-// refresh_playlist == 1 means "call plug_trigger_event_playlistchanged if metadata had been changed"
+// refresh_playlist == 1 means "send playlistchanged event if metadata had been changed"
 // refresh_playlist == 2 means "don't change memory, just check for changes"
 static int
 update_vorbis_comments (DB_playItem_t *it, vorbis_comment *vc, int refresh_playlist) {
@@ -165,8 +165,9 @@ update_vorbis_comments (DB_playItem_t *it, vorbis_comment *vc, int refresh_playl
     f &= ~DDB_TAG_MASK;
     f |= DDB_TAG_VORBISCOMMENTS;
     deadbeef->pl_set_item_flags (it, f);
+    deadbeef->plt_modified (deadbeef->plt_get_handle (deadbeef->plt_get_curr ()));
     if (refresh_playlist) {
-        deadbeef->plug_trigger_event_playlistchanged ();
+        deadbeef->sendmessage (DB_EV_PLAYLISTCHANGED, 0, 0, 0);
     }
     return 0;
 }
@@ -320,7 +321,12 @@ cvorbis_read (DB_fileinfo_t *_info, char *bytes, int size) {
                 info->last_comment_update = info->currentsample;
                 vorbis_comment *vc = ov_comment (&info->vorbis_file, -1);
                 update_vorbis_comments (info->ptrack, vc, 1);
-                deadbeef->plug_trigger_event_trackinfochanged (info->ptrack);
+                ddb_event_track_t *ev = (ddb_event_track_t *)deadbeef->event_alloc (DB_EV_TRACKINFOCHANGED);
+                ev->track = info->ptrack;
+                if (ev->track) {
+                    deadbeef->pl_item_ref (ev->track);
+                }
+                deadbeef->event_send ((ddb_event_t *)ev, 0, 0);
             }
             else {
                 info->ptrack = NULL;