Просмотр исходного кода

Define several private functions and variables as static

Guillem Jover лет назад: 18
Родитель
Сommit
151d780cc8
8 измененных файлов с 30 добавлено и 14 удалено
  1. 9 0
      ChangeLog
  2. 3 0
      debian/changelog
  3. 1 1
      dpkg-deb/build.c
  4. 4 1
      lib/dbmodify.c
  5. 7 6
      lib/fields.c
  6. 1 1
      lib/tarfn.c
  7. 4 4
      src/packages.c
  8. 1 1
      utils/start-stop-daemon.c

+ 9 - 0
ChangeLog

@@ -1,3 +1,12 @@
+2007-11-29  Guillem Jover  <guillem@debian.org>
+
+	* dpkg-deb/build.c (arbitrary_fields): Define as static.
+	* lib/dbmodify.c (fnis): Likewise.
+	* lib/fields.c (conffvalue_lastword): Likewise.
+	* lib/tarfn.c (StoC): Likewise.
+	* src/packages.c (breaks_check_target): Likewise.
+	* utils/start-stop-daemon.c (siglist): Likewise.
+
 2007-11-28  Raphael Hertzog  <hertzog@debian.org>
 
 	* scripts/dpkg-gensymbols.pl: The environment variable

+ 3 - 0
debian/changelog

@@ -14,6 +14,9 @@ dpkg (1.14.12) UNRELEASED; urgency=low
   * The environment variable DPKG_GENSYMBOLS_CHECK_LEVEL can be used to force
     dpkg-gensymbols to use a precise level of checks. Closes: #452022
 
+  [ Guillem Jover ]
+  * Define several private functions and variables as static.
+
  -- Guillem Jover <guillem@debian.org>  Sat, 24 Nov 2007 07:38:13 +0200
 
 dpkg (1.14.11) unstable; urgency=low

+ 1 - 1
dpkg-deb/build.c

@@ -56,7 +56,7 @@ struct _finfo {
   struct _finfo* next;
 };
 
-const char *arbitrary_fields[]= {
+static const char *arbitrary_fields[] = {
   "Package-Type",
   "Subarchitecture",
   "Kernel-Version",

+ 4 - 1
lib/dbmodify.c

@@ -122,7 +122,10 @@ static void createimptmp(void) {
   onerr_abort--;
 }
 
-const struct fni { const char *suffix; char **store; } fnis[]= {
+static const struct fni {
+  const char *suffix;
+  char **store;
+} fnis[] = {
   {   STATUSFILE,                 &statusfile         },
   {   AVAILFILE,                  &availablefile      },
   {   UPDATESDIR IMPORTANTTMP,    &importanttmpfile   },

+ 7 - 6
lib/fields.c

@@ -224,12 +224,13 @@ void f_configversion(struct pkginfo *pigp, struct pkginfoperfile *pifp,
                      "in Config-Version string `%.250s': %.250s"),value,emsg);
 }
 
-void conffvalue_lastword(const char *value, const char *from,
-			 const char *endent,
-			 const char **word_start_r, int *word_len_r,
-			 const char **new_from_r,
-			 const char *filename, int lno,
-			 FILE *warnto, int *warncount, struct pkginfo *pigp) {
+static void conffvalue_lastword(const char *value, const char *from,
+                                const char *endent,
+                                const char **word_start_r, int *word_len_r,
+                                const char **new_from_r,
+                                const char *filename, int lno, FILE *warnto,
+                                int *warncount, struct pkginfo *pigp)
+{
   /* the code in f_conffiles ensures that value[-1]==' ', which is helpful */
   const char *lastspc;
   

+ 1 - 1
lib/tarfn.c

@@ -54,7 +54,7 @@ OtoL(const char * s, int size)
 }
 
 /* String block to C null-terminated string */
-char *
+static char *
 StoC(const char *s, int size)
 {
 	int	len;

+ 4 - 4
src/packages.c

@@ -381,10 +381,10 @@ static void breaks_check_one(struct varbuf *aemsgs, int *ok,
   *ok= 0;
 }
 
-void breaks_check_target(struct varbuf *aemsgs, int *ok,
-                         struct pkginfo *broken,
-                         struct pkginfo *target,
-                         struct pkginfo *virtbroken) {
+static void breaks_check_target(struct varbuf *aemsgs, int *ok,
+                                struct pkginfo *broken,
+                                struct pkginfo *target,
+                                struct pkginfo *virtbroken) {
   struct deppossi *possi;
 
   for (possi= target->installed.depended; possi; possi= possi->nextrev) {

+ 1 - 1
utils/start-stop-daemon.c

@@ -371,7 +371,7 @@ struct sigpair {
 	int signal;
 };
 
-const struct sigpair siglist[] = {
+static const struct sigpair siglist[] = {
 	{ "ABRT",	SIGABRT	},
 	{ "ALRM",	SIGALRM	},
 	{ "FPE",	SIGFPE	},