Changeset d199207

Timestamp:

Mar 11, 2012, 10:58:01 PM (13 years ago)

Author:

David Benjamin <davidben@mit.edu>

Branches:

master, release-1.10, release-1.9

Children:

3e34a69, 5111af4, d6b8b50

Parents:

a03a409

git-author:

Anders Kaseorg <andersk@mit.edu> (01/19/12 21:32:33)

git-committer:

David Benjamin <davidben@mit.edu> (03/11/12 22:58:01)

Message:

zephyr: Use field iterator interface to avoid quadratic loops

Signed-off-by: Anders Kaseorg <andersk@mit.edu>

Files:

• functions.c (modified) (2 diffs)
• perlconfig.c (modified) (2 diffs)

functions.c

@@ -1431 +1431 @@
       char *tmpbuff, *tmpbuff2;
       int i, fields;
+      const char *f;
 
       if (!owl_message_is_pseudo(m)) {
@@ -1472 +1473 @@
         owl_fmtext_appendf_normal(&fm, "  Fields    : %i\n", fields);
 
-        for (i = 0; i < fields; i++) {
-          tmpbuff = owl_zephyr_get_field_as_utf8(n, i + 1);
+        for (i = 0, f = owl_zephyr_first_raw_field(n); f != NULL;
+             i++, f = owl_zephyr_next_raw_field(n, f)) {
+          tmpbuff = owl_zephyr_field_as_utf8(n, f);
           tmpbuff2 = owl_text_indent(tmpbuff, 14, false);
           owl_fmtext_appendf_normal(&fm, "  Field %i   : %s\n", i + 1, tmpbuff2);

perlconfig.c

@@ -75 +75 @@
   const char *type;
   char *ptr, *utype, *blessas;
-  int i, j;
+  const char *f;
+  int i;
   const owl_pair *pair;
   const owl_filter *wrap;
@@ -96 +97 @@
 
     av_zfields = newAV();
-    j=owl_zephyr_get_num_fields(owl_message_get_notice(m));
-    for (i=0; i<j; i++) {
-      ptr=owl_zephyr_get_field_as_utf8(owl_message_get_notice(m), i+1);
+    for (f = owl_zephyr_first_raw_field(owl_message_get_notice(m)); f != NULL;
+         f = owl_zephyr_next_raw_field(owl_message_get_notice(m), f)) {
+      ptr=owl_zephyr_field_as_utf8(owl_message_get_notice(m), f);
       av_push(av_zfields, owl_new_sv(ptr));
       g_free(ptr);