derick          Fri Aug 29 12:05:46 2008 UTC

  Modified files:              (Branch: PHP_5_3)
    /php-src/ext/fileinfo/libmagic      apprentice.c file.h funcs.c magic.c 
                                        print.c 
  Log:
  - MFH: Some more cleanups, and fixed some warnings.
  
  
http://cvs.php.net/viewvc.cgi/php-src/ext/fileinfo/libmagic/apprentice.c?r1=1.7.2.2&r2=1.7.2.3&diff_format=u
Index: php-src/ext/fileinfo/libmagic/apprentice.c
diff -u php-src/ext/fileinfo/libmagic/apprentice.c:1.7.2.2 
php-src/ext/fileinfo/libmagic/apprentice.c:1.7.2.3
--- php-src/ext/fileinfo/libmagic/apprentice.c:1.7.2.2  Fri Aug 29 09:49:01 2008
+++ php-src/ext/fileinfo/libmagic/apprentice.c  Fri Aug 29 12:05:46 2008
@@ -235,7 +235,6 @@
                return rv;
        }
 
-#ifndef COMPILE_ONLY
        if ((rv = apprentice_map(ms, &magic, &nmagic, fn)) == -1) {
                if (ms->flags & MAGIC_CHECK)
                        file_magwarn(ms, "using regular magic file `%s'", fn);
@@ -263,7 +262,6 @@
        mlist->prev = ml;
 
        return 0;
-#endif /* COMPILE_ONLY */
 }
 
 protected void
@@ -464,7 +462,7 @@
                return 1;
 }
 
-private int
+private void
 set_test_type(struct magic *mstart, struct magic *m)
 {
        switch (m->type) {
@@ -599,7 +597,7 @@
        if (stat(fn, &st) == 0 && S_ISDIR(st.st_mode)) {
                dir = opendir(fn);
                if (dir) {
-                       while (d = readdir(dir)) {
+                       while ((d = readdir(dir))) {
                                snprintf(subfn, sizeof(subfn), "%s/%s",
                                    fn, d->d_name);
                                if (stat(subfn, &st) == 0 && 
S_ISREG(st.st_mode)) {
@@ -1266,11 +1264,9 @@
                if (check_format(ms, m) == -1)
                        return -1;
        }
-#ifndef COMPILE_ONLY
        if (action == FILE_CHECK) {
                file_mdump(m);
        }
-#endif
        m->mimetype[0] = '\0';          /* initialise MIME type to none */
        if (m->cont_level == 0)
                ++(*nmentryp);          /* make room for next */
@@ -1862,7 +1858,6 @@
        fd = -1;
 
 internal_loaded:
-
        *magicp = mm;
        ptr = (uint32_t *)(void *)*magicp;
        if (*ptr != MAGICNO) {
http://cvs.php.net/viewvc.cgi/php-src/ext/fileinfo/libmagic/file.h?r1=1.1.2.2&r2=1.1.2.3&diff_format=u
Index: php-src/ext/fileinfo/libmagic/file.h
diff -u php-src/ext/fileinfo/libmagic/file.h:1.1.2.2 
php-src/ext/fileinfo/libmagic/file.h:1.1.2.3
--- php-src/ext/fileinfo/libmagic/file.h:1.1.2.2        Fri Aug 29 09:49:01 2008
+++ php-src/ext/fileinfo/libmagic/file.h        Fri Aug 29 12:05:46 2008
@@ -354,10 +354,8 @@
 protected int file_check_mem(struct magic_set *, unsigned int);
 protected int file_looks_utf8(const unsigned char *, size_t, unichar *, size_t 
*);
 
-#ifndef COMPILE_ONLY
 extern const char *file_names[];
 extern const size_t file_nnames;
-#endif
 
 #ifndef HAVE_STRERROR
 extern int sys_nerr;
http://cvs.php.net/viewvc.cgi/php-src/ext/fileinfo/libmagic/funcs.c?r1=1.3.2.3&r2=1.3.2.4&diff_format=u
Index: php-src/ext/fileinfo/libmagic/funcs.c
diff -u php-src/ext/fileinfo/libmagic/funcs.c:1.3.2.3 
php-src/ext/fileinfo/libmagic/funcs.c:1.3.2.4
--- php-src/ext/fileinfo/libmagic/funcs.c:1.3.2.3       Wed Aug 27 00:17:27 2008
+++ php-src/ext/fileinfo/libmagic/funcs.c       Fri Aug 29 12:05:46 2008
@@ -145,7 +145,6 @@
        file_error(ms, errno, "error reading");
 }
 
-#ifndef COMPILE_ONLY
 protected int
 file_buffer(struct magic_set *ms, int fd, const char *inname, const void *buf,
     size_t nb)
@@ -218,7 +217,6 @@
 #endif
        return m;
 }
-#endif
 
 protected int
 file_reset(struct magic_set *ms)
http://cvs.php.net/viewvc.cgi/php-src/ext/fileinfo/libmagic/magic.c?r1=1.1.2.1&r2=1.1.2.2&diff_format=u
Index: php-src/ext/fileinfo/libmagic/magic.c
diff -u php-src/ext/fileinfo/libmagic/magic.c:1.1.2.1 
php-src/ext/fileinfo/libmagic/magic.c:1.1.2.2
--- php-src/ext/fileinfo/libmagic/magic.c:1.1.2.1       Tue Aug 26 12:23:29 2008
+++ php-src/ext/fileinfo/libmagic/magic.c       Fri Aug 29 12:05:46 2008
@@ -82,9 +82,7 @@
 private void close_and_restore(const struct magic_set *, const char *, int,
     const struct stat *);
 private int info_from_stat(struct magic_set *, mode_t);
-#ifndef COMPILE_ONLY
 private const char *file_or_fd(struct magic_set *, const char *, int);
-#endif
 
 #ifndef        STDIN_FILENO
 #define        STDIN_FILENO    0
@@ -235,7 +233,6 @@
        }
 }
 
-#ifndef COMPILE_ONLY
 
 /*
  * find type of descriptor
@@ -372,7 +369,6 @@
        }
        return file_getbuffer(ms);
 }
-#endif
 
 public const char *
 magic_error(struct magic_set *ms)
http://cvs.php.net/viewvc.cgi/php-src/ext/fileinfo/libmagic/print.c?r1=1.1.2.1&r2=1.1.2.2&diff_format=u
Index: php-src/ext/fileinfo/libmagic/print.c
diff -u php-src/ext/fileinfo/libmagic/print.c:1.1.2.1 
php-src/ext/fileinfo/libmagic/print.c:1.1.2.2
--- php-src/ext/fileinfo/libmagic/print.c:1.1.2.1       Fri Aug 29 09:49:01 2008
+++ php-src/ext/fileinfo/libmagic/print.c       Fri Aug 29 12:05:46 2008
@@ -46,7 +46,6 @@
 
 #define SZOF(a)        (sizeof(a) / sizeof(a[0]))
 
-#ifndef COMPILE_ONLY
 protected void
 file_mdump(struct magic *m)
 {
@@ -177,7 +176,6 @@
        }
        (void) fprintf(stderr, ",\"%s\"]\n", m->desc);
 }
-#endif
 
 /*VARARGS*/
 protected void



-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to