Changeset 16271

Show
Ignore:
Timestamp:
04/28/08 14:27:21 (7 months ago)
Author:
piotras
Message:

Count rows only in get_model

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • trunk/midgard/core/midgard/src/midgard_collector.c

    r16074 r16271  
    633633        const GValue *key_value; 
    634634        GValue *ck_value; 
    635          
    636         ret_rows = gda_data_model_get_n_rows(model); 
    637  
    638         if(ret_rows == 0) { 
    639                 g_object_unref(model); 
    640                 return FALSE; 
    641         } 
    642635 
    643636        for (rows = 0; rows < ret_rows; rows++) { 
    …… 
    657650                        //g_value_init(new_value, G_TYPE_STRING); 
    658651                        g_value_copy(gda_value, new_value); 
    659  
    660652 
    661653                        ck_value = g_new0(GValue, 1); 
    …… 
    689681 
    690682                        g_value_unset(ck_value); 
     683                        g_value_unset(new_value); 
    691684                        g_free(ck_value); 
    692685                } 

  • trunk/midgard/core/midgard/src/midgard_core_query.c

    r16270 r16271  
    110110        gda_command_free(command); 
    111111 
     112        /* This is query error */ 
     113        if(!model) 
     114                return NULL; 
     115 
    112116        gint rows = gda_data_model_get_n_rows(model); 
    113117 
     118        /* Check if there's at least one row */ 
    114119        if(rows < 1) { 
    115120         
    …… 
    142147                midgard_core_query_get_model(mgd, query); 
    143148        g_free(query); 
    144          
    145         if(gda_data_model_get_n_rows(model) == 0){ 
    146  
    147                 g_object_unref(model); 
     149 
     150        if(!model) 
    148151                return NULL; 
    149  
    150         } 
    151152 
    152153        const GValue *value = 

  • trunk/midgard/core/midgard/src/midgard_quota.c

    r15473 r16271  
    4343        GdaDataModel *model =  
    4444                midgard_core_query_get_model(mgd, query->str); 
    45         rows = gda_data_model_get_n_rows(model); 
     45 
    4646        g_string_free(query, TRUE); 
    47          
     47 
    4848        /* Sitegroup limit is not set */ 
    49         if(rows == 0) { 
    50                 g_object_unref(model); 
     49        if(!model) 
    5150                return FALSE; 
    52         } 
    5351         
    5452        limit_tmp_size_value = gda_data_model_get_value_at(model, 0, 0);