BoomMicrophone/nixpkgs-suyu
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

totem: add 3.12.1 version

Browse source
This commit is contained in:
Luca Bruno 2014-05-18 23:21:50 +02:00
parent a1dbe6ba43
commit 2708d9798a
2 changed files with 77 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
pkgs/desktops/gnome-3/3.12/core/totem/default.nix
View file

@ -5,11 +5,11 @@
, gnome3, librsvg, gdk_pixbuf, file }:
stdenv.mkDerivation rec {
name = "totem-3.10.1";
name = "totem-3.12.1";
src = fetchurl {
url = "mirror://gnome/sources/totem/3.10/${name}.tar.xz";
sha256 = "b6b6038c9104965671a6d25e98496a487c3a9c590c9c104f668bd9f4fa7be9e2";
url = "mirror://gnome/sources/totem/3.12/${name}.tar.xz";
sha256 = "dd168cdd4051d01131d47c24fa45bfd08b6ccf45900ac4b64bae47f6f47a35e3";
};
doCheck = true;
@ -39,6 +39,8 @@ stdenv.mkDerivation rec {
rm $out/share/icons/hicolor/icon-theme.cache
'';
patches = [ ./fix_files_loss.patch ];
meta = with stdenv.lib; {
homepage = https://wiki.gnome.org/Apps/Videos;
description = "Movie player for the GNOME desktop based on GStreamer";

72
pkgs/desktops/gnome-3/3.12/core/totem/fix_files_loss.patch Normal file
View file

@ -0,0 +1,72 @@
From 524c9e39a0ca6f2d1699e6e6d9ba3db1ea80d9f9 Mon Sep 17 00:00:00 2001
From: Bastien Nocera <hadess@hadess.net>
Date: Thu, 15 May 2014 19:28:35 +0200
Subject: main: Fix potential data loss when removing multiple files
We were using a list of GtkTreePaths and deleting the items one-by-one
when the user clicked the "Delete" button. But after the first deletion,
the tree model would have changed, and instead of pointing to the 2nd
item we wanted to delete, the GtkTreePath would have pointed to another
item, usually not one that we wanted to delete.
We now use GtkTreeRowReferences, which will always point to the same
row, as long as it exists.
https://bugzilla.gnome.org/show_bug.cgi?id=729778
diff --git a/src/totem-grilo.c b/src/totem-grilo.c
index 2133d77..0cff0d6 100644
--- a/src/totem-grilo.c
+++ b/src/totem-grilo.c
@@ -2074,7 +2074,8 @@ static void
delete_foreach (gpointer data,
gpointer user_data)
{
- GtkTreePath *path = data;
+ GtkTreeRowReference *ref = data;
+ GtkTreePath *path;
GtkTreeModel *view_model = user_data;
GtkTreeIter iter;
GrlSource *source;
@@ -2085,8 +2086,11 @@ delete_foreach (gpointer data,
GtkTreeModel *model;
GtkTreeIter real_model_iter;
- if (!gtk_tree_model_get_iter (view_model, &iter, path))
+ path = gtk_tree_row_reference_get_path (ref);
+ if (!path || !gtk_tree_model_get_iter (view_model, &iter, path)) {
+ g_warning ("An item that was scheduled for removal isn't available any more");
return;
+ }
gtk_tree_model_get (view_model, &iter,
MODEL_RESULTS_CONTENT, &media,
@@ -2151,14 +2155,23 @@ delete_cb (TotemSelectionToolbar *bar,
TotemGrilo *self)
{
GtkTreeModel *model;
- GList *list;
+ GList *list, *l;
g_signal_handlers_block_by_func (self->priv->browser, "view-selection-changed", self);
model = gd_main_view_get_model (GD_MAIN_VIEW (self->priv->browser));
list = gd_main_view_get_selection (GD_MAIN_VIEW (self->priv->browser));
+
+ /* GList of GtkTreePaths to a GList of GtkTreeRowReferences */
+ for (l = list; l != NULL; l = l->next) {
+ GtkTreeRowReference *ref;
+
+ ref = gtk_tree_row_reference_new (model, l->data);
+ gtk_tree_path_free (l->data);
+ l->data = ref;
+ }
g_list_foreach (list, delete_foreach, model);
- g_list_free_full (list, (GDestroyNotify) gtk_tree_path_free);
+ g_list_free_full (list, (GDestroyNotify) gtk_tree_row_reference_free);
g_signal_handlers_unblock_by_func (self->priv->browser, "view-selection-changed", self);
--
cgit v0.10.1