*** empty log message ***

[citadel.git] / citadel / housekeeping.c

diff --git a/citadel/housekeeping.c b/citadel/housekeeping.c
index 451f6491dfebbf9bb682b25e0c6cf688c7bf7bd9..09dbad4eb143aca6d3b3fb76b22dcb9706d6823c 100644 (file)
--- a/citadel/housekeeping.c
+++ b/citadel/housekeeping.c
@@ -37,7 +37,7 @@
 #include "tools.h"
 #include "citadel.h"
 #include "server.h"
-#include "dynloader.h"
+#include "serv_extensions.h"
 #include "citserver.h"
 #include "config.h"
 #include "housekeeping.h"
@@ -92,7 +92,7 @@ void check_sched_shutdown(void) {
  * NOTE: this function pair should ONLY be called during startup.  It is NOT
  * thread safe.
  */
-void check_ref_counts_backend(struct quickroom *qrbuf, void *data) {
+void check_ref_counts_backend(struct room *qrbuf, void *data) {
        struct floor flbuf;
 
        getfloor(&flbuf, qrbuf->QRfloor);
@@ -128,6 +128,7 @@ void do_housekeeping(void) {
        static int housekeeping_in_progress = 0;
        static time_t last_timer = 0L;
        int do_housekeeping_now = 0;
+       time_t now;
 
        /*
         * We do it this way instead of wrapping the whole loop in an
@@ -135,14 +136,19 @@ void do_housekeeping(void) {
         * potentially have multiple concurrent mutexes in progress.
         */
        begin_critical_section(S_HOUSEKEEPING);
-       if ( ((time(NULL) - last_timer) > 60L)
-          && (housekeeping_in_progress == 0) ) {
-               do_housekeeping_now = 1;
-               housekeeping_in_progress = 1;
-               last_timer = time(NULL);
+       now = time(NULL);
+       if ( (now - last_timer) > (time_t)60 ) {
+               if (housekeeping_in_progress == 0) {
+                       do_housekeeping_now = 1;
+                       housekeeping_in_progress = 1;
+                       last_timer = time(NULL);
+               }
        }
        end_critical_section(S_HOUSEKEEPING);
-       if (do_housekeeping_now == 0) return;
+
+       if (do_housekeeping_now == 0) {
+               return;
+       }
 
        /*
         * Ok, at this point we've made the decision to run the housekeeping