Move nmem_reset up to beginning of loop, as it is a pre-condition.
authorDennis Schafroth <dennis@indexdata.com>
Thu, 10 May 2012 06:50:01 +0000 (08:50 +0200)
committerDennis Schafroth <dennis@indexdata.com>
Thu, 10 May 2012 06:50:01 +0000 (08:50 +0200)
Should work as previous, but will report fewer nmem 0 bytes usage.
One extra nmem_reset (on a empty) call is the price.

src/client.c

index 9e52b0a..59e06cd 100644 (file)
@@ -1092,7 +1092,7 @@ static int apply_limit(struct session_database *sdb,
     for (i = 0; (name = facet_limits_get(facet_limits, i, &value)); i++)
     {
         struct setting *s = 0;
-        
+        nmem_reset(nmem_tmp);
         for (s = sdb->settings[PZ_LIMITMAP]; s; s = s->next)
         {
             const char *p = strchr(s->name + 3, ':');
@@ -1148,7 +1148,6 @@ static int apply_limit(struct session_database *sdb,
                 break;
             }
         }
-        nmem_reset(nmem_tmp);
         if (!s)
         {
             yaz_log(YLOG_WARN, "Target %s: limit %s used, but no limitmap defined",