SVN - 审查变更

杰瑞已将array.c文件添加到版本库。汤姆也检出了最新代码并开始工作。

[tom@CentOS ~]$ svn co http://svn.server.com/svn/project_repo --username=tom

上述命令将产生以下结果。

A    project_repo/trunk
A    project_repo/trunk/array.c
A    project_repo/branches
A    project_repo/tags
Checked out revision 2.

但是，他发现有人已经添加了代码。于是他好奇地想知道是谁这么做的，并使用以下命令检查日志消息以查看更多详细信息

[tom@CentOS trunk]$ svn log

上述命令将产生以下结果。

------------------------------------------------------------------------
r2 | jerry | 2013-08-17 20:40:43 +0530 (Sat, 17 Aug 2013) | 1 line

Initial commit
------------------------------------------------------------------------
r1 | jerry | 2013-08-04 23:43:08 +0530 (Sun, 04 Aug 2013) | 1 line

Create trunk, branches, tags directory structure
------------------------------------------------------------------------

当汤姆观察杰瑞的代码时，他立即注意到其中存在一个错误。杰瑞没有检查数组溢出，这可能会导致严重问题。因此，汤姆决定解决此问题。修改后，array.c将如下所示。

#include <stdio.h>

#define MAX 16

int main(void)
{
   int i, n, arr[MAX];

   printf("Enter the total number of elements: ");
   scanf("%d", &n);

   /* handle array overflow condition */
   if (n > MAX) {
      fprintf(stderr, "Number of elements must be less than %d\n", MAX);
      return 1;
   }

   printf("Enter the elements\n");

   for (i = 0; i < n; ++i)
      scanf("%d", &arr[i]);

   printf("Array has following elements\n");
   for (i = 0; i < n; ++i)
      printf("|%d| ", arr[i]);
      printf("\n");

   return 0;
}

汤姆希望使用状态操作查看待处理的变更列表。

[tom@CentOS trunk]$ svn status
M       array.c

array.c文件已被修改，因此Subversion在文件名之前显示了字母M。接着，汤姆编译并测试了他的代码，效果很好。在提交更改之前，他想通过审查他所做的更改来仔细检查一下。

[tom@CentOS trunk]$ svn diff
Index: array.c
===================================================================
--- array.c   (revision 2)
+++ array.c   (working copy)
@@ -9,6 +9,11 @@
    printf("Enter the total number of elements: ");
    scanf("%d", &n);
 
+   if (n > MAX) {
+      fprintf(stderr, "Number of elements must be less than %d\n", MAX);
+      return 1;
+   }
+
    printf("Enter the elements\n");
 
    for (i = 0; i < n; ++i)

汤姆在array.c文件中添加了几行，因此Subversion在每一行之前都显示了 + 符号。现在他已经准备提交他的更改。

[tom@CentOS trunk]$ svn commit -m "Fix array overflow problem"

上述命令将产生以下结果。

Sending        trunk/array.c
Transmitting file data .
Committed revision 3.

汤姆的更改已成功提交到版本库。