[GRASS-QA] GRASS QA: soeren | ./grass6/raster/r.le/r.le.patch/patch.c |no clones | 5 monsters | 4 warnings

grass-qa at proxy.soccerlab.polymtl.ca grass-qa at proxy.soccerlab.polymtl.ca
Mon Aug 14 14:49:24 EDT 2006



Hello soeren,

here a report on your recent GRASS-CVS change:

ANALYSIS

your changes to files/functions:


raster/r.le/r.le.patch/patch.c functions


df_patch (Begins at: 41 Ends at: 301)
--------------+---------+-------+---------+
Complexity:   |      67 |   *** | MONSTER |
ParamNBR:     |       1 |       |      OK |
CalledNBR:    |     106 |   *** | MONSTER |
LOC:          |     261 |   *** | MONSTER |
--------------+---------+-------+---------+
3 metric are OUT OF RANGE!

df_att (Begins at: 309 Ends at: 348)
--------------+---------+-------+---------+
Complexity:   |       3 |       |      OK |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       3 |       |      OK |
LOC:          |      40 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

save_att (Begins at: 356 Ends at: 430)
--------------+---------+-------+---------+
Complexity:   |       9 |       |      OK |
ParamNBR:     |       8 |    ** | Warning |
CalledNBR:    |      10 |       |      OK |
LOC:          |      75 |       |      OK |
--------------+---------+-------+---------+
at least 1 values are close to an outlier please consider revising code

df_core (Begins at: 438 Ends at: 551)
--------------+---------+-------+---------+
Complexity:   |      12 |       |      OK |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |      23 |       |      OK |
LOC:          |     114 |     * |         |
--------------+---------+-------+---------+
metrics are OK

save_core (Begins at: 553 Ends at: 686)
--------------+---------+-------+---------+
Complexity:   |      16 |     * |         |
ParamNBR:     |      13 |   *** | MONSTER |
CalledNBR:    |      23 |       |      OK |
LOC:          |     134 |     * |         |
--------------+---------+-------+---------+
1 metric are OUT OF RANGE!

df_size (Begins at: 694 Ends at: 773)
--------------+---------+-------+---------+
Complexity:   |      10 |       |      OK |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |      12 |       |      OK |
LOC:          |      80 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

save_size (Begins at: 775 Ends at: 877)
--------------+---------+-------+---------+
Complexity:   |      11 |       |      OK |
ParamNBR:     |       7 |    ** | Warning |
CalledNBR:    |      13 |       |      OK |
LOC:          |     103 |       |      OK |
--------------+---------+-------+---------+
at least 1 values are close to an outlier please consider revising code

df_shape (Begins at: 879 Ends at: 1045)
--------------+---------+-------+---------+
Complexity:   |      20 |     * |         |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |      37 |     * |         |
LOC:          |     167 |     * |         |
--------------+---------+-------+---------+
metrics are OK

save_shape (Begins at: 1047 Ends at: 1294)
--------------+---------+-------+---------+
Complexity:   |      30 |    ** | Warning |
ParamNBR:     |      19 |   *** | MONSTER |
CalledNBR:    |      36 |     * |         |
LOC:          |     248 |    ** | Warning |
--------------+---------+-------+---------+
1 metric are OUT OF RANGE!

df_boundary (Begins at: 1296 Ends at: 1357)
--------------+---------+-------+---------+
Complexity:   |       5 |       |      OK |
ParamNBR:     |       1 |       |      OK |
CalledNBR:    |       3 |       |      OK |
LOC:          |      62 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

df_perim (Begins at: 1359 Ends at: 1464)
--------------+---------+-------+---------+
Complexity:   |      12 |       |      OK |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |      13 |       |      OK |
LOC:          |     106 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

mv_patch (Begins at: 1466 Ends at: 1512)
--------------+---------+-------+---------+
Complexity:   |      11 |       |      OK |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       8 |       |      OK |
LOC:          |      47 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

recl_coh (Begins at: 1514 Ends at: 1537)
--------------+---------+-------+---------+
Complexity:   |       2 |       |      OK |
ParamNBR:     |       1 |       |      OK |
CalledNBR:    |       1 |       |      OK |
LOC:          |      24 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

in_group (Begins at: 1539 Ends at: 1595)
--------------+---------+-------+---------+
Complexity:   |       4 |       |      OK |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       0 |       |      OK |
LOC:          |      57 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

index_coh (Begins at: 1597 Ends at: 1622)
--------------+---------+-------+---------+
Complexity:   |       2 |       |      OK |
ParamNBR:     |       2 |       |      OK |
CalledNBR:    |       0 |       |      OK |
LOC:          |      26 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

m_att (Begins at: 1624 Ends at: 1755)
--------------+---------+-------+---------+
Complexity:   |      21 |     * |         |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       4 |       |      OK |
LOC:          |     132 |     * |         |
--------------+---------+-------+---------+
metrics are OK

m_size (Begins at: 1762 Ends at: 1890)
--------------+---------+-------+---------+
Complexity:   |      19 |     * |         |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       4 |       |      OK |
LOC:          |     129 |     * |         |
--------------+---------+-------+---------+
metrics are OK

m_core (Begins at: 1892 Ends at: 2076)
--------------+---------+-------+---------+
Complexity:   |      32 |    ** | Warning |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       7 |       |      OK |
LOC:          |     185 |    ** | Warning |
--------------+---------+-------+---------+
at least 2 values are close to an outlier please consider revising code

m_shape (Begins at: 2078 Ends at: 2211)
--------------+---------+-------+---------+
Complexity:   |      24 |     * |         |
ParamNBR:     |       4 |       |      OK |
CalledNBR:    |       6 |       |      OK |
LOC:          |     134 |     * |         |
--------------+---------+-------+---------+
metrics are OK

m_boundary (Begins at: 2213 Ends at: 2296)
--------------+---------+-------+---------+
Complexity:   |      10 |       |      OK |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       2 |       |      OK |
LOC:          |      84 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

m_perim (Begins at: 2298 Ends at: 2388)
--------------+---------+-------+---------+
Complexity:   |      16 |     * |         |
ParamNBR:     |       3 |       |      OK |
CalledNBR:    |       3 |       |      OK |
LOC:          |      91 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

eu_d (Begins at: 2392 Ends at: 2396)
--------------+---------+-------+---------+
Complexity:   |       1 |       |      OK |
ParamNBR:     |       4 |       |      OK |
CalledNBR:    |       1 |       |      OK |
LOC:          |       5 |       |      OK |
--------------+---------+-------+---------+
metrics are OK

Clone analysis:

This file does not contain  cloned functions

__________________________________________

Threshold GRASS6 key metrics values at 2006-08-14 are:
---------------------------+-----------+---------+
                    median | upper     | outlier |
                           | quartile  | limit   |
---------------------------+-----------+---------+
Complexity (Cyclo):      5 |        25 |      36 |
ParamNBR:                2 |         6 |       8 |
CalledNBR:               9 |        55 |      80 |
LOC:                    39 |       179 |     251 |
---------------------------+-----------+---------+

Measures (see below Web page for details):
 Complexity (Cyclo): a complexity index
 ParamNBR: number of passed parameters
 CalledNBR: number of called functions
 LOC: Lines Of Code

Message interpretation:
 OK : Nothing to say at all
 *  : above 75 % of value warning
 ** : this is getting close to be out of range, consider revising the code
 ***: this is a monster, definitely you should revise this function


__________________________________________
Brought to you by:
SOftware Cost-effective Change and Evolution Research (SOCCER) laboratory
http://web.soccerlab.polymtl.ca/grass-evolution/grass-browsers/grass-index-en.html




More information about the grass-qa mailing list