Changeset 10642


Ignore:
Timestamp:
May 12, 2010, 1:34:08 AM (11 years ago)
Author:
charles
Message:

(trunk gtk) minor code cleanup to the file list

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/gtk/file-list.c

    r10173 r10642  
    4343    FC_PRIORITY,
    4444    FC_ENABLED,
    45     FC_IS_FILE,
    4645    N_FILE_COLS
    4746};
     
    9089{
    9190    FileData * data = gdata;
    92     gboolean is_file;
    9391    unsigned int index;
    9492    uint64_t size;
     
    9795    int old_priority;
    9896    int old_enabled;
    99 
    100     gtk_tree_model_get( model, iter, FC_IS_FILE, &is_file,
    101                                     FC_ENABLED, &old_enabled,
     97    const gboolean is_file = !gtk_tree_model_iter_has_child( model, iter );
     98
     99    gtk_tree_model_get( model, iter, FC_ENABLED, &old_enabled,
    102100                                     FC_PRIORITY, &old_priority,
    103101                                     FC_INDEX, &index,
     
    254252                         gpointer       gdata )
    255253{
    256     struct ActiveData * data = gdata;
    257     unsigned int        i;
    258     gboolean            is_file = FALSE;
    259     gboolean            is_active = FALSE;
    260 
    261     /* active == if it's selected, or any ancestor is selected */
    262     gtk_tree_model_get( model, iter, FC_IS_FILE, &is_file,
    263                                      FC_INDEX, &i,
    264                                      -1 );
     254    const gboolean is_file = !gtk_tree_model_iter_has_child( model, iter );
     255
    265256    if( is_file )
    266257    {
    267         is_active = gtk_tree_selection_iter_is_selected( data->sel, iter );
     258        struct ActiveData * data = gdata;
     259   
     260        /* active means: if it's selected or any ancestor is selected */
     261
     262        gboolean is_active = gtk_tree_selection_iter_is_selected( data->sel, iter );
     263
    268264        if( !is_active )
    269265        {
     
    276272            }
    277273        }
    278     }
    279 
    280     if( is_active )
    281         g_array_append_val( data->array, i );
     274
     275        if( is_active )
     276        {
     277            unsigned int i;
     278            gtk_tree_model_get( model, iter, FC_INDEX, &i, -1 );
     279            g_array_append_val( data->array, i );
     280        }
     281    }
    282282
    283283    return FALSE; /* keep walking */
     
    307307                   gpointer         gdata )
    308308{
    309     struct SubtreeForeachData * data = gdata;
    310     unsigned int i;
    311     gboolean is_file = FALSE;
    312 
    313     gtk_tree_model_get( model, iter, FC_IS_FILE, &is_file,
    314                                      FC_INDEX, &i,
    315                                      -1 );
     309    const gboolean is_file = !gtk_tree_model_iter_has_child( model, iter );
     310
    316311    if( is_file )
     312    {
     313        struct SubtreeForeachData * data = gdata;
     314
    317315        if( !gtk_tree_path_compare( path, data->path ) || gtk_tree_path_is_descendant( path, data->path ) )
     316        {
     317            unsigned int i;
     318            gtk_tree_model_get( model, iter, FC_INDEX, &i, -1 );
    318319            g_array_append_val( data->array, i );
     320        }
     321    }
    319322
    320323    return FALSE; /* keep walking */
     
    400403                                       FC_PRIORITY, priority,
    401404                                       FC_ENABLED, enabled,
    402                                        FC_IS_FILE, isLeaf,
    403405                                       -1 );
    404406#else
     
    411413                        FC_PRIORITY, priority,
    412414                        FC_ENABLED, enabled,
    413                         FC_IS_FILE, isLeaf,
    414415                        -1 );
    415416#endif
     
    460461                                 G_TYPE_UINT64,    /* have */
    461462                                 G_TYPE_INT,       /* priority */
    462                                  G_TYPE_INT,       /* dl enabled */
    463                                  G_TYPE_BOOLEAN,   /* is file */
    464                                  G_TYPE_UINT64,    /* sub size */
    465                                  G_TYPE_UINT64,    /* sub have */
    466                                  G_TYPE_INT );     /* sub state */
     463                                 G_TYPE_INT );     /* dl enabled */
    467464
    468465    data->store = store;
Note: See TracChangeset for help on using the changeset viewer.