[Cvs-cvs] ccvs/src ChangeLog fileattr.c

[Derek Robert Price]

CVSROOT:        /cvsroot/cvs
Module name:    ccvs
Changes by:     Derek Robert Price <dprice>     08/09/17 16:29:24

Modified files:
        src            : ChangeLog fileattr.c 

Log message:
        * fileattr.c: Some cleanup.
        (fileattr_set): New assertion on global.

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/ccvs/src/ChangeLog?cvsroot=cvs&r1=1.3594&r2=1.3595
http://cvs.savannah.gnu.org/viewcvs/ccvs/src/fileattr.c?cvsroot=cvs&r1=1.41&r2=1.42

Patches:
Index: ChangeLog
===================================================================
RCS file: /cvsroot/cvs/ccvs/src/ChangeLog,v
retrieving revision 1.3594
retrieving revision 1.3595
diff -u -b -r1.3594 -r1.3595
--- ChangeLog   17 Sep 2008 16:06:13 -0000      1.3594
+++ ChangeLog   17 Sep 2008 16:29:23 -0000      1.3595
@@ -1,5 +1,8 @@
 2008-09-17  Derek R. Price  <address@hidden>
 
+       * fileattr.c: Some cleanup.
+       (fileattr_set): New assertion on global.
+
        * diff.c (diff_fileproc, diff_dirproc): Some cleanup.
 
        * entries.h: "rcs.h" already includes "hash.h".

Index: fileattr.c
===================================================================
RCS file: /cvsroot/cvs/ccvs/src/fileattr.c,v
retrieving revision 1.41
retrieving revision 1.42
diff -u -b -r1.41 -r1.42
--- fileattr.c  12 Sep 2008 19:55:26 -0000      1.41
+++ fileattr.c  17 Sep 2008 16:29:24 -0000      1.42
@@ -72,7 +72,7 @@
 static void
 fileattr_delproc (Node *node)
 {
-    assert (node->data != NULL);
+    assert (node->data);
     free (node->data);
     node->data = NULL;
 }
@@ -93,7 +93,7 @@
 
     /* If NULL was passed to fileattr_startdir, then it isn't kosher to look
        at attributes.  */
-    assert (fileattr_stored_repos != NULL);
+    assert (fileattr_stored_repos);
 
     fname = Xasprintf ("%s/%s", fileattr_stored_repos, CVSREP_FILEATTR);
 
@@ -341,6 +341,8 @@
     Node *node;
     char *p;
 
+    assert (fileattr_stored_repos);
+
     if (filename == NULL)
     {
        p = fileattr_modify (fileattr_default_attrs, attrname, attrval,
@@ -588,7 +590,7 @@
 
     /* If NULL was passed to fileattr_startdir, then it isn't kosher to set
        attributes.  */
-    assert (fileattr_stored_repos != NULL);
+    assert (fileattr_stored_repos);
 
     fname = Xasprintf ("%s/%s", fileattr_stored_repos, CVSREP_FILEATTR);