Changeset 5536

Timestamp:

Apr 6, 2008, 2:07:41 PM (14 years ago)

Author:

charles

Message:

(gtk) #745: sort transfers by ratio

Location:

trunk/gtk

Files:

• actions.c (modified) (1 diff)
• makemeta-ui.c (modified) (1 diff)
• tr-core.c (modified) (3 diffs)
• tr-prefs.c (modified) (1 diff)
• ui.h (modified) (1 diff)

trunk/gtk/actions.c

@@ -48 +48 @@
   { "sort-by-name",       NULL, N_("Sort by _Name"),       NULL, NULL, 1 },
   { "sort-by-progress",   NULL, N_("Sort by _Progress"),   NULL, NULL, 2 },
-  { "sort-by-state",      NULL, N_("Sort by _State"),      NULL, NULL, 3 },
-  { "sort-by-tracker",    NULL, N_("Sort by _Tracker"),    NULL, NULL, 4 }
+  { "sort-by-ratio",      NULL, N_("Sort by _Ratio"),      NULL, NULL, 3 },
+  { "sort-by-state",      NULL, N_("Sort by _State"),      NULL, NULL, 4 },
+  { "sort-by-tracker",    NULL, N_("Sort by _Tracker"),    NULL, NULL, 5 }
 };
 

trunk/gtk/makemeta-ui.c

@@ -115 +115 @@
 
             case TR_MAKEMETA_IO_WRITE: {
-                char * tmp = g_strdup_printf( _( "Couldn't write \"%s\": %s" ), ui->builder->errfile, g_strerror( ui->builder->my_errno ) );
+                char * tmp = g_strdup_printf( _( "Couldn't create \"%1$s\": %2$s" ), ui->builder->errfile, g_strerror( ui->builder->my_errno ) );
                 txt = g_strdup_printf( _( "Torrent creation failed: %s" ), tmp );
                 g_free( tmp  );

trunk/gtk/tr-core.c

@@ -168 +168 @@
 
 static int
+compareRatio( double a, double b )
+{
+    if( (int)a == TR_RATIO_INF && (int)b == TR_RATIO_INF ) return 0;
+    if( (int)a == TR_RATIO_INF ) return 1;
+    if( (int)b == TR_RATIO_INF ) return -1;
+    return compareDouble( a, b );
+}
+
+static int
+compareByRatio( GtkTreeModel * model,
+                GtkTreeIter  * a,
+                GtkTreeIter  * b,
+                gpointer       user_data UNUSED )
+{
+    tr_torrent *ta, *tb;
+    const tr_stat *sa, *sb;
+
+    gtk_tree_model_get( model, a, MC_TORRENT_RAW, &ta, -1 );
+    gtk_tree_model_get( model, b, MC_TORRENT_RAW, &tb, -1 );
+
+    sa = tr_torrentStatCached( ta );
+    sb = tr_torrentStatCached( tb );
+
+    return compareRatio( sa->ratio, sb->ratio );
+}
+
+static int
 compareByActivity( GtkTreeModel * model,
                    GtkTreeIter  * a,
@@ -224 +251 @@
     ret = compareDouble( sa->percentDone, sb->percentDone );
     if( !ret )
-        ret = compareDouble( sa->ratio, sb->ratio );
+        ret = compareRatio( sa->ratio, sb->ratio );
     return ret;
 }
@@ -264 +291 @@
 setSort( TrCore * core, const char * mode, gboolean isReversed  )
 {
-    int col = MC_TORRENT_RAW;
+    const int col = MC_TORRENT_RAW;
+    GtkTreeIterCompareFunc sort_func;
     GtkSortType type = isReversed ? GTK_SORT_ASCENDING : GTK_SORT_DESCENDING;
-    GtkTreeModel * model = tr_core_model( core );
-    GtkTreeSortable * sortable = GTK_TREE_SORTABLE( model );
+    GtkTreeSortable * sortable = GTK_TREE_SORTABLE( tr_core_model( core )  );
 
     if( !strcmp( mode, "sort-by-activity" ) )
-        gtk_tree_sortable_set_sort_func( sortable, col, compareByActivity, NULL, NULL );
+        sort_func = compareByActivity;
     else if( !strcmp( mode, "sort-by-progress" ) )
-        gtk_tree_sortable_set_sort_func( sortable, col, compareByProgress, NULL, NULL );
+        sort_func = compareByProgress;
+    else if( !strcmp( mode, "sort-by-ratio" ) )
+        sort_func = compareByRatio;
     else if( !strcmp( mode, "sort-by-state" ) )
-        gtk_tree_sortable_set_sort_func( sortable, col, compareByState, NULL, NULL );
+        sort_func = compareByState;
     else if( !strcmp( mode, "sort-by-tracker" ) )
-        gtk_tree_sortable_set_sort_func( sortable, col, compareByTracker, NULL, NULL );
+        sort_func = compareByTracker;
     else {
+        sort_func = compareByName;
         type = isReversed ? GTK_SORT_DESCENDING : GTK_SORT_ASCENDING;
-        gtk_tree_sortable_set_sort_func( sortable, col, compareByName, NULL, NULL );
-    }
-
+    }
+ 
+    gtk_tree_sortable_set_sort_func( sortable, col, sort_func, NULL, NULL );
     gtk_tree_sortable_set_sort_column_id( sortable, col, type );
 }

trunk/gtk/tr-prefs.c

@@ -462 +462 @@
         hig_workarea_add_wide_control( t, &row, h );
         
-        s = _("Ignore _unencrypted peers");
+        s = _("_Ignore unencrypted peers");
         w = new_check_button( s, PREF_KEY_ENCRYPTED_ONLY, core );
         hig_workarea_add_wide_control( t, &row, w );

trunk/gtk/ui.h

@@ -33 +33 @@
 "      <menuitem action='sort-by-name'/>\n"
 "      <menuitem action='sort-by-progress'/>\n"
+"      <menuitem action='sort-by-ratio'/>\n"
 "      <menuitem action='sort-by-state'/>\n"
 "      <menuitem action='sort-by-tracker'/>\n"