CCL: slight reformat
[yaz-moved-to-github.git] / src / cclfind.c
index fc67b80..5d7f821 100644 (file)
@@ -1,48 +1,8 @@
-/*
- * Copyright (c) 1995, the EUROPAGATE consortium (see below).
- *
- * The EUROPAGATE consortium members are:
- *
- *    University College Dublin
- *    Danmarks Teknologiske Videnscenter
- *    An Chomhairle Leabharlanna
- *    Consejo Superior de Investigaciones Cientificas
- *
- * Permission to use, copy, modify, distribute, and sell this software and
- * its documentation, in whole or in part, for any purpose, is hereby granted,
- * provided that:
- *
- * 1. This copyright and permission notice appear in all copies of the
- * software and its documentation. Notices of copyright or attribution
- * which appear at the beginning of any file must remain unchanged.
- *
- * 2. The names of EUROPAGATE or the project partners may not be used to
- * endorse or promote products derived from this software without specific
- * prior written permission.
- *
- * 3. Users of this software (implementors and gateway operators) agree to
- * inform the EUROPAGATE consortium of their use of the software. This
- * information will be used to evaluate the EUROPAGATE project and the
- * software, and to plan further developments. The consortium may use
- * the information in later publications.
- * 
- * 4. Users of this software agree to make their best efforts, when
- * documenting their use of the software, to acknowledge the EUROPAGATE
- * consortium, and the role played by the software in their work.
- *
- * THIS SOFTWARE IS PROVIDED "AS IS" AND WITHOUT WARRANTY OF ANY KIND,
- * EXPRESS, IMPLIED, OR OTHERWISE, INCLUDING WITHOUT LIMITATION, ANY
- * WARRANTY OF MERCHANTABILITY OR FITNESS FOR A PARTICULAR PURPOSE.
- * IN NO EVENT SHALL THE EUROPAGATE CONSORTIUM OR ITS MEMBERS BE LIABLE
- * FOR ANY SPECIAL, INCIDENTAL, INDIRECT OR CONSEQUENTIAL DAMAGES OF
- * ANY KIND, OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA
- * OR PROFITS, WHETHER OR NOT ADVISED OF THE POSSIBILITY OF DAMAGE, AND
- * ON ANY THEORY OF LIABILITY, ARISING OUT OF OR IN CONNECTION WITH THE
- * USE OR PERFORMANCE OF THIS SOFTWARE.
- *
+/* This file is part of the YAZ toolkit.
+ * Copyright (C) Index Data
+ * See the file LICENSE for details.
  */
-
-/** 
+/**
  * \file cclfind.c
  * \brief Implements parsing of a CCL FIND query.
  *
  * of lookahead in the handling of relational operations.. So
  * it's not really pure.
  */
-
-
-/* CCL find (to rpn conversion)
- * Europagate, 1995
- *
- * $Id: cclfind.c,v 1.11 2007-04-26 21:45:17 adam Exp $
- *
- * Old Europagate log:
- *
- * Revision 1.16  1996/01/08  08:41:13  adam
- * Removed unused function.
- *
- * Revision 1.15  1995/07/20  08:14:34  adam
- * Qualifiers were observed too often. Instead tokens are treated as
- * qualifiers only when separated by comma.
- *
- * Revision 1.14  1995/05/16  09:39:26  adam
- * LICENSE.
- *
- * Revision 1.13  1995/04/17  09:31:42  adam
- * Improved handling of qualifiers. Aliases or reserved words.
- *
- * Revision 1.12  1995/03/20  15:27:43  adam
- * Minor changes.
- *
- * Revision 1.11  1995/02/23  08:31:59  adam
- * Changed header.
- *
- * Revision 1.9  1995/02/16  13:20:06  adam
- * Spell fix.
- *
- * Revision 1.8  1995/02/14  19:59:42  adam
- * Removed a syntax error.
- *
- * Revision 1.7  1995/02/14  19:55:10  adam
- * Header files ccl.h/cclp.h are gone! They have been merged an
- * moved to ../include/ccl.h.
- * Node kind(s) in ccl_rpn_node have changed names.
- *
- * Revision 1.6  1995/02/14  16:20:55  adam
- * Qualifiers are read from a file now.
- *
- * Revision 1.5  1995/02/14  14:12:41  adam
- * Ranges for ordered qualfiers implemented (e.g. pd=1980-1990).
- *
- * Revision 1.4  1995/02/14  13:16:29  adam
- * Left and/or right truncation implemented.
- *
- * Revision 1.3  1995/02/14  10:25:56  adam
- * The constructions 'qualifier rel term ...' implemented.
- *
- * Revision 1.2  1995/02/13  15:15:07  adam
- * Added handling of qualifiers. Not finished yet.
- *
- * Revision 1.1  1995/02/13  12:35:20  adam
- * First version of CCL. Qualifiers aren't handled yet.
- *
- */
+#if HAVE_CONFIG_H
+#include <config.h>
+#endif
 
 #include <stdlib.h>
 #include <string.h>
+#include <assert.h>
 
 #include "cclp.h"
 
  * value:  Value of attribute to seach for
  * return: 1 if found; 0 otherwise.
  */
-static int qual_val_type (struct ccl_rpn_attr **qa, int type, int value,
-                           char **attset)
+static int qual_val_type(ccl_qualifier_t *qa, int type, int value,
+                         char **attset)
 {
     int i;
-    struct ccl_rpn_attr *q;
 
     if (!qa)
         return 0;
-    for (i = 0;  (q=qa[i]); i++)
+    for (i = 0; qa[i]; i++)
+    {
+        struct ccl_rpn_attr *q = ccl_qual_get_attr(qa[i]);
         while (q)
         {
             if (q->type == type && q->kind == CCL_RPN_ATTR_NUMERIC &&
@@ -148,16 +55,17 @@ static int qual_val_type (struct ccl_rpn_attr **qa, int type, int value,
             }
             q = q->next;
         }
+    }
     return 0;
 }
 
 /**
  * strxcat: concatenate strings.
- * n:      Null-terminated Destination string 
+ * n:      Null-terminated Destination string
  * src:    Source string to be appended (not null-terminated)
  * len:    Length of source string.
  */
-static void strxcat (char *n, const char *src, int len)
+static void strxcat(char *n, const char *src, int len)
 {
     while (*n)
         n++;
@@ -171,11 +79,11 @@ static void strxcat (char *n, const char *src, int len)
  * tp:      Pointer to token info.
  * return:  malloc(3) allocated copy of token name.
  */
-static char *copy_token_name (struct ccl_token *tp)
+static char *copy_token_name(struct ccl_token *tp)
 {
-    char *str = (char *)xmalloc (tp->len + 1);
-    ccl_assert (str);
-    memcpy (str, tp->name, tp->len);
+    char *str = (char *)xmalloc(tp->len + 1);
+    ccl_assert(str);
+    memcpy(str, tp->name, tp->len);
     str[tp->len] = '\0';
     return str;
 }
@@ -188,15 +96,16 @@ static char *copy_token_name (struct ccl_token *tp)
 struct ccl_rpn_node *ccl_rpn_node_create(enum ccl_rpn_kind kind)
 {
     struct ccl_rpn_node *p;
-    p = (struct ccl_rpn_node *)xmalloc (sizeof(*p));
-    ccl_assert (p);
+    p = (struct ccl_rpn_node *)xmalloc(sizeof(*p));
+    ccl_assert(p);
     p->kind = kind;
 
-    switch(kind)
+    switch (kind)
     {
     case CCL_RPN_TERM:
         p->u.t.attr_list = 0;
         p->u.t.term = 0;
+        p->u.t.qual = 0;
         break;
     default:
         break;
@@ -208,7 +117,7 @@ struct ccl_rpn_node *ccl_rpn_node_create(enum ccl_rpn_kind kind)
  * ccl_rpn_delete: Delete RPN tree.
  * rpn:   Pointer to tree.
  */
-void ccl_rpn_delete (struct ccl_rpn_node *rpn)
+void ccl_rpn_delete(struct ccl_rpn_node *rpn)
 {
     struct ccl_rpn_attr *attr, *attr1;
     if (!rpn)
@@ -218,54 +127,51 @@ void ccl_rpn_delete (struct ccl_rpn_node *rpn)
     case CCL_RPN_AND:
     case CCL_RPN_OR:
     case CCL_RPN_NOT:
-        ccl_rpn_delete (rpn->u.p[0]);
-        ccl_rpn_delete (rpn->u.p[1]);
+        ccl_rpn_delete(rpn->u.p[0]);
+        ccl_rpn_delete(rpn->u.p[1]);
         break;
     case CCL_RPN_TERM:
-        xfree (rpn->u.t.term);
+        xfree(rpn->u.t.term);
+        xfree(rpn->u.t.qual);
         for (attr = rpn->u.t.attr_list; attr; attr = attr1)
         {
             attr1 = attr->next;
             if (attr->kind == CCL_RPN_ATTR_STRING)
                 xfree(attr->value.str);
             if (attr->set)
-                xfree (attr->set);
-            xfree (attr);
+                xfree(attr->set);
+            xfree(attr);
         }
         break;
     case CCL_RPN_SET:
-        xfree (rpn->u.setname);
+        xfree(rpn->u.setname);
         break;
     case CCL_RPN_PROX:
-        ccl_rpn_delete (rpn->u.p[0]);
-        ccl_rpn_delete (rpn->u.p[1]);
-        ccl_rpn_delete (rpn->u.p[2]);
+        ccl_rpn_delete(rpn->u.p[0]);
+        ccl_rpn_delete(rpn->u.p[1]);
+        ccl_rpn_delete(rpn->u.p[2]);
         break;
     }
-    xfree (rpn);
+    xfree(rpn);
 }
 
-static struct ccl_rpn_node *find_spec (CCL_parser cclp,
-                                       struct ccl_rpn_attr **qa);
+static struct ccl_rpn_node *find_spec(CCL_parser cclp, ccl_qualifier_t *qa);
 
-static int is_term_ok (int look, int *list)
+static int is_term_ok(int look, int *list)
 {
-    for (;*list >= 0; list++)
+    for (; *list >= 0; list++)
         if (look == *list)
             return 1;
     return 0;
 }
 
-static struct ccl_rpn_node *search_terms (CCL_parser cclp,
-                                          struct ccl_rpn_attr **qa);
+static struct ccl_rpn_node *search_terms(CCL_parser cclp, ccl_qualifier_t *qa);
 
-static struct ccl_rpn_attr *add_attr_node (struct ccl_rpn_node *p,
+static struct ccl_rpn_attr *add_attr_node(struct ccl_rpn_node *p,
                                            const char *set, int type)
 {
-    struct ccl_rpn_attr *n;
-    
-    n = (struct ccl_rpn_attr *)xmalloc (sizeof(*n));
-    ccl_assert (n);
+    struct ccl_rpn_attr *n = (struct ccl_rpn_attr *) xmalloc(sizeof(*n));
+    ccl_assert(n);
     if (set)
         n->set = xstrdup(set);
     else
@@ -273,7 +179,6 @@ static struct ccl_rpn_attr *add_attr_node (struct ccl_rpn_node *p,
     n->type = type;
     n->next = p->u.t.attr_list;
     p->u.t.attr_list = n;
-    
     return n;
 }
 
@@ -287,9 +192,7 @@ static struct ccl_rpn_attr *add_attr_node (struct ccl_rpn_node *p,
 void ccl_add_attr_numeric(struct ccl_rpn_node *p, const char *set,
                           int type, int value)
 {
-    struct ccl_rpn_attr *n;
-
-    n = add_attr_node(p, set, type);
+    struct ccl_rpn_attr *n = add_attr_node(p, set, type);
     n->kind = CCL_RPN_ATTR_NUMERIC;
     n->value.numeric = value;
 }
@@ -297,124 +200,196 @@ void ccl_add_attr_numeric(struct ccl_rpn_node *p, const char *set,
 void ccl_add_attr_string(struct ccl_rpn_node *p, const char *set,
                          int type, char *value)
 {
-    struct ccl_rpn_attr *n;
-
-    n = add_attr_node(p, set, type);
+    struct ccl_rpn_attr *n = add_attr_node(p, set, type);
     n->kind = CCL_RPN_ATTR_STRING;
     n->value.str = xstrdup(value);
 }
 
+static size_t cmp_operator(const char **aliases, const char *input)
+{
+    for (; *aliases; aliases++)
+    {
+        const char *cp = *aliases;
+        size_t i;
+        for (i = 0; *cp && *cp == input[i]; i++, cp++)
+            ;
+        if (*cp == '\0')
+            return i;
+    }
+    return 0;
+}
+
 
-/**
- * search_term: Parse CCL search term. 
- * cclp:   CCL Parser
- * qa:     Qualifier attributes already applied.
- * term_list: tokens we accept as terms in context
- * multi:  whether we accept "multiple" tokens
- * return: pointer to node(s); NULL on error.
- */
-static struct ccl_rpn_node *search_term_x (CCL_parser cclp,
-                                           struct ccl_rpn_attr **qa,
-                                           int *term_list, int multi)
+#define REGEX_CHARS "^[]{}()|.*+?!$"
+#define CCL_CHARS "#?\\"
+
+static int has_ccl_masking(const char *src_str,
+                           size_t src_len,
+                           const char **truncation_aliases,
+                           const char **mask_aliases)
 {
-    struct ccl_rpn_node *p_top = 0;
-    struct ccl_token *lookahead = cclp->look_token;
-    int and_list = 0;
-    int or_list = 0;
-    char *attset;
-    const char *truncation_aliases;
+    size_t j;
+    int quote_mode = 0;
 
-    truncation_aliases =
-        ccl_qual_search_special(cclp->bibset, "truncation");
-    if (!truncation_aliases)
-        truncation_aliases = "?";
+    for (j = 0; j < src_len; j++)
+    {
+        size_t op_size;
+        if (j > 0 && src_str[j-1] == '\\')
+            ;
+        else if (src_str[j] == '"')
+            quote_mode = !quote_mode;
+        else if (!quote_mode &&
+                 (op_size = cmp_operator(truncation_aliases,
+                                         src_str + j)))
+            return 1;
+        else if (!quote_mode &&
+                 (op_size = cmp_operator(mask_aliases,
+                                          src_str + j)))
+            return 1;
+    }
+    return 0;
+}
 
-    if (qual_val_type (qa, CCL_BIB1_STR, CCL_BIB1_STR_AND_LIST, 0))
-        and_list = 1;
-    if (qual_val_type (qa, CCL_BIB1_STR, CCL_BIB1_STR_OR_LIST, 0))
-        or_list = 1;
-    while (1)
+static int append_term(CCL_parser cclp, const char *src_str, size_t src_len,
+                       char *dst_term, int regex_trunc, int z3958_trunc,
+                       const char **truncation_aliases,
+                       const char **mask_aliases,
+                       int is_first, int is_last,
+                       int *left_trunc, int *right_trunc)
+{
+    size_t j;
+    int quote_mode = 0;
+
+    for (j = 0; j < src_len; j++)
     {
-        struct ccl_rpn_node *p;
-        size_t no, i;
-        int no_spaces = 0;
-        int left_trunc = 0;
-        int right_trunc = 0;
-        int mid_trunc = 0;
-        int relation_value = -1;
-        int position_value = -1;
-        int structure_value = -1;
-        int truncation_value = -1;
-        int completeness_value = -1;
-        int len = 0;
-        size_t max = 200;
-        if (and_list || or_list || !multi)
-            max = 1;
-        
-        /* ignore commas when dealing with and-lists .. */
-        if (and_list && lookahead && lookahead->kind == CCL_TOK_COMMA)
+        size_t op_size;
+        if (j > 0 && src_str[j-1] == '\\')
         {
-            lookahead = lookahead->next;
-            ADVANCE;
-            continue;
+            if (regex_trunc && strchr(REGEX_CHARS "\\", src_str[j]))
+                strcat(dst_term, "\\");
+            else if (z3958_trunc && strchr(CCL_CHARS "\\", src_str[j]))
+                strcat(dst_term, "\\");
+            strxcat(dst_term, src_str + j, 1);
         }
-        /* go through each TERM token. If no truncation attribute is yet
-           met, then look for left/right truncation markers (?) and
-           set left_trunc/right_trunc/mid_trunc accordingly */
-        for (no = 0; no < max && is_term_ok(lookahead->kind, term_list); no++)
+        else if (src_str[j] == '"')
+            quote_mode = !quote_mode;
+        else if (!quote_mode &&
+                 (op_size = cmp_operator(truncation_aliases,
+                                         src_str + j))
+            )
         {
-            for (i = 0; i<lookahead->len; i++)
-                if (lookahead->name[i] == ' ')
-                    no_spaces++;
-                else if (strchr(truncation_aliases, lookahead->name[i]))
-                {
-                    if (no == 0 && i == 0 && lookahead->len >= 1)
-                        left_trunc = 1;
-                    else if (!is_term_ok(lookahead->next->kind, term_list) &&
-                             i == lookahead->len-1 && i >= 1)
-                        right_trunc = 1;
-                    else
-                        mid_trunc = 1;
-                }
-            len += 1+lookahead->len+lookahead->ws_prefix_len;
-            lookahead = lookahead->next;
+            j += (op_size - 1);  /* j++ in for loop */
+            if (regex_trunc)
+                strcat(dst_term, ".*");
+            else if (z3958_trunc)
+                strcat(dst_term, "?");
+            else if (is_first && j == 0)
+                *left_trunc = 1;
+            else if (is_last && j == src_len - 1)
+                *right_trunc = 1;
+            else
+            {
+                cclp->error_code = CCL_ERR_TRUNC_NOT_EMBED;
+                return -1;
+            }
         }
-
-        if (len == 0)
-            break;      /* no more terms . stop . */
-
-
-        if (p_top)
+        else if (!quote_mode &&
+                 (op_size = cmp_operator(mask_aliases, src_str + j)))
         {
-            if (or_list)
-                p = ccl_rpn_node_create(CCL_RPN_OR);
-            else if (and_list)
-                p = ccl_rpn_node_create(CCL_RPN_AND);
+            j += (op_size - 1);  /* j++ in for loop */
+            if (regex_trunc)
+                strcat(dst_term, ".");
+            else if (z3958_trunc)
+                strcat(dst_term, "#");
             else
-                p = ccl_rpn_node_create(CCL_RPN_AND);
-            p->u.p[0] = p_top;
-            p_top = p;
+            {
+                cclp->error_code = CCL_ERR_TRUNC_NOT_SINGLE;
+                return -1;
+            }
+        }
+        else if (src_str[j] != '\\')
+        {
+            if (regex_trunc && strchr(REGEX_CHARS, src_str[j]))
+                strcat(dst_term, "\\");
+            else if (z3958_trunc && strchr(CCL_CHARS, src_str[j]))
+                strcat(dst_term, "\\");
+            strxcat(dst_term, src_str + j, 1);
         }
-                
-        /* create the term node, but wait a moment before adding the term */
-        p = ccl_rpn_node_create(CCL_RPN_TERM);
-        p->u.t.attr_list = NULL;
-        p->u.t.term = NULL;
+    }
+    return 0;
+}
 
-        /* make the top node point to us.. */
-        if (p_top)
-            p_top->u.p[1] = p;
-        else
-            p_top = p;
 
-        
-        /* go through all attributes and add them to the attribute list */
-        for (i=0; qa && qa[i]; i++)
-        {
-            struct ccl_rpn_attr *attr;
-            
-            for (attr = qa[i]; attr; attr = attr->next)
-                switch(attr->kind)
+static struct ccl_rpn_node *ccl_term_one_use(CCL_parser cclp,
+                                             struct ccl_rpn_attr *attr_use,
+                                             ccl_qualifier_t *qa,
+                                             size_t no, int term_len,
+                                             int is_phrase,
+                                             int auto_group)
+{
+    struct ccl_rpn_node *p;
+    size_t i;
+    int relation_value = -1;
+    int position_value = -1;
+    int structure_value = -1;
+    int truncation_value = -1;
+    int completeness_value = -1;
+
+    int left_trunc = 0;
+    int right_trunc = 0;
+    int regex_trunc = 0;
+    int z3958_trunc = 0;
+    int is_ccl_masked = 0;
+    char *attset;
+    struct ccl_token *lookahead = cclp->look_token;
+    const char **truncation_aliases;
+    const char *t_default[2];
+    const char **mask_aliases;
+    const char *m_default[2];
+
+    truncation_aliases =
+        ccl_qual_search_special(cclp->bibset, "truncation");
+    if (!truncation_aliases)
+    {
+        truncation_aliases = t_default;
+        t_default[0] = "?";
+        t_default[1] = 0;
+    }
+    mask_aliases =
+        ccl_qual_search_special(cclp->bibset, "mask");
+    if (!mask_aliases)
+    {
+        mask_aliases = m_default;
+        m_default[0] = "#";
+        m_default[1] = 0;
+    }
+    for (i = 0; i < no; i++)
+    {
+        if (has_ccl_masking(lookahead->name, lookahead->len,
+                            truncation_aliases,
+                            mask_aliases))
+            is_ccl_masked = 1;
+        lookahead = lookahead->next;
+    }
+    lookahead = cclp->look_token;
+
+    p = ccl_rpn_node_create(CCL_RPN_TERM);
+    p->u.t.attr_list = NULL;
+    p->u.t.term = NULL;
+    if (qa && qa[0])
+    {
+        const char *n = ccl_qual_get_name(qa[0]);
+        if (n)
+            p->u.t.qual = xstrdup(n);
+    }
+    /* go through all attributes and add them to the attribute list */
+    for (i = 0; qa && qa[i]; i++)
+    {
+        struct ccl_rpn_attr *attr;
+        for (attr = ccl_qual_get_attr(qa[i]); attr; attr = attr->next)
+            if (attr->type != 1 || attr == attr_use)
+            {
+                switch (attr->kind)
                 {
                 case CCL_RPN_ATTR_STRING:
                     ccl_add_attr_string(p, attr->set, attr->type,
@@ -444,7 +419,6 @@ static struct ccl_rpn_node *search_term_x (CCL_parser cclp,
                             if (truncation_value != -1)
                                 continue;
                             truncation_value = attr->value.numeric;
-                            left_trunc = right_trunc = mid_trunc = 0;
                             break;
                         case CCL_BIB1_COM:
                             if (completeness_value != -1)
@@ -456,85 +430,222 @@ static struct ccl_rpn_node *search_term_x (CCL_parser cclp,
                                              attr->value.numeric);
                     }
                 }
+            }
+    }
+    attset = 0;
+    if (structure_value == -1 && (
+            auto_group ||
+            qual_val_type(qa, CCL_BIB1_STR, CCL_BIB1_STR_WP, &attset))
+        )
+    {
+        if (!is_phrase)
+            ccl_add_attr_numeric(p, attset, CCL_BIB1_STR, 2);
+        else
+            ccl_add_attr_numeric(p, attset, CCL_BIB1_STR, 1);
+    }
+    if (qual_val_type(qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_REGEX,
+                      &attset))
+    {
+        if (is_ccl_masked)
+            regex_trunc = 1; /* regex trunc (102) allowed */
+    }
+    else if (qual_val_type(qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_Z3958,
+                           &attset))
+    {
+        if (is_ccl_masked)
+            z3958_trunc = 1; /* Z39.58 trunc (CCL) trunc allowed */
+    }
+    /* make the RPN token */
+    p->u.t.term = (char *)xmalloc(term_len * 2 + 2);
+    ccl_assert(p->u.t.term);
+    p->u.t.term[0] = '\0';
+
+    for (i = 0; i < no; i++)
+    {
+        const char *src_str = lookahead->name;
+        size_t src_len = lookahead->len;
+
+        if (p->u.t.term[0] && lookahead->ws_prefix_len)
+        {
+            strxcat(p->u.t.term, lookahead->ws_prefix_buf,
+                    lookahead->ws_prefix_len);
         }
-        /* len now holds the number of characters in the RPN term */
-        /* no holds the number of CCL tokens (1 or more) */
-        
-        if (structure_value == -1 && 
-            qual_val_type (qa, CCL_BIB1_STR, CCL_BIB1_STR_WP, &attset))
-        {   /* no structure attribute met. Apply either structure attribute 
-               WORD or PHRASE depending on number of CCL tokens */
-            if (no == 1 && no_spaces == 0)
-                ccl_add_attr_numeric(p, attset, CCL_BIB1_STR, 2);
-            else
-                ccl_add_attr_numeric(p, attset, CCL_BIB1_STR, 1);
+        if (append_term(cclp, src_str, src_len, p->u.t.term, regex_trunc,
+                        z3958_trunc, truncation_aliases, mask_aliases,
+                        i == 0, i == no - 1,
+                        &left_trunc, &right_trunc))
+        {
+            ccl_rpn_delete(p);
+            return NULL;
+        }
+        lookahead = lookahead->next;
+    }
+    if (left_trunc && right_trunc)
+    {
+        if (!qual_val_type(qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_BOTH,
+                           &attset))
+        {
+            cclp->error_code = CCL_ERR_TRUNC_NOT_BOTH;
+            ccl_rpn_delete(p);
+            return NULL;
+        }
+        ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 3);
+    }
+    else if (right_trunc)
+    {
+        if (!qual_val_type(qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_RIGHT,
+                           &attset))
+        {
+            cclp->error_code = CCL_ERR_TRUNC_NOT_RIGHT;
+            ccl_rpn_delete(p);
+            return NULL;
+        }
+        ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 1);
+    }
+    else if (left_trunc)
+    {
+        if (!qual_val_type(qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_LEFT,
+                           &attset))
+        {
+            cclp->error_code = CCL_ERR_TRUNC_NOT_LEFT;
+            ccl_rpn_delete(p);
+            return NULL;
         }
+        ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 2);
+    }
+    else if (regex_trunc)
+    {
+        ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 102);
+    }
+    else if (z3958_trunc)
+    {
+        ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 104);
+    }
+    else
+    {
+        if (qual_val_type(qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_NONE,
+                          &attset))
+            ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 100);
+    }
+    return p;
+}
+
+/**
+ * search_term: Parse CCL search term.
+ * cclp:   CCL Parser
+ * qa:     Qualifier attributes already applied.
+ * term_list: tokens we accept as terms in context
+ * multi:  whether we accept "multiple" tokens
+ * return: pointer to node(s); NULL on error.
+ */
+static struct ccl_rpn_node *search_term_x(CCL_parser cclp,
+                                          ccl_qualifier_t *qa,
+                                          int *term_list, int multi)
+{
+    struct ccl_rpn_node *p_top = 0;
+    struct ccl_token *lookahead = cclp->look_token;
+    int and_list = 0;
+    int auto_group = 0;
+    int or_list = 0;
+
+    if (qual_val_type(qa, CCL_BIB1_STR, CCL_BIB1_STR_AND_LIST, 0))
+        and_list = 1;
+    if (qual_val_type(qa, CCL_BIB1_STR, CCL_BIB1_STR_AUTO_GROUP, 0))
+        auto_group = 1;
+    if (qual_val_type(qa, CCL_BIB1_STR, CCL_BIB1_STR_OR_LIST, 0))
+        or_list = 1;
+    while (1)
+    {
+        struct ccl_rpn_node *p = 0;
+        size_t no, i;
+        int len = 0;
+        int is_phrase = 0;
+        size_t max = 200;
+        if (and_list || or_list || !multi)
+            max = 1;
 
-        /* make the RPN token */
-        p->u.t.term = (char *)xmalloc (len);
-        ccl_assert (p->u.t.term);
-        p->u.t.term[0] = '\0';
-        for (i = 0; i<no; i++)
+        /* ignore commas when dealing with and-lists .. */
+        if (and_list && lookahead && lookahead->kind == CCL_TOK_COMMA)
         {
-            const char *src_str = cclp->look_token->name;
-            int src_len = cclp->look_token->len;
-            
-            if (i == 0 && left_trunc)
-            {
-                src_len--;
-                src_str++;
-            }
-            if (i == no-1 && right_trunc)
-                src_len--;
-            if (i && cclp->look_token->ws_prefix_len)
-            {
-                size_t len = strlen(p->u.t.term);
-                memcpy(p->u.t.term + len, cclp->look_token->ws_prefix_buf,
-                                cclp->look_token->ws_prefix_len);
-                p->u.t.term[len + cclp->look_token->ws_prefix_len] = '\0';
-            }
-            strxcat (p->u.t.term, src_str, src_len);
+            lookahead = lookahead->next;
             ADVANCE;
+            continue;
         }
-        if (left_trunc && right_trunc)
+        for (no = 0; no < max && is_term_ok(lookahead->kind, term_list); no++)
         {
-            if (!qual_val_type (qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_BOTH,
-                                &attset))
+            int this_is_phrase = 0;
+            for (i = 0; i<lookahead->len; i++)
+                if (lookahead->name[i] == ' ')
+                    this_is_phrase = 1;
+            if (auto_group)
             {
-                cclp->error_code = CCL_ERR_TRUNC_NOT_BOTH;
-                ccl_rpn_delete (p);
-                return NULL;
+                if (no > 0 && (is_phrase || is_phrase != this_is_phrase))
+                    break;
+                is_phrase = this_is_phrase;
             }
-            ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 3);
+            else if (this_is_phrase || no > 0)
+                is_phrase = 1;
+            len += 1+lookahead->len+lookahead->ws_prefix_len;
+            lookahead = lookahead->next;
         }
-        else if (right_trunc)
+
+        if (len == 0)
+            break;      /* no more terms . stop . */
+
+        /* go through all attributes and add them to the attribute list */
+        for (i = 0; qa && qa[i]; i++)
         {
-            if (!qual_val_type (qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_RIGHT,
-                                 &attset))
-            {
-                cclp->error_code = CCL_ERR_TRUNC_NOT_RIGHT;
-                ccl_rpn_delete (p);
-                return NULL;
-            }
-            ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 1);
+            struct ccl_rpn_attr *attr;
+
+            for (attr = ccl_qual_get_attr(qa[i]); attr; attr = attr->next)
+                if (attr->type == 1)
+                {
+                    struct ccl_rpn_node *tmp2;
+                    tmp2 = ccl_term_one_use(cclp, attr, qa, no, len,
+                                            is_phrase, auto_group);
+                    if (!tmp2)
+                    {
+                        ccl_rpn_delete(p);
+                        return 0;
+                    }
+                    if (!p)
+                        p = tmp2;
+                    else
+                    {
+                        struct ccl_rpn_node *tmp1;
+                        tmp1 = ccl_rpn_node_create(CCL_RPN_OR);
+                        tmp1->u.p[0] = p;
+                        tmp1->u.p[1] = tmp2;
+                        p = tmp1;
+                    }
+                }
         }
-        else if (left_trunc)
+        if (!p)
+            p = ccl_term_one_use(cclp, 0 /* attr: no use */, qa, no, len,
+                                 is_phrase, auto_group);
+        for (i = 0; i < no; i++)
+            ADVANCE;
+        if (!p)
+            return 0;
+        /* make the top node point to us.. */
+        if (p_top)
         {
-            if (!qual_val_type (qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_LEFT,
-                                &attset))
-            {
-                cclp->error_code = CCL_ERR_TRUNC_NOT_LEFT;
-                ccl_rpn_delete (p);
-                return NULL;
-            }
-            ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 2);
+            struct ccl_rpn_node *tmp;
+
+            if (or_list)
+                tmp = ccl_rpn_node_create(CCL_RPN_OR);
+            else if (and_list)
+                tmp = ccl_rpn_node_create(CCL_RPN_AND);
+            else
+                tmp = ccl_rpn_node_create(CCL_RPN_AND);
+            tmp->u.p[0] = p_top;
+            tmp->u.p[1] = p;
+
+            p_top = tmp;
         }
         else
-        {
-            if (qual_val_type (qa, CCL_BIB1_TRU, CCL_BIB1_TRU_CAN_NONE,
-                               &attset))
-                ccl_add_attr_numeric(p, attset, CCL_BIB1_TRU, 100);
-        }
+            p_top = p;
+
         if (!multi)
             break;
     }
@@ -543,16 +654,45 @@ static struct ccl_rpn_node *search_term_x (CCL_parser cclp,
     return p_top;
 }
 
-static struct ccl_rpn_node *search_term (CCL_parser cclp,
-                                         struct ccl_rpn_attr **qa)
+static struct ccl_rpn_node *search_term(CCL_parser cclp, ccl_qualifier_t *qa)
 {
     static int list[] = {CCL_TOK_TERM, CCL_TOK_COMMA, -1};
     return search_term_x(cclp, qa, list, 0);
 }
 
+
+static struct ccl_rpn_node *search_terms2(CCL_parser cclp,
+                                          ccl_qualifier_t *qa)
+{
+    if (KIND == CCL_TOK_LP)
+    {
+        struct ccl_rpn_node *p;
+        ADVANCE;
+        if (!(p = find_spec(cclp, qa)))
+            return NULL;
+        if (KIND != CCL_TOK_RP)
+        {
+            cclp->error_code = CCL_ERR_RP_EXPECTED;
+            ccl_rpn_delete(p);
+            return NULL;
+        }
+        ADVANCE;
+        return p;
+    }
+    else
+    {
+        static int list[] = {
+            CCL_TOK_TERM, CCL_TOK_COMMA,CCL_TOK_EQ,
+            CCL_TOK_REL, CCL_TOK_SET, -1};
+
+        return search_term_x(cclp, qa, list, 1);
+    }
+}
+
+
 static
-struct ccl_rpn_node *qualifiers_order (CCL_parser cclp,
-                                       struct ccl_rpn_attr **ap, char *attset)
+struct ccl_rpn_node *qualifiers_order(CCL_parser cclp,
+                                      ccl_qualifier_t *ap, char *attset)
 {
     int rel = 0;
     struct ccl_rpn_node *p;
@@ -568,11 +708,11 @@ struct ccl_rpn_node *qualifiers_order (CCL_parser cclp,
     }
     else if (cclp->look_token->len == 2)
     {
-        if (!memcmp (cclp->look_token->name, "<=", 2))
+        if (!memcmp(cclp->look_token->name, "<=", 2))
             rel = 2;
-        else if (!memcmp (cclp->look_token->name, ">=", 2))
+        else if (!memcmp(cclp->look_token->name, ">=", 2))
             rel = 4;
-        else if (!memcmp (cclp->look_token->name, "<>", 2))
+        else if (!memcmp(cclp->look_token->name, "<>", 2))
             rel = 6;
     }
     if (!rel)
@@ -589,15 +729,20 @@ struct ccl_rpn_node *qualifiers_order (CCL_parser cclp,
         if (KIND == CCL_TOK_TERM)
         {
             size_t i;
+            int quote_mode = 0;
             for (i = 0; i<cclp->look_token->len; i++)
             {
-                if (cclp->look_token->name[i] == '-')
+                if (i > 0 && cclp->look_token->name[i] == '\\')
+                    ;
+                else if (cclp->look_token->name[i] == '"')
+                    quote_mode = !quote_mode;
+                else if (cclp->look_token->name[i] == '-' && !quote_mode)
                     break;
             }
-            
+
             if (cclp->look_token->len > 1 && i == 0)
             {   /*  -xx*/
-                struct ccl_token *ntoken = ccl_token_add (cclp->look_token);
+                struct ccl_token *ntoken = ccl_token_add(cclp->look_token);
 
                 ntoken->kind = CCL_TOK_TERM;
                 ntoken->name = cclp->look_token->name + 1;
@@ -608,7 +753,7 @@ struct ccl_rpn_node *qualifiers_order (CCL_parser cclp,
             }
             else if (cclp->look_token->len > 1 && i == cclp->look_token->len-1)
             {   /* xx- */
-                struct ccl_token *ntoken = ccl_token_add (cclp->look_token);
+                struct ccl_token *ntoken = ccl_token_add(cclp->look_token);
 
                 ntoken->kind = CCL_TOK_TERM;
                 ntoken->name = "-";
@@ -618,8 +763,8 @@ struct ccl_rpn_node *qualifiers_order (CCL_parser cclp,
             }
             else if (cclp->look_token->len > 2 && i < cclp->look_token->len)
             {   /* xx-yy */
-                struct ccl_token *ntoken1 = ccl_token_add (cclp->look_token);
-                struct ccl_token *ntoken2 = ccl_token_add (ntoken1);
+                struct ccl_token *ntoken1 = ccl_token_add(cclp->look_token);
+                struct ccl_token *ntoken2 = ccl_token_add(ntoken1);
 
                 ntoken1->kind = CCL_TOK_TERM;  /* generate - */
                 ntoken1->name = "-";
@@ -636,37 +781,37 @@ struct ccl_rpn_node *qualifiers_order (CCL_parser cclp,
                      cclp->look_token->next->kind == CCL_TOK_TERM &&
                      cclp->look_token->next->len > 1 &&
                      cclp->look_token->next->name[0] == '-')
-                     
+
             {   /* xx -yy */
                 /* we _know_ that xx does not have - in it */
-                struct ccl_token *ntoken = ccl_token_add (cclp->look_token);
+                struct ccl_token *ntoken = ccl_token_add(cclp->look_token);
 
                 ntoken->kind = CCL_TOK_TERM;    /* generate - */
                 ntoken->name = "-";
                 ntoken->len = 1;
 
                 (ntoken->next->name)++;        /* adjust yy */
-                (ntoken->next->len)--; 
+                (ntoken->next->len)--;
             }
         }
     }
-        
+
     if (rel == 3 &&
         KIND == CCL_TOK_TERM &&
         cclp->look_token->next && cclp->look_token->next->len == 1 &&
         cclp->look_token->next->name[0] == '-')
     {
         struct ccl_rpn_node *p1;
-        if (!(p1 = search_term (cclp, ap)))
+        if (!(p1 = search_term(cclp, ap)))
             return NULL;
         ADVANCE;                   /* skip '-' */
         if (KIND == CCL_TOK_TERM)  /* = term - term  ? */
         {
             struct ccl_rpn_node *p2;
-            
-            if (!(p2 = search_term (cclp, ap)))
+
+            if (!(p2 = search_term(cclp, ap)))
             {
-                ccl_rpn_delete (p1);
+                ccl_rpn_delete(p1);
                 return NULL;
             }
             p = ccl_rpn_node_create(CCL_RPN_AND);
@@ -687,42 +832,28 @@ struct ccl_rpn_node *qualifiers_order (CCL_parser cclp,
              cclp->look_token->name[0] == '-')   /* = - term  ? */
     {
         ADVANCE;
-        if (!(p = search_term (cclp, ap)))
+        if (!(p = search_term(cclp, ap)))
             return NULL;
         ccl_add_attr_numeric(p, attset, CCL_BIB1_REL, 2);
         return p;
     }
-    else if (KIND == CCL_TOK_LP)
-    {
-        ADVANCE;
-        if (!(p = find_spec (cclp, ap)))
-            return NULL;
-        if (KIND != CCL_TOK_RP)
-        {
-            cclp->error_code = CCL_ERR_RP_EXPECTED;
-            ccl_rpn_delete (p);
-            return NULL;
-        }
-        ADVANCE;
-        return p;
-    }
     else
     {
-        if (!(p = search_terms (cclp, ap)))
+        if (!(p = search_terms(cclp, ap)))
             return NULL;
-        ccl_add_attr_numeric(p, attset, CCL_BIB1_REL, rel);
+        if (rel != 3 ||
+            !qual_val_type(ap, CCL_BIB1_REL, CCL_BIB1_REL_OMIT_EQUALS, 0))
+            ccl_add_attr_numeric(p, attset, CCL_BIB1_REL, rel);
         return p;
     }
-    cclp->error_code = CCL_ERR_TERM_EXPECTED;
     return NULL;
 }
 
 static
-struct ccl_rpn_node *qualifiers2 (CCL_parser cclp, struct ccl_rpn_attr **ap)
+struct ccl_rpn_node *qualifier_relation(CCL_parser cclp, ccl_qualifier_t *ap)
 {
     char *attset;
-    struct ccl_rpn_node *p;
-    
+
     if (qual_val_type(ap, CCL_BIB1_REL, CCL_BIB1_REL_ORDER, &attset)
         || qual_val_type(ap, CCL_BIB1_REL, CCL_BIB1_REL_PORDER, &attset))
         return qualifiers_order(cclp, ap, attset);
@@ -734,41 +865,25 @@ struct ccl_rpn_node *qualifiers2 (CCL_parser cclp, struct ccl_rpn_attr **ap)
         return NULL;
     }
     ADVANCE;
-    if (KIND == CCL_TOK_LP)
-    {
-        ADVANCE;
-        if (!(p = find_spec (cclp, ap)))
-        {
-            return NULL;
-        }
-        if (KIND != CCL_TOK_RP)
-        {
-            cclp->error_code = CCL_ERR_RP_EXPECTED;
-            ccl_rpn_delete (p);
-            return NULL;
-        }
-        ADVANCE;
-    }
-    else
-        p = search_terms (cclp, ap);
-    return p;
+    return search_terms(cclp, ap);
 }
 
 /**
- * qualifiers1: Parse CCL qualifiers and search terms. 
+ * qualifier_list: Parse CCL qualifiers and search terms.
  * cclp:   CCL Parser
  * la:     Token pointer to RELATION token.
  * qa:     Qualifier attributes already applied.
  * return: pointer to node(s); NULL on error.
  */
-static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
-                                         struct ccl_rpn_attr **qa)
+static struct ccl_rpn_node *qualifier_list(CCL_parser cclp,
+                                           struct ccl_token *la,
+                                           ccl_qualifier_t *qa)
 {
     struct ccl_token *lookahead = cclp->look_token;
     struct ccl_token *look_start = cclp->look_token;
-    struct ccl_rpn_attr **ap;
+    ccl_qualifier_t *ap;
     struct ccl_rpn_node *node = 0;
-    const char *field_str;
+    const char **field_str;
     int no = 0;
     int seq = 0;
     int i;
@@ -786,15 +901,15 @@ static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
     if (qa)
         for (i=0; qa[i]; i++)
             no++;
-    ap = (struct ccl_rpn_attr **)xmalloc ((no ? (no+1) : 2) * sizeof(*ap));
-    ccl_assert (ap);
+    ap = (ccl_qualifier_t *)xmalloc((no ? (no+1) : 2) * sizeof(*ap));
+    ccl_assert(ap);
 
     field_str = ccl_qual_search_special(cclp->bibset, "field");
     if (field_str)
     {
-        if (!strcmp (field_str, "or"))
+        if (!strcmp(field_str[0], "or"))
             mode_merge = 0;
-        else if (!strcmp (field_str, "merge"))
+        else if (!strcmp(field_str[0], "merge"))
             mode_merge = 1;
     }
     if (!mode_merge)
@@ -805,22 +920,22 @@ static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
         {
             ap[1] = 0;
             seq = 0;
-            while ((ap[0] = ccl_qual_search (cclp, lookahead->name,
-                                             lookahead->len, seq)) != 0)
+            while ((ap[0] = ccl_qual_search(cclp, lookahead->name,
+                                            lookahead->len, seq)) != 0)
             {
                 struct ccl_rpn_node *node_sub;
                 cclp->look_token = la;
-                
-                node_sub = qualifiers2(cclp, ap);
+
+                node_sub = qualifier_relation(cclp, ap);
                 if (!node_sub)
                 {
-                    ccl_rpn_delete (node);
-                    xfree (ap);
+                    ccl_rpn_delete(node);
+                    xfree(ap);
                     return 0;
                 }
                 if (node)
                 {
-                    struct ccl_rpn_node *node_this = 
+                    struct ccl_rpn_node *node_this =
                         ccl_rpn_node_create(CCL_RPN_OR);
                     node_this->u.p[0] = node;
                     node_this->u.p[1] = node_sub;
@@ -834,7 +949,7 @@ static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
             {
                 cclp->look_token = lookahead;
                 cclp->error_code = CCL_ERR_UNKNOWN_QUAL;
-                xfree (ap);
+                xfree(ap);
                 return NULL;
             }
             lookahead = lookahead->next;
@@ -852,18 +967,18 @@ static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
             lookahead = look_start;
             for (i = 0; lookahead != la; i++)
             {
-                ap[i] = ccl_qual_search (cclp, lookahead->name,
+                ap[i] = ccl_qual_search(cclp, lookahead->name,
                                          lookahead->len, seq);
                 if (ap[i])
                     found++;
                 if (!ap[i] && seq > 0)
-                    ap[i] = ccl_qual_search (cclp, lookahead->name,
+                    ap[i] = ccl_qual_search(cclp, lookahead->name,
                                              lookahead->len, 0);
                 if (!ap[i])
                 {
                     cclp->look_token = lookahead;
                     cclp->error_code = CCL_ERR_UNKNOWN_QUAL;
-                    xfree (ap);
+                    xfree(ap);
                     return NULL;
                 }
                 lookahead = lookahead->next;
@@ -872,27 +987,27 @@ static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
             }
             if (qa)
             {
-                struct ccl_rpn_attr **qa0 = qa;
-                
+                ccl_qualifier_t *qa0 = qa;
+
                 while (*qa0)
                     ap[i++] = *qa0++;
             }
             ap[i] = NULL;
-            
+
             if (!found)
                 break;
-            
+
             cclp->look_token = lookahead;
-            
-            node_sub = qualifiers2(cclp, ap);
+
+            node_sub = qualifier_relation(cclp, ap);
             if (!node_sub)
             {
-                ccl_rpn_delete (node);
+                ccl_rpn_delete(node);
                 break;
             }
             if (node)
             {
-                struct ccl_rpn_node *node_this = 
+                struct ccl_rpn_node *node_this =
                     ccl_rpn_node_create(CCL_RPN_OR);
                 node_this->u.p[0] = node;
                 node_this->u.p[1] = node_sub;
@@ -903,7 +1018,7 @@ static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
             seq++;
         }
     }
-    xfree (ap);
+    xfree(ap);
     return node;
 }
 
@@ -914,13 +1029,13 @@ static struct ccl_rpn_node *qualifiers1 (CCL_parser cclp, struct ccl_token *la,
  * qa:     Qualifier attributes already applied.
  * return: pointer to node(s); NULL on error.
  */
-static struct ccl_rpn_node *search_terms (CCL_parser cclp,
-                                          struct ccl_rpn_attr **qa)
+static struct ccl_rpn_node *search_terms(CCL_parser cclp, ccl_qualifier_t *qa)
 {
     static int list[] = {
-        CCL_TOK_TERM, CCL_TOK_COMMA,CCL_TOK_EQ, CCL_TOK_REL, CCL_TOK_SET, -1};
+        CCL_TOK_TERM, CCL_TOK_COMMA,CCL_TOK_EQ,
+        CCL_TOK_REL, CCL_TOK_SET, -1};
     struct ccl_rpn_node *p1, *p2, *pn;
-    p1 = search_term_x (cclp, qa, list, 1);
+    p1 = search_terms2(cclp, qa);
     if (!p1)
         return NULL;
     while (1)
@@ -938,10 +1053,10 @@ static struct ccl_rpn_node *search_terms (CCL_parser cclp,
             p_prox->u.t.attr_list = 0;
 
             ADVANCE;
-            p2 = search_term_x (cclp, qa, list, 1);
+            p2 = search_terms2(cclp, qa);
             if (!p2)
             {
-                ccl_rpn_delete (p1);
+                ccl_rpn_delete(p1);
                 return NULL;
             }
             pn = ccl_rpn_node_create(CCL_RPN_PROX);
@@ -952,10 +1067,10 @@ static struct ccl_rpn_node *search_terms (CCL_parser cclp,
         }
         else if (is_term_ok(KIND, list))
         {
-            p2 = search_term_x (cclp, qa, list, 1);
+            p2 = search_terms2(cclp, qa);
             if (!p2)
             {
-                ccl_rpn_delete (p1);
+                ccl_rpn_delete(p1);
                 return NULL;
             }
             pn = ccl_rpn_node_create(CCL_RPN_PROX);
@@ -976,27 +1091,12 @@ static struct ccl_rpn_node *search_terms (CCL_parser cclp,
  * qa:     Qualifier attributes already applied.
  * return: pointer to node(s); NULL on error.
  */
-static struct ccl_rpn_node *search_elements (CCL_parser cclp,
-                                             struct ccl_rpn_attr **qa)
+static struct ccl_rpn_node *search_elements(CCL_parser cclp,
+                                            ccl_qualifier_t *qa)
 {
     struct ccl_rpn_node *p1;
     struct ccl_token *lookahead;
-    if (KIND == CCL_TOK_LP)
-    {
-        ADVANCE;
-        p1 = find_spec (cclp, qa);
-        if (!p1)
-            return NULL;
-        if (KIND != CCL_TOK_RP)
-        {
-            cclp->error_code = CCL_ERR_RP_EXPECTED;
-            ccl_rpn_delete (p1);
-            return NULL;
-        }
-        ADVANCE;
-        return p1;
-    }
-    else if (KIND == CCL_TOK_SET)
+    if (KIND == CCL_TOK_SET)
     {
         ADVANCE;
         if (KIND == CCL_TOK_EQ)
@@ -1007,7 +1107,7 @@ static struct ccl_rpn_node *search_elements (CCL_parser cclp,
             return NULL;
         }
         p1 = ccl_rpn_node_create(CCL_RPN_SET);
-        p1->u.setname = copy_token_name (cclp->look_token);
+        p1->u.setname = copy_token_name(cclp->look_token);
         ADVANCE;
         return p1;
     }
@@ -1017,16 +1117,16 @@ static struct ccl_rpn_node *search_elements (CCL_parser cclp,
     {
         lookahead = lookahead->next;
         if (lookahead->kind == CCL_TOK_REL || lookahead->kind == CCL_TOK_EQ)
-            return qualifiers1 (cclp, lookahead, qa);
+            return qualifier_list(cclp, lookahead, qa);
         if (lookahead->kind != CCL_TOK_COMMA)
             break;
         lookahead = lookahead->next;
     }
-    if (qa)
-        return search_terms (cclp, qa);
+    if (qa || lookahead->kind == CCL_TOK_LP)
+        return search_terms(cclp, qa);
     else
     {
-        struct ccl_rpn_attr *qa[2];
+        ccl_qualifier_t qa[2];
         struct ccl_rpn_node *node = 0;
         int seq;
         lookahead = cclp->look_token;
@@ -1041,15 +1141,15 @@ static struct ccl_rpn_node *search_elements (CCL_parser cclp,
 
             cclp->look_token = lookahead;
 
-            node_sub = search_terms (cclp, qa);
+            node_sub = search_terms(cclp, qa);
             if (!node_sub)
             {
-                ccl_rpn_delete (node);
+                ccl_rpn_delete(node);
                 return 0;
             }
             if (node)
             {
-                struct ccl_rpn_node *node_this = 
+                struct ccl_rpn_node *node_this =
                     ccl_rpn_node_create(CCL_RPN_OR);
                 node_this->u.p[0] = node;
                 node_this->u.p[1] = node_sub;
@@ -1060,7 +1160,7 @@ static struct ccl_rpn_node *search_elements (CCL_parser cclp,
                 node = node_sub;
         }
         if (!node)
-            node = search_terms (cclp, 0);
+            node = search_terms(cclp, 0);
         return node;
     }
 }
@@ -1071,11 +1171,10 @@ static struct ccl_rpn_node *search_elements (CCL_parser cclp,
  * qa:     Qualifier attributes already applied.
  * return: pointer to node(s); NULL on error.
  */
-static struct ccl_rpn_node *find_spec (CCL_parser cclp,
-                                       struct ccl_rpn_attr **qa)
+static struct ccl_rpn_node *find_spec(CCL_parser cclp, ccl_qualifier_t *qa)
 {
     struct ccl_rpn_node *p1, *p2, *pn;
-    if (!(p1 = search_elements (cclp, qa)))
+    if (!(p1 = search_elements(cclp, qa)))
         return NULL;
     while (1)
     {
@@ -1083,10 +1182,10 @@ static struct ccl_rpn_node *find_spec (CCL_parser cclp,
         {
         case CCL_TOK_AND:
             ADVANCE;
-            p2 = search_elements (cclp, qa);
+            p2 = search_elements(cclp, qa);
             if (!p2)
             {
-                ccl_rpn_delete (p1);
+                ccl_rpn_delete(p1);
                 return NULL;
             }
             pn = ccl_rpn_node_create(CCL_RPN_AND);
@@ -1097,10 +1196,10 @@ static struct ccl_rpn_node *find_spec (CCL_parser cclp,
             continue;
         case CCL_TOK_OR:
             ADVANCE;
-            p2 = search_elements (cclp, qa);
+            p2 = search_elements(cclp, qa);
             if (!p2)
             {
-                ccl_rpn_delete (p1);
+                ccl_rpn_delete(p1);
                 return NULL;
             }
             pn = ccl_rpn_node_create(CCL_RPN_OR);
@@ -1111,10 +1210,10 @@ static struct ccl_rpn_node *find_spec (CCL_parser cclp,
             continue;
         case CCL_TOK_NOT:
             ADVANCE;
-            p2 = search_elements (cclp, qa);
+            p2 = search_elements(cclp, qa);
             if (!p2)
             {
-                ccl_rpn_delete (p1);
+                ccl_rpn_delete(p1);
                 return NULL;
             }
             pn = ccl_rpn_node_create(CCL_RPN_NOT);
@@ -1138,20 +1237,20 @@ struct ccl_rpn_node *ccl_parser_find_str(CCL_parser cclp, const char *str)
     return p;
 }
 
-struct ccl_rpn_node *ccl_parser_find_token(CCL_parser cclp, 
+struct ccl_rpn_node *ccl_parser_find_token(CCL_parser cclp,
                                            struct ccl_token *list)
 {
     struct ccl_rpn_node *p;
 
     cclp->look_token = list;
-    p = find_spec (cclp, NULL);
+    p = find_spec(cclp, NULL);
     if (p && KIND != CCL_TOK_EOL)
     {
         if (KIND == CCL_TOK_RP)
             cclp->error_code = CCL_ERR_BAD_RP;
         else
             cclp->error_code = CCL_ERR_OP_EXPECTED;
-        ccl_rpn_delete (p);
+        ccl_rpn_delete(p);
         p = NULL;
     }
     cclp->error_pos = cclp->look_token->name;
@@ -1173,23 +1272,25 @@ struct ccl_rpn_node *ccl_parser_find_token(CCL_parser cclp,
 struct ccl_rpn_node *ccl_find_str(CCL_bibset bibset, const char *str,
                                   int *error, int *pos)
 {
-    CCL_parser cclp = ccl_parser_create (bibset);
+    CCL_parser cclp = ccl_parser_create(bibset);
     struct ccl_token *list;
     struct ccl_rpn_node *p;
 
-    list = ccl_parser_tokenize (cclp, str);
+    list = ccl_parser_tokenize(cclp, str);
     p = ccl_parser_find_token(cclp, list);
 
     *error = cclp->error_code;
     if (*error)
         *pos = cclp->error_pos - str;
-    ccl_parser_destroy (cclp);
-    ccl_token_del (list);
+    ccl_parser_destroy(cclp);
+    ccl_token_del(list);
     return p;
 }
+
 /*
  * Local variables:
  * c-basic-offset: 4
+ * c-file-style: "Stroustrup"
  * indent-tabs-mode: nil
  * End:
  * vim: shiftwidth=4 tabstop=8 expandtab