[gentoo-commits] gentoo-projects commit in portage-utils: .depend qcheck.c

[Mike Frysinger (vapier)]

vapier      11/10/03 16:18:25

  Modified:             .depend qcheck.c
  Log:
  use array helpers from pax-utils

Revision  Changes    Path
1.45                 portage-utils/.depend

file : http://sources.gentoo.org/viewvc.cgi/gentoo-projects/portage-utils/.depend?rev=1.45&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-projects/portage-utils/.depend?rev=1.45&content-type=text/plain
diff : http://sources.gentoo.org/viewvc.cgi/gentoo-projects/portage-utils/.depend?r1=1.44&r2=1.45

Index: .depend
===================================================================
RCS file: /var/cvsroot/gentoo-projects/portage-utils/.depend,v
retrieving revision 1.44
retrieving revision 1.45
diff -u -r1.44 -r1.45
--- .depend	3 Oct 2011 01:25:54 -0000	1.44
+++ .depend	3 Oct 2011 16:18:25 -0000	1.45
@@ -3,8 +3,8 @@
  libq/hash_fd.c libq/md5_sha1_sum.c libq/human_readable.c libq/rmspace.c \
  libq/which.c libq/compat.c libq/copy_file.c libq/safe_io.c libq/xchdir.c \
  libq/xgetcwd.c libq/xmkdir.c libq/xreadlink.c libq/xregex.c \
- libq/xsystem.c libq/atom_explode.c libq/atom_compare.c libq/basename.c \
- libq/prelink.c libq/profile.c libq/vdb_get_next_dir.c libq/virtuals.c \
- applets.h include_applets.h q.c qcheck.c qdepends.c qfile.c qlist.c \
- qlop.c qsearch.c qsize.c qtbz2.c quse.c qxpak.c qpkg.c qgrep.c qatom.c \
- qmerge.c qcache.c qglsa.c
+ libq/xsystem.c libq/xarray.c libq/atom_explode.c libq/atom_compare.c \
+ libq/basename.c libq/prelink.c libq/profile.c libq/vdb_get_next_dir.c \
+ libq/virtuals.c applets.h include_applets.h q.c qcheck.c qdepends.c \
+ qfile.c qlist.c qlop.c qsearch.c qsize.c qtbz2.c quse.c qxpak.c qpkg.c \
+ qgrep.c qatom.c qmerge.c qcache.c qglsa.c



1.51                 portage-utils/qcheck.c

file : http://sources.gentoo.org/viewvc.cgi/gentoo-projects/portage-utils/qcheck.c?rev=1.51&view=markup
plain: http://sources.gentoo.org/viewvc.cgi/gentoo-projects/portage-utils/qcheck.c?rev=1.51&content-type=text/plain
diff : http://sources.gentoo.org/viewvc.cgi/gentoo-projects/portage-utils/qcheck.c?r1=1.50&r2=1.51

Index: qcheck.c
===================================================================
RCS file: /var/cvsroot/gentoo-projects/portage-utils/qcheck.c,v
retrieving revision 1.50
retrieving revision 1.51
diff -u -r1.50 -r1.51
--- qcheck.c	3 Oct 2011 03:18:10 -0000	1.50
+++ qcheck.c	3 Oct 2011 16:18:25 -0000	1.51
@@ -1,7 +1,7 @@
 /*
  * Copyright 2005-2010 Gentoo Foundation
  * Distributed under the terms of the GNU General Public License v2
- * $Header: /var/cvsroot/gentoo-projects/portage-utils/qcheck.c,v 1.50 2011/10/03 03:18:10 vapier Exp $
+ * $Header: /var/cvsroot/gentoo-projects/portage-utils/qcheck.c,v 1.51 2011/10/03 16:18:25 vapier Exp $
  *
  * Copyright 2005-2010 Ned Ludd        - <solar@g.o>
  * Copyright 2005-2010 Mike Frysinger  - <vapier@g.o>
@@ -34,30 +34,16 @@
 	"Undo prelink when calculating checksums",
 	COMMON_OPTS_HELP
 };
-static const char qcheck_rcsid[] = "$Id: qcheck.c,v 1.50 2011/10/03 03:18:10 vapier Exp $";
+static const char qcheck_rcsid[] = "$Id: qcheck.c,v 1.51 2011/10/03 16:18:25 vapier Exp $";
 #define qcheck_usage(ret) usage(ret, QCHECK_FLAGS, qcheck_long_opts, qcheck_opts_help, lookup_applet_idx("qcheck"))
 
 static bool bad_only = false;
 #define qcprintf(fmt, args...) if (!bad_only) printf(_(fmt), ## args)
 
-static void qcheck_cleanup(regex_t **regex_head, const size_t regex_count)
-{
-	size_t i;
-
-	if (regex_head == NULL)
-		return;
-
-	for (i = 0; i < regex_count; ++i) {
-		regfree(regex_head[i]);
-		free(regex_head[i]);
-	}
-	free(regex_head);
-}
-
 static int qcheck_process_contents(int portroot_fd, int pkg_fd,
-	const char *catname, const char *pkgname, regex_t **regex_head,
-	size_t regex_count, bool qc_update, bool chk_afk,
-	bool chk_hash, bool chk_mtime, bool undo_prelink)
+	const char *catname, const char *pkgname, array_t *regex_arr,
+	bool qc_update, bool chk_afk, bool chk_hash, bool chk_mtime,
+	bool undo_prelink)
 {
 	int fd;
 	FILE *fp, *fpx;
@@ -104,12 +90,13 @@
 
 		/* run our little checks */
 		++num_files;
-		if (regex_count) {
-			size_t j;
-			for (j = 0; j < regex_count; ++j)
-				if (!regexec(regex_head[j], e->name, 0, NULL, 0))
+		if (array_cnt(regex_arr)) {
+			size_t n;
+			regex_t *regex;
+			array_for_each(regex_arr, n, regex)
+				if (!regexec(regex, e->name, 0, NULL, 0))
 					break;
-			if (j < regex_count) {
+			if (n < array_cnt(regex_arr)) {
 				--num_files;
 				++num_files_ignored;
 				continue;
@@ -290,8 +277,7 @@
 	bool chk_hash = true;
 	bool chk_mtime = true;
 	bool undo_prelink = false;
-	regex_t **regex_head = NULL;
-	size_t regex_count = 0;
+	DECLARE_ARRAY(regex_arr);
 
 	DBG("argc=%d argv[0]=%s argv[1]=%s",
 	    argc, argv[0], argc > 1 ? argv[1] : "NULL?");
@@ -302,12 +288,11 @@
 		case 'a': search_all = true; break;
 		case 'e': exact = 1; break;
 		case 's': {
-				regex_head = xrealloc(regex_head, (regex_count + 1) * sizeof(*regex_head));
-				regex_head[regex_count] = xmalloc(sizeof(*regex_head[0]));
-				xregcomp(regex_head[regex_count], optarg, REG_EXTENDED|REG_NOSUB);
-				++regex_count;
-			}
+			regex_t regex;
+			xregcomp(&regex, optarg, REG_EXTENDED|REG_NOSUB);
+			xarraypush(regex_arr, &regex, sizeof(regex));
 			break;
+		}
 		case 'u': qc_update = true; break;
 		case 'A': chk_afk = false; break;
 		case 'B': bad_only = true; break;
@@ -379,14 +364,14 @@
 				continue;
 
 			ret = qcheck_process_contents(portroot_fd, pkg_fd,
-				dentry->d_name, de->d_name, regex_head, regex_count,
+				dentry->d_name, de->d_name, regex_arr,
 				qc_update, chk_afk, chk_hash, chk_mtime, undo_prelink);
 			close(pkg_fd);
 		}
 		closedir(dirp);
 	}
 
-	qcheck_cleanup(regex_head, regex_count);
+	xarrayfree(regex_arr);
 	close(portroot_fd);
 	return ret;
 }