Changeset 305 for trunk/xcache.c

Timestamp:

2006-12-08T17:11:19+01:00 (6 years ago)

Author:

moo

Message:

TRACE() instead of ifdef/fprintf

File:

• trunk/xcache.c (modified) (17 diffs)

trunk/xcache.c

@@ -30 +30 @@
 #include "xcache_globals.h"
 #include "processor.h"
-#include "utils.h"
 #include "const_string.h"
 #include "opcode_spec.h"
-
-#ifdef DEBUG
-#   undef NDEBUG
-#   undef inline
-#   define inline
-#else
-#   ifndef NDEBUG
-#       define NDEBUG
-#   endif
-#endif
-#include <assert.h>
+#include "utils.h"
 
 #define VAR_ENTRY_EXPIRED(pentry) ((pentry)->ttl && XG(request_time) > pentry->ctime + (pentry)->ttl)
@@ -243 +232 @@
 static void xc_entry_hold_php_dmz(xc_entry_t *xce TSRMLS_DC) /* {{{ */
 {
-#ifdef DEBUG
-    fprintf(stderr, "hold %s\n", xce->name.str.val);
-#endif
+    TRACE("hold %s", xce->name.str.val);
     xce->refcount ++;
     xc_stack_push(&XG(php_holds)[xce->cache->cacheid], (void *)xce);
@@ -290 +277 @@
 static XC_ENTRY_APPLY_FUNC(xc_gc_expires_php_entry_dmz) /* {{{ */
 {
-#ifdef DEBUG
-    fprintf(stderr, "ttl %lu, %lu %lu\n", (zend_ulong) XG(request_time), (zend_ulong) entry->atime, xc_php_ttl);
-#endif
+    TRACE("ttl %lu, %lu %lu", (zend_ulong) XG(request_time), (zend_ulong) entry->atime, xc_php_ttl);
     if (XG(request_time) > entry->atime + xc_php_ttl) {
         return 1;
@@ -309 +294 @@
 static void xc_gc_expires_one(xc_cache_t *cache, zend_ulong gc_interval, cache_apply_dmz_func_t apply_func TSRMLS_DC) /* {{{ */
 {
-#ifdef DEBUG
-    fprintf(stderr, "interval %lu, %lu %lu\n", (zend_ulong) XG(request_time), (zend_ulong) cache->last_gc_expires, gc_interval);
-#endif
+    TRACE("interval %lu, %lu %lu", (zend_ulong) XG(request_time), (zend_ulong) cache->last_gc_expires, gc_interval);
     if (XG(request_time) - cache->last_gc_expires >= gc_interval) {
         ENTER_LOCK(cache) {
@@ -623 +606 @@
     for (i = 0; i < cachecount; i ++) {
         s = &holds[i];
-#ifdef DEBUG
-        fprintf(stderr, "holded %d\n", xc_stack_size(s));
-#endif
+        TRACE("holded %d", xc_stack_size(s));
         if (xc_stack_size(s)) {
             cache = ((xc_entry_t *)xc_stack_top(s))->cache;
@@ -631 +612 @@
                 while (xc_stack_size(s)) {
                     xce = (xc_entry_t*) xc_stack_pop(s);
-#ifdef DEBUG
-                    fprintf(stderr, "unhold %s\n", xce->name.str.val);
-#endif
+                    TRACE("unhold %s", xce->name.str.val);
                     xce->refcount --;
                     assert(xce->refcount >= 0);
@@ -811 +790 @@
         assert(0);
     }
-#ifdef DEBUG
-    fprintf(stderr, "got ZEND_DECLARE_INHERITED_CLASS: %s\n", class_name + 1);
-#endif
+    TRACE("got ZEND_DECLARE_INHERITED_CLASS: %s", class_name + 1);
     /* let's see which class */
     for (i = 0; i < php->classinfo_cnt; i ++) {
@@ -891 +868 @@
         /* found */
         if (stored_xce) {
-#ifdef DEBUG
-            fprintf(stderr, "found %s, catch it\n", stored_xce->name.str.val);
-#endif
+            TRACE("found %s, catch it", stored_xce->name.str.val);
             xc_entry_hold_php_dmz(stored_xce TSRMLS_CC);
             cache->hits ++;
@@ -920 +895 @@
 
     /* {{{ compile */
-#ifdef DEBUG
-    fprintf(stderr, "compiling %s\n", filename);
-#endif
+    TRACE("compiling %s", filename);
 
     /* make compile inside sandbox */
@@ -1088 +1061 @@
     } LEAVE_LOCK_EX(cache);
     /* }}} */
-#ifdef DEBUG
-    fprintf(stderr, "stored\n");
-#endif
+    TRACE("%s", "stored");
 
 #define X_FREE(var) \
@@ -1146 +1117 @@
     CG(compiled_filename) = stored_xce->name.str.val;
     CG(zend_lineno)       = 0;
-#ifdef DEBUG
-    fprintf(stderr, "restoring %s\n", stored_xce->name.str.val);
-#endif
+    TRACE("restoring %s", stored_xce->name.str.val);
     xc_processor_restore_xc_entry_t(&xce, stored_xce, xc_readonly_protection TSRMLS_CC);
 #ifdef SHOW_DPRINT
@@ -1181 +1150 @@
     CG(in_compilation)    = 0;
     CG(compiled_filename) = NULL;
-#ifdef DEBUG
-    fprintf(stderr, "restored  %s\n", stored_xce->name.str.val);
-#endif
+    TRACE("restored  %s", stored_xce->name.str.val);
     return op_array;
 }
@@ -1875 +1842 @@
         stored_xce = xc_entry_find_dmz(&xce TSRMLS_CC);
         if (stored_xce) {
-#ifdef DEBUG
-            fprintf(stderr, "incdec: gotxce %s\n", xce.name.str.val);
-#endif
+            TRACE("incdec: gotxce %s", xce.name.str.val);
             /* timeout */
             if (VAR_ENTRY_EXPIRED(stored_xce)) {
-#ifdef DEBUG
-                fprintf(stderr, "incdec: expired\n");
-#endif
+                TRACE("%s", "incdec: expired");
                 xc_entry_remove_dmz(stored_xce TSRMLS_CC);
                 stored_xce = NULL;
@@ -1892 +1855 @@
                     stored_xce->ctime = XG(request_time);
                     stored_xce->ttl   = xce.ttl;
-#ifdef DEBUG
-                    fprintf(stderr, "incdec: islong\n");
-#endif
+                    TRACE("%s", "incdec: islong");
                     value = Z_LVAL_P(stored_var->value);
                     value += (inc == 1 ? count : - count);
@@ -1902 +1863 @@
                 }
                 else {
-#ifdef DEBUG
-                    fprintf(stderr, "incdec: notlong\n");
-#endif
+                    TRACE("%s", "incdec: notlong");
                     xc_processor_restore_zval(&oldzval, stored_xce->data.var->value, stored_xce->have_references TSRMLS_CC);
                     convert_to_long(&oldzval);
@@ -1912 +1871 @@
             }
         }
-#ifdef DEBUG
         else {
-            fprintf(stderr, "incdec: %s not found\n", xce.name.str.val);
-        }
-#endif
+            TRACE("incdec: %s not found", xce.name.str.val);
+        }
 
         value += (inc == 1 ? count : - count);
@@ -2360 +2317 @@
 
     zend_llist_prepend_element(&zend_extensions, &extension);
-#ifdef DEBUG
-    fprintf(stderr, "registered\n");
-#endif
+    TRACE("%s", "registered");
 }