如何简单计算patch代码变化量?

最新推荐文章于 2022-04-28 08:52:34 发布

lida2003

最新推荐文章于 2022-04-28 08:52:34 发布

阅读量2.8k

点赞数

分类专栏： Linux 文章标签： patch debugging 脚本 report download 数据分析

本文链接：https://blog.csdn.net/lida2003/article/details/6971653

版权

Linux 专栏收录该内容

191 篇文章 12 订阅

订阅专栏

计算代码量从程序员的角度来说，真是个变态的想法。从系统集成的角度来看，模块代码量越大，显然出错的概率越高，可能需要更多的检视或者测试。

但是很多项目上，很多项目经理，或者管理人员比较喜欢算代码量，个人角度非常不赞成这种变态的绩效参考标准。但是统计下patch代码量从某个角度来看，可以看出代码变化的情况，也能说明很多设计上的问题。从敏捷的角度看，说明每次迭代导致的设计变更情况。

这里写个简单的脚本来回答这个问题：如何简单计算patch代码变化量？请注意，这里说patch代码变化量，刚开始我想用“如何简单计算patch代码工作量？”真是上了那些变态绩效的怪圈了。

好了，说了这么多，我们进入正题，如何简单实现patch代码变化量的统计？

首先，分析下patch里面哪些是与代码变化量相关的？这里我们截取u-boot-1.1.6_100ask24x0.patch中的一段

diff -urN -w u-boot-1.1.6/board/100ask24x0/config.mk u-boot-1.1.6_100ask24x0/board/100ask24x0/config.mk
--- u-boot-1.1.6/board/100ask24x0/config.mk	1970-01-01 08:00:00.000000000 +0800
+++ u-boot-1.1.6_100ask24x0/board/100ask24x0/config.mk	2007-09-08 19:14:43.000000000 +0800
@@ -0,0 +1,25 @@
+#
+# (C) Copyright 2002
+# Gary Jennejohn, DENX Software Engineering, <gj@denx.de>
+# David Mueller, ELSOFT AG, <d.mueller@elsoft.ch>
+#
+# SAMSUNG SMDK2410 board with S3C2410X (ARM920T) cpu
+#
+# see http://www.samsung.com/ for more information on SAMSUNG
+#
+
+#
+# SMDK2410 has 1 bank of 64 MB DRAM
+#
+# 3000'0000 to 3400'0000
+#
+# Linux-Kernel is expected to be at 3000'8000, entry 3000'8000
+# optionally with a ramdisk at 3080'0000
+#
+# we load ourself to 33F8'0000
+#
+# download area is 3300'0000
+#
+
+
+TEXT_BASE = 0x33F80000

从上面这段代码中可以看出真正与统计相关的只有一行，只要理解了这行的含义，一个patch文件里面所有相关统计行的数据分析整理清楚，就能回答我们的问题。

@@ -0,0 +1,25 @@

然后，我们看下这行统计数据究竟是什么意思？

"-" 代表需要删除的

"+" 代表需要增加的

-0,0 代表需要删除部分从源文件0行开始，一共0行

+1,25 代表需要增加部分从目标文件第一行开始，一共25行

到这里，我们应该已经基本上了解了这些统计数据的意思的。但是这里只是简单的统计数据，详细的数据需要从代码行的粒度来分析哪些行是真正需要添加，哪些是真正需要删除的。我们这里只是简单计算，对于详细的统计数据可以参考文献【2】

根据上面我们理解的统计信息，我们给出linux的脚本文件，对这些数据进行计算分析，这里对u-boot-1.1.6_100ask24x0.patch进行测试：

对于一些特殊格式的patch，统计的时候，脚本只是简单的跳过，以内核补丁3.2-rc4为例：（如果有朋友知道这个格式是什么意思，如何理解，请告知，本人将万分感谢 ^_^）

daniel@ubuntu:~/work/tmp$ cat patch-3.2-rc4 |grep "@@" |./patch_statistic.sh
Welcome to use ./patch_statistic.sh script. -- Daniel Li
Please issuing the following command to analyze your patch file.
 xxx.patch |grep @@ |./patch_statistic.sh [30] {debug}
WARNING: Default comment ratios is 30%
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ debug using ARM EmbeddedICE DCC channel
WARNING: [skip] @@ -1,120 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -1,74 +1 @@
WARNING: [skip] @@ -1 +1 @@
WARNING: [skip] @@ -1,26 +1 @@
WARNING: [skip] @@ -1 +1 @@
WARNING: [skip] @@ -1,35 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -1,6 +1 @@
WARNING: [skip] @@ -1,74 +1 @@
WARNING: [skip] @@ -1,98 +1 @@
WARNING: [skip] @@ -1,60 +1 @@
WARNING: [skip] @@ -1,294 +1 @@
WARNING: [skip] @@ -1,6 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -1,6 +1 @@
WARNING: [skip] @@ -1,39 +1 @@
WARNING: [skip] @@ -1,24 +1 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] -       /* @@@ Force down.  */
WARNING: [skip] -//@@@0xff00, 0x0600, OP_EQ, 18, S1_TCPSQ, 0, S1_ESP6,  LD_LEN, 0x03f,  1, 0x0,                                      0xffff,
WARNING: [skip] -//@@@0x00ff, 0x0032, OP_EQ,  0, S1_CLNP2, 0, S1_AH6, IM_CTL, 0x021, 1,  0x0, 0x                                     ffff,
WARNING: [skip] -//@@@0x00ff, 0x0033, OP_EQ,  0, S1_CLNP2, 0, S1_CLNP, IM_CTL, 0x021, 1,  0x0, 0                                     xffff,
WARNING: [skip] +       /* @@@ Force down.  */
WARNING: [skip] +//@@@0xff00, 0x0600, OP_EQ, 18, S1_TCPSQ, 0, S1_ESP6,  LD_LEN, 0x03f,  1, 0x0,                                      0xffff,
WARNING: [skip] +//@@@0x00ff, 0x0032, OP_EQ,  0, S1_CLNP2, 0, S1_AH6, IM_CTL, 0x021, 1,  0x0, 0x                                     ffff,
WARNING: [skip] +//@@@0x00ff, 0x0033, OP_EQ,  0, S1_CLNP2, 0, S1_CLNP, IM_CTL, 0x021, 1,  0x0, 0                                     xffff,
WARNING: [skip] +/* @@@ Poll the SFP moddef0 line too.  */
WARNING: [skip] +/* @@@ Try to use the pm3386 DOOL interrupt as well.  */
WARNING: [skip] +                       /* @@@ Should report autonegotiation status.  */
WARNING: [skip] +               /* @@@ Flush out pending packets.  */
WARNING: [skip] +               /* @@@ Count drops.  */
WARNING: [skip] +               /* @@@ Make FCS stripping configurable.  */
WARNING: [skip] +               /* @@@ Check whether entries come back in order.  */
WARNING: [skip] +       /* @@@ Maybe we shouldn't be preallocating TX buffers.  */
WARNING: [skip] +       /* @@@ Flush out pending packets.  */
WARNING: [skip] +        * for more information.  (@@@ We also need to issue a reset
WARNING: [skip] +       /* @@@ Should set 0x337/0x437 (RX forwarding threshold.)  */
WARNING: [skip] +       /* @@@ Add other stats.  */
WARNING: [skip] -/* @@@ Poll the SFP moddef0 line too.  */
WARNING: [skip] -/* @@@ Try to use the pm3386 DOOL interrupt as well.  */
WARNING: [skip] -                       /* @@@ Should report autonegotiation status.  */
WARNING: [skip] -               /* @@@ Flush out pending packets.  */
WARNING: [skip] -               /* @@@ Count drops.  */
WARNING: [skip] -               /* @@@ Make FCS stripping configurable.  */
WARNING: [skip] -               /* @@@ Check whether entries come back in order.  */
WARNING: [skip] -       /* @@@ Maybe we shouldn't be preallocating TX buffers.  */
WARNING: [skip] -       /* @@@ Flush out pending packets.  */
WARNING: [skip] -        * for more information.  (@@@ We also need to issue a reset
WARNING: [skip] -       /* @@@ Should set 0x337/0x437 (RX forwarding threshold.)  */
WARNING: [skip] -       /* @@@ Add other stats.  */
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -1 +0,0 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] @@ -0,0 +1 @@
WARNING: [skip] -       BH_JWrite,              /* Being written to log (@@@ DEBUGGING) */
WARNING: [skip] -       BH_JWrite,              /* Being written to log (@@@ DEBUGGING) */
WARNING: [skip] +       BH_JWrite,              /* Being written to log (@@@ DEBUGGING) */
WARNING: [skip] $realfile =~ s@^([^/]*)/@@;
WARNING: [skip] $realfile =~ s@^([^/]*)/@@;


RESULT:
ratio of actual code is .7
total 224684 added, 1688741 modified.
actual line 157278.8 added, 1688741 modified, 67405.2 comments.

【1】Unix Patch

【2】用Diff和Patch工具维护源码

【3】脚本下载链接