]> git.draconx.ca Git - cdecl99.git/blobdiff - src/explain.c
tests: Add canned test case for incomplete arrays.
[cdecl99.git] / src / explain.c
index 8c86530bc7318aeaafec3f97b0dd9da802cedc9a..fe3ed7d9904ea4bc9f4bb127858a23e1d3965a30 100644 (file)
@@ -1,6 +1,6 @@
 /*
  *  Render C declarations as English.
- *  Copyright © 2011 Nick Bowler
+ *  Copyright © 2011, 2021, 2023 Nick Bowler
  *
  *  This program is free software: you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
  *  You should have received a copy of the GNU General Public License
  *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
  */
+
+#include <config.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <stdarg.h>
+#include <inttypes.h>
 #include <stdbool.h>
 #include <assert.h>
 
 #include "cdecl.h"
-#include "typemap.h"
-
-/* declare [ident] as [storage] [function specs] [other crap] [qualifiers] [type specs] */
-
-static size_t
-voutput(char *buf, size_t n, size_t off, const char *fmt, va_list ap)
-{
-       if (off >= n)
-               return vsnprintf(NULL, 0, fmt, ap);
-       return vsnprintf(buf+off, n-off, fmt, ap);
-}
-
-static size_t output(char *buf, size_t n, size_t off, const char *fmt, ...)
-{
-       va_list ap;
-       size_t ret;
-
-       va_start(ap, fmt);
-       ret = voutput(buf, n, off, fmt, ap);
-       va_end(ap);
+#include "cdecl-internal.h"
 
-       return ret;
-}
-
-static size_t advance_(char **buf, size_t *n, size_t amount)
-{
-       if (amount >= *n) {
-               *n   = 0;
-               *buf = 0;
-       } else {
-               *buf += amount;
-               *n   -= amount;
-       }
-
-       return amount;
-}
-
-static size_t advance(char **buf, size_t *n, size_t amount)
+static size_t explain_pre_specs(char *buf, size_t n, struct cdecl_declspec *s)
 {
-       size_t ret, rc;
-
-       if (!amount)
-               return 0;
-
-       ret = advance_(buf, n, amount);
-       rc = snprintf(*buf, *n, " ");
-       return ret + advance_(buf, n, rc);
+       return cdecl__explain_specs(buf, n, s, CDECL_SPEC_FUNC|CDECL_SPEC_STOR);
 }
 
-static size_t
-explain_qualifiers(char *buf, size_t n, struct cdecl_declspec *s)
+static size_t explain_post_specs(char *buf, size_t n, struct cdecl_declspec *s)
 {
-       unsigned long qualmap = 0;
-       size_t ret = 0, rc = 0;
-
-       for (struct cdecl_declspec *c = s; c; c = c->next) {
-               if (cdecl_spec_kind(c) != CDECL_SPEC_QUAL)
-                       continue;
-               qualmap |= 1ul << (c->type & 0xff);
-       }
-
-       if (qualmap & (1ul << (CDECL_QUAL_RESTRICT & 0xff))) {
-               ret += advance(&buf, &n, rc);
-               rc = snprintf(buf, n, "restrict");
-       }
-       if (qualmap & (1ul << (CDECL_QUAL_VOLATILE & 0xff))) {
-               ret += advance(&buf, &n, rc);
-               rc = snprintf(buf, n, "volatile");
-       }
-       if (qualmap & (1ul << (CDECL_QUAL_CONST & 0xff))) {
-               ret += advance(&buf, &n, rc);
-               rc = snprintf(buf, n, "const");
-       }
-
-       return ret + rc;
+       return cdecl__explain_specs(buf, n, s, CDECL_SPEC_QUAL|CDECL_SPEC_TYPE);
 }
 
-/* Renders the type qualifiers and type specifiers in canonical form. */
 static size_t
-explain_post_specs(char *buf, size_t n, struct cdecl_declspec *s)
+explain_name(char *buf, size_t n, struct cdecl_declarator *d)
 {
-       const char *tag = NULL;
-       unsigned long typemap;
-       size_t ret = 0, rc;
+       while (d->child)
+               d = d->child;
 
-       typemap = cdecl__build_typemap(s);
-       if (typemap == -1)
+       if (d->type != CDECL_DECL_IDENT)
                return 0;
 
-       for (struct cdecl_declspec *c = s; c; c = c->next) {
-               if (cdecl_spec_kind(c) != CDECL_SPEC_TYPE)
-                       continue;
-
-               /* Valid C types have at most one identifier. */
-               if (c->ident)
-                       tag = c->ident;
-       }
-
-       rc = explain_qualifiers(buf, n, s);
-       ret += advance(&buf, &n, rc);
-
-       rc = snprintf(buf, n, "%s", cdecl__explain_typemap(typemap));
-       if (tag) {
-               ret += advance(&buf, &n, rc);
-               rc = snprintf(buf, n, "%s", tag);
-       }
-
-       return ret + rc;
-}
-
-static const char *explain_storage(unsigned spec)
-{
-       switch (spec) {
-       case CDECL_STOR_TYPEDEF:
-               return "typedef";
-       case CDECL_STOR_EXTERN:
-               return "extern";
-       case CDECL_STOR_STATIC:
-               return "static";
-       case CDECL_STOR_AUTO:
-               return "auto";
-       case CDECL_STOR_REGISTER:
-               return "register";
-       default:
-               assert(0);
-       }
-}
-
-/* Renders the storage-class and function specifiers in canonical form. */
-static size_t explain_pre_specs(char *buf, size_t n, struct cdecl_declspec *s)
-{
-       unsigned long funcmap = 0;
-       const char *storage = NULL;
-       size_t ret = 0;
-
-       for (struct cdecl_declspec *c = s; c; c = c->next) {
-               switch (cdecl_spec_kind(c)) {
-               case CDECL_SPEC_FUNC:
-                       funcmap |= 1ul << (c->type & 0xff);
-                       break;
-               case CDECL_SPEC_STOR:
-                       /* Valid C declarations have at most one
-                        * storage-class specifier. */
-                       storage = explain_storage(c->type);
-                       break;
-               }
-       }
-
-       if (storage)
-               ret += output(buf, n, ret, "%s", storage);
-       if (funcmap & (1ul << (CDECL_FUNC_INLINE & 0xff)))
-               ret += output(buf, n, ret, "%.*s%s", !!ret, "", "inline");
-       return ret;
-}
-
-/*
- * Renders the start of the thing being declared.  If top is true, print
- * the "declare" or "type" keywords at the front, as appropriate.
- */
-static size_t
-explain_prologue(char *buf, size_t n, struct cdecl_declarator *d, bool top)
-{
-       size_t ret = 0, rc = 0;
-
-       while (d) {
-               switch (d->type) {
-               case CDECL_DECL_NULL:
-                       if (top)
-                               return snprintf(buf, n, "type");
-                       return 0;
-               case CDECL_DECL_IDENT:
-                       if (top)
-                               rc = snprintf(buf, n, "declare");
-                       ret += advance(&buf, &n, rc);
-                       return ret + snprintf(buf, n, "%s as", d->u.ident);
-               }
-
-               d = d->child;
-       }
+       return snprintf(buf, n, "%s as", d->u.ident);
 }
 
 static size_t
@@ -208,8 +54,8 @@ explain_pointer(char *buf, size_t n, struct cdecl_pointer *p)
 {
        size_t ret = 0, rc;
 
-       rc = explain_qualifiers(buf, n, p->qualifiers);
-       ret += advance(&buf, &n, rc);
+       rc = cdecl__explain_specs(buf, n, p->qualifiers, CDECL_SPEC_QUAL);
+       ret += cdecl__advance(&buf, &n, rc);
 
        return ret + snprintf(buf, n, "pointer to");
 }
@@ -223,14 +69,14 @@ explain_array(char *buf, size_t n, struct cdecl_array *a)
                rc = snprintf(buf, n, "variable-length array");
        else
                rc = snprintf(buf, n, "array");
-       ret += advance(&buf, &n, rc);
+       ret += cdecl__advance(&buf, &n, rc);
 
        if (a->vla) {
                rc = snprintf(buf, n, "%s", a->vla);
-               ret += advance(&buf, &n, rc);
+               ret += cdecl__advance(&buf, &n, rc);
        } else if (a->length) {
-               rc = snprintf(buf, n, "%ju", a->length);
-               ret += advance(&buf, &n, rc);
+               rc = snprintf(buf, n, "%" PRIuMAX, a->length);
+               ret += cdecl__advance(&buf, &n, rc);
        }
 
        return ret + snprintf(buf, n, "of");
@@ -239,18 +85,18 @@ explain_array(char *buf, size_t n, struct cdecl_array *a)
 static size_t
 explain_declarators(char *buf, size_t n, struct cdecl_declarator *decl);
 
-static size_t explain_decl(char *buf, size_t n, struct cdecl *decl, bool top)
+static size_t explain_decl(char *buf, size_t n, struct cdecl *decl)
 {
        size_t ret = 0, rc;
 
-       rc = explain_prologue(buf, n, decl->declarators, top);
-       ret += advance(&buf, &n, rc);
+       rc = explain_name(buf, n, decl->declarators);
+       ret += cdecl__advance(&buf, &n, rc);
 
        rc = explain_pre_specs(buf, n, decl->specifiers);
-       ret += advance(&buf, &n, rc);
+       ret += cdecl__advance(&buf, &n, rc);
 
        rc = explain_declarators(buf, n, decl->declarators);
-       ret += advance(&buf, &n, rc);
+       ret += cdecl__advance(&buf, &n, rc);
 
        return ret + explain_post_specs(buf, n, decl->specifiers);
 }
@@ -260,15 +106,15 @@ static size_t explain_function(char *buf, size_t n, struct cdecl_function *f)
        size_t ret = 0, rc = 0;
 
        rc = snprintf(buf, n, "function");
-       ret += advance(&buf, &n, rc);
+       ret += cdecl__advance(&buf, &n, rc);
 
        if (f->parameters) {
                rc = snprintf(buf, n, "(");
-               ret += advance_(&buf, &n, rc);
+               ret += cdecl__advance_(&buf, &n, rc);
 
                for (struct cdecl *p = f->parameters; p; p = p->next) {
-                       rc = explain_decl(buf, n, p, false);
-                       ret += advance_(&buf, &n, rc);
+                       rc = explain_decl(buf, n, p);
+                       ret += cdecl__advance_(&buf, &n, rc);
 
                        if (p->next)
                                rc = snprintf(buf, n, ",");
@@ -276,7 +122,7 @@ static size_t explain_function(char *buf, size_t n, struct cdecl_function *f)
                                rc = snprintf(buf, n, ", ...)");
                        else
                                rc = snprintf(buf, n, ")");
-                       ret += advance(&buf, &n, rc);
+                       ret += cdecl__advance(&buf, &n, rc);
                }
        }
 
@@ -292,7 +138,7 @@ explain_declarators(char *buf, size_t n, struct cdecl_declarator *d)
                return 0;
 
        rc = explain_declarators(buf, n, d->child);
-       ret += advance(&buf, &n, rc);
+       ret += cdecl__advance(&buf, &n, rc);
 
        switch (d->type) {
        case CDECL_DECL_POINTER:
@@ -308,5 +154,13 @@ explain_declarators(char *buf, size_t n, struct cdecl_declarator *d)
 
 size_t cdecl_explain(char *buf, size_t n, struct cdecl *decl)
 {
-       return explain_decl(buf, n, decl, true);
+       size_t ret, rc;
+
+       if (cdecl_is_abstract(decl->declarators))
+               rc = snprintf(buf, n, "type");
+       else
+               rc = snprintf(buf, n, "declare");
+       ret = cdecl__advance(&buf, &n, rc);
+
+       return ret + explain_decl(buf, n, decl);
 }