- more test cases
[cparser] / type.c
diff --git a/type.c b/type.c
index 4346a8a..97f38a8 100644 (file)
--- a/type.c
+++ b/type.c
@@ -1,15 +1,37 @@
+/*
+ * This file is part of cparser.
+ * Copyright (C) 2007-2008 Matthias Braun <matze@braunis.de>
+ *
+ * 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 the Free Software Foundation; either version 2
+ * of the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA
+ * 02111-1307, USA.
+ */
 #include <config.h>
 
 #include <stdio.h>
 #include <assert.h>
+
 #include "type_t.h"
+#include "symbol_t.h"
 #include "type_hash.h"
 #include "adt/error.h"
 
 static struct obstack   _type_obst;
-struct obstack         *type_obst = &_type_obst;
 static FILE            *out;
-static int              type_visited = 0;
+struct obstack         *type_obst                 = &_type_obst;
+static int              type_visited              = 0;
+static bool             print_implicit_array_size = false;
 
 static void intern_print_type_pre(const type_t *type, bool top);
 static void intern_print_type_post(const type_t *type, bool top);
@@ -41,13 +63,18 @@ void print_type_qualifiers(type_qualifiers_t qualifiers)
        if(qualifiers & TYPE_QUALIFIER_RESTRICT) fputs("restrict ", out);
 }
 
+/**
+ * Prints the name of a atomic type.
+ *
+ * @param type  The type.
+ */
 static
 void print_atomic_type(const atomic_type_t *type)
 {
        print_type_qualifiers(type->type.qualifiers);
 
        const char *s;
-       switch(type->atype) {
+       switch(type->akind) {
        case ATOMIC_TYPE_INVALID:     s = "INVALIDATOMIC";      break;
        case ATOMIC_TYPE_VOID:        s = "void";               break;
        case ATOMIC_TYPE_BOOL:        s = "_Bool";              break;
@@ -70,6 +97,12 @@ void print_atomic_type(const atomic_type_t *type)
        fputs(s, out);
 }
 
+/**
+ * Print the first part (the prefix) of a type.
+ *
+ * @param type   The type to print.
+ * @param top    true, if this is the top type, false if it's an embedded type.
+ */
 static void print_function_type_pre(const function_type_t *type, bool top)
 {
        print_type_qualifiers(type->type.qualifiers);
@@ -81,8 +114,14 @@ static void print_function_type_pre(const function_type_t *type, bool top)
                fputc('(', out);
 }
 
+/**
+ * Print the second part (the postfix) of a type.
+ *
+ * @param type   The type to print.
+ * @param top    true, if this is the top type, false if it's an embedded type.
+ */
 static void print_function_type_post(const function_type_t *type,
-                                     const context_t *context, bool top)
+                                     const scope_t *scope, bool top)
 {
        intern_print_type_post(type->return_type, false);
        /* don't emit braces if we're the toplevel type... */
@@ -91,8 +130,8 @@ static void print_function_type_post(const function_type_t *type,
 
        fputc('(', out);
 
-       int                 first     = 1;
-       if(context == NULL) {
+       int first = 1;
+       if(scope == NULL) {
                function_parameter_t *parameter = type->parameters;
                for( ; parameter != NULL; parameter = parameter->next) {
                        if(first) {
@@ -103,7 +142,7 @@ static void print_function_type_post(const function_type_t *type,
                        print_type(parameter->type);
                }
        } else {
-               declaration_t *parameter = context->declarations;
+               declaration_t *parameter = scope->declarations;
                for( ; parameter != NULL; parameter = parameter->next) {
                        if(first) {
                                first = 0;
@@ -111,7 +150,7 @@ static void print_function_type_post(const function_type_t *type,
                                fputs(", ", out);
                        }
                        print_type_ext(parameter->type, parameter->symbol,
-                                      &parameter->context);
+                                      &parameter->scope);
                }
        }
        if(type->variadic) {
@@ -128,6 +167,11 @@ static void print_function_type_post(const function_type_t *type,
        fputc(')', out);
 }
 
+/**
+ * Prints the prefix part of a pointer type.
+ *
+ * @param type   The pointer type.
+ */
 static void print_pointer_type_pre(const pointer_type_t *type)
 {
        intern_print_type_pre(type->points_to, false);
@@ -135,16 +179,31 @@ static void print_pointer_type_pre(const pointer_type_t *type)
        print_type_qualifiers(type->type.qualifiers);
 }
 
+/**
+ * Prints the postfix part of a pointer type.
+ *
+ * @param type   The pointer type.
+ */
 static void print_pointer_type_post(const pointer_type_t *type)
 {
        intern_print_type_post(type->points_to, false);
 }
 
+/**
+ * Prints the prefix part of an array type.
+ *
+ * @param type   The array type.
+ */
 static void print_array_type_pre(const array_type_t *type)
 {
        intern_print_type_pre(type->element_type, false);
 }
 
+/**
+ * Prints the postfix part of an array type.
+ *
+ * @param type   The array type.
+ */
 static void print_array_type_post(const array_type_t *type)
 {
        fputc('[', out);
@@ -152,13 +211,31 @@ static void print_array_type_post(const array_type_t *type)
                fputs("static ", out);
        }
        print_type_qualifiers(type->type.qualifiers);
-       if(type->size != NULL) {
-               print_expression(type->size);
+       if(type->size_expression != NULL
+                       && (print_implicit_array_size || !type->has_implicit_size)) {
+               print_expression(type->size_expression);
        }
        fputc(']', out);
        intern_print_type_post(type->element_type, false);
 }
 
+/**
+ * Prints the postfix part of a bitfield type.
+ *
+ * @param type   The array type.
+ */
+static void print_bitfield_type_post(const bitfield_type_t *type)
+{
+       fputs(" : ", out);
+       print_expression(type->size);
+       intern_print_type_post(type->base, false);
+}
+
+/**
+ * Prints an enum definition.
+ *
+ * @param declaration  The enum's type declaration.
+ */
 void print_enum_definition(const declaration_t *declaration)
 {
        fputs("{\n", out);
@@ -173,7 +250,13 @@ void print_enum_definition(const declaration_t *declaration)
                fprintf(out, "%s", entry->symbol->string);
                if(entry->init.initializer != NULL) {
                        fprintf(out, " = ");
-                       print_expression(entry->init.enum_value);
+
+                       /* skip the implicit cast */
+                       expression_t *expression = entry->init.enum_value;
+                       if(expression->kind == EXPR_UNARY_CAST_IMPLICIT) {
+                               expression = expression->unary.value;
+                       }
+                       print_expression(expression);
                }
                fprintf(out, ",\n");
        }
@@ -183,6 +266,11 @@ void print_enum_definition(const declaration_t *declaration)
        fputs("}", out);
 }
 
+/**
+ * Prints an enum type.
+ *
+ * @param type  The enum type.
+ */
 static void print_type_enum(const enum_type_t *type)
 {
        print_type_qualifiers(type->type.qualifiers);
@@ -197,12 +285,17 @@ static void print_type_enum(const enum_type_t *type)
        }
 }
 
+/**
+ * Print the compound part of a compound type.
+ *
+ * @param declaration  The declaration of the compound type.
+ */
 void print_compound_definition(const declaration_t *declaration)
 {
        fputs("{\n", out);
        change_indent(1);
 
-       declaration_t *iter = declaration->context.declarations;
+       declaration_t *iter = declaration->scope.declarations;
        for( ; iter != NULL; iter = iter->next) {
                print_indent();
                print_declaration(iter);
@@ -214,6 +307,11 @@ void print_compound_definition(const declaration_t *declaration)
        fputs("}", out);
 }
 
+/**
+ * Prints a compound type.
+ *
+ * @param type  The compound type.
+ */
 static void print_compound_type(const compound_type_t *type)
 {
        print_type_qualifiers(type->type.qualifiers);
@@ -234,11 +332,22 @@ static void print_compound_type(const compound_type_t *type)
        }
 }
 
+/**
+ * Prints the prefix part of a typedef type.
+ *
+ * @param type   The typedef type.
+ */
 static void print_typedef_type_pre(const typedef_type_t *const type)
 {
+       print_type_qualifiers(type->type.qualifiers);
        fputs(type->declaration->symbol->string, out);
 }
 
+/**
+ * Prints the prefix part of a typeof type.
+ *
+ * @param type   The typeof type.
+ */
 static void print_typeof_type_pre(const typeof_type_t *const type)
 {
        fputs("typeof(", out);
@@ -251,11 +360,19 @@ static void print_typeof_type_pre(const typeof_type_t *const type)
        fputc(')', out);
 }
 
+/**
+ * Prints the prefix part of a type.
+ *
+ * @param type   The type.
+ * @param top    true if we print the toplevel type, false else.
+ */
 static void intern_print_type_pre(const type_t *const type, const bool top)
 {
        switch(type->kind) {
+       case TYPE_ERROR:
+               fputs("<error>", out);
        case TYPE_INVALID:
-               fputs("invalid", out);
+               fputs("<invalid>", out);
                return;
        case TYPE_ENUM:
                print_type_enum(&type->enumt);
@@ -276,6 +393,9 @@ static void intern_print_type_pre(const type_t *const type, const bool top)
        case TYPE_POINTER:
                print_pointer_type_pre(&type->pointer);
                return;
+       case TYPE_BITFIELD:
+               intern_print_type_pre(type->bitfield.base, top);
+               return;
        case TYPE_ARRAY:
                print_array_type_pre(&type->array);
                return;
@@ -289,6 +409,12 @@ static void intern_print_type_pre(const type_t *const type, const bool top)
        fputs("unknown", out);
 }
 
+/**
+ * Prints the postfix part of a type.
+ *
+ * @param type   The type.
+ * @param top    true if we print the toplevel type, false else.
+ */
 static void intern_print_type_post(const type_t *const type, const bool top)
 {
        switch(type->kind) {
@@ -301,6 +427,10 @@ static void intern_print_type_post(const type_t *const type, const bool top)
        case TYPE_ARRAY:
                print_array_type_post(&type->array);
                return;
+       case TYPE_BITFIELD:
+               print_bitfield_type_post(&type->bitfield);
+               return;
+       case TYPE_ERROR:
        case TYPE_INVALID:
        case TYPE_ATOMIC:
        case TYPE_ENUM:
@@ -313,13 +443,18 @@ static void intern_print_type_post(const type_t *const type, const bool top)
        }
 }
 
+/**
+ * Prints a type.
+ *
+ * @param type   The type.
+ */
 void print_type(const type_t *const type)
 {
        print_type_ext(type, NULL, NULL);
 }
 
 void print_type_ext(const type_t *const type, const symbol_t *symbol,
-                    const context_t *context)
+                    const scope_t *scope)
 {
        if(type == NULL) {
                fputs("nil type", out);
@@ -332,13 +467,18 @@ void print_type_ext(const type_t *const type, const symbol_t *symbol,
                fputs(symbol->string, out);
        }
        if(type->kind == TYPE_FUNCTION) {
-               print_function_type_post(&type->function, context, true);
+               print_function_type_post(&type->function, scope, true);
        } else {
                intern_print_type_post(type, true);
        }
 }
 
-static size_t get_type_size(type_t *type)
+/**
+ * Return the size of a type AST node.
+ *
+ * @param type  The type.
+ */
+static size_t get_type_size(const type_t *type)
 {
        switch(type->kind) {
        case TYPE_ATOMIC:          return sizeof(atomic_type_t);
@@ -351,16 +491,22 @@ static size_t get_type_size(type_t *type)
        case TYPE_BUILTIN:         return sizeof(builtin_type_t);
        case TYPE_TYPEDEF:         return sizeof(typedef_type_t);
        case TYPE_TYPEOF:          return sizeof(typeof_type_t);
+       case TYPE_BITFIELD:        return sizeof(bitfield_type_t);
+       case TYPE_ERROR:           panic("error type found");
        case TYPE_INVALID:         panic("invalid type found");
        }
        panic("unknown type found");
 }
 
 /**
- * duplicates a type
- * note that this does not produce a deep copy!
+ * Duplicates a type.
+ *
+ * @param type  The type to copy.
+ * @return A copy of the type.
+ *
+ * @note This does not produce a deep copy!
  */
-type_t *duplicate_type(type_t *type)
+type_t *duplicate_type(const type_t *type)
 {
        size_t size = get_type_size(type);
 
@@ -370,6 +516,12 @@ type_t *duplicate_type(type_t *type)
        return copy;
 }
 
+/**
+ * Returns the unqualified type of a given type.
+ *
+ * @param type  The type.
+ * @returns The unqualified type.
+ */
 type_t *get_unqualified_type(type_t *type)
 {
        if(type->base.qualifiers == TYPE_QUALIFIER_NONE)
@@ -386,11 +538,23 @@ type_t *get_unqualified_type(type_t *type)
        return result;
 }
 
+/**
+ * Check if a type is valid.
+ *
+ * @param type  The type to check.
+ * @return true if type represents a valid type.
+ */
 bool type_valid(const type_t *type)
 {
        return type->kind != TYPE_INVALID;
 }
 
+/**
+ * Returns true if the given type is an integer type.
+ *
+ * @param type  The type to check.
+ * @return True if type is an integer type.
+ */
 bool is_type_integer(const type_t *type)
 {
        assert(!is_typeref(type));
@@ -401,7 +565,7 @@ bool is_type_integer(const type_t *type)
        if(type->kind != TYPE_ATOMIC)
                return false;
 
-       switch(type->atomic.atype) {
+       switch(type->atomic.akind) {
        case ATOMIC_TYPE_BOOL:
        case ATOMIC_TYPE_CHAR:
        case ATOMIC_TYPE_SCHAR:
@@ -420,14 +584,20 @@ bool is_type_integer(const type_t *type)
        }
 }
 
-bool is_type_floating(const type_t *type)
+/**
+ * Returns true if the given type is an floating point type.
+ *
+ * @param type  The type to check.
+ * @return True if type is a floating point type.
+ */
+bool is_type_float(const type_t *type)
 {
        assert(!is_typeref(type));
 
        if(type->kind != TYPE_ATOMIC)
                return false;
 
-       switch(type->atomic.atype) {
+       switch(type->atomic.akind) {
        case ATOMIC_TYPE_FLOAT:
        case ATOMIC_TYPE_DOUBLE:
        case ATOMIC_TYPE_LONG_DOUBLE:
@@ -445,6 +615,12 @@ bool is_type_floating(const type_t *type)
        }
 }
 
+/**
+ * Returns true if the given type is a signed type.
+ *
+ * @param type  The type to check.
+ * @return True if type is a signed type.
+ */
 bool is_type_signed(const type_t *type)
 {
        assert(!is_typeref(type));
@@ -456,7 +632,7 @@ bool is_type_signed(const type_t *type)
        if(type->kind != TYPE_ATOMIC)
                return false;
 
-       switch(type->atomic.atype) {
+       switch(type->atomic.akind) {
        case ATOMIC_TYPE_CHAR:
        case ATOMIC_TYPE_SCHAR:
        case ATOMIC_TYPE_SHORT:
@@ -494,16 +670,31 @@ bool is_type_signed(const type_t *type)
        return false;
 }
 
+/**
+ * Returns true if the given type represents an arithmetic type.
+ *
+ * @param type  The type to check.
+ * @return True if type represents an arithmetic type.
+ */
 bool is_type_arithmetic(const type_t *type)
 {
        assert(!is_typeref(type));
 
-       if(is_type_integer(type) || is_type_floating(type))
+       if(type->kind == TYPE_BITFIELD)
+               return true;
+
+       if(is_type_integer(type) || is_type_float(type))
                return true;
 
        return false;
 }
 
+/**
+ * Returns true if the given type represents a scalar type.
+ *
+ * @param type  The type to check.
+ * @return True if type represents a scalar type.
+ */
 bool is_type_scalar(const type_t *type)
 {
        assert(!is_typeref(type));
@@ -511,12 +702,18 @@ bool is_type_scalar(const type_t *type)
        switch (type->kind) {
                case TYPE_POINTER: return true;
                case TYPE_BUILTIN: return is_type_scalar(type->builtin.real_type);
-               default:           break;
+               default:            break;
        }
 
        return is_type_arithmetic(type);
 }
 
+/**
+ * Check if a given type is incomplete.
+ *
+ * @param type  The type to check.
+ * @return True if the given type is incomplete (ie. just forward).
+ */
 bool is_type_incomplete(const type_t *type)
 {
        assert(!is_typeref(type));
@@ -528,16 +725,24 @@ bool is_type_incomplete(const type_t *type)
                declaration_t         *declaration   = compound_type->declaration;
                return !declaration->init.is_defined;
        }
+       case TYPE_ENUM: {
+               const enum_type_t *enum_type   = &type->enumt;
+               declaration_t     *declaration = enum_type->declaration;
+               return !declaration->init.is_defined;
+       }
+       case TYPE_BITFIELD:
        case TYPE_FUNCTION:
                return true;
 
        case TYPE_ARRAY:
-               return type->array.size == NULL;
+               return type->array.size_expression == NULL;
 
        case TYPE_ATOMIC:
+               return type->atomic.akind == ATOMIC_TYPE_VOID;
+
        case TYPE_POINTER:
-       case TYPE_ENUM:
        case TYPE_BUILTIN:
+       case TYPE_ERROR:
                return false;
 
        case TYPE_TYPEDEF:
@@ -550,10 +755,15 @@ bool is_type_incomplete(const type_t *type)
        panic("invalid type found");
 }
 
+/**
+ * Check if two function types are compatible.
+ */
 static bool function_types_compatible(const function_type_t *func1,
                                       const function_type_t *func2)
 {
-       if(!types_compatible(func1->return_type, func2->return_type))
+       const type_t* const ret1 = skip_typeref(func1->return_type);
+       const type_t* const ret2 = skip_typeref(func2->return_type);
+       if (!types_compatible(ret1, ret2))
                return false;
 
        /* can parameters be compared? */
@@ -586,6 +796,9 @@ static bool function_types_compatible(const function_type_t *func1,
        return true;
 }
 
+/**
+ * Check if two array types are compatible.
+ */
 static bool array_types_compatible(const array_type_t *array1,
                                    const array_type_t *array2)
 {
@@ -594,14 +807,15 @@ static bool array_types_compatible(const array_type_t *array1,
        if(!types_compatible(element_type1, element_type2))
                return false;
 
-       if(array1->size != NULL && array2->size != NULL) {
-               /* TODO: check if size expression evaulate to the same value
-                * if they are constant */
-       }
+       if(!array1->size_constant || !array2->size_constant)
+               return true;
 
-       return true;
+       return array1->size == array2->size;
 }
 
+/**
+ * Check if two types are compatible.
+ */
 bool types_compatible(const type_t *type1, const type_t *type2)
 {
        assert(!is_typeref(type1));
@@ -620,12 +834,16 @@ bool types_compatible(const type_t *type1, const type_t *type2)
        case TYPE_FUNCTION:
                return function_types_compatible(&type1->function, &type2->function);
        case TYPE_ATOMIC:
-               return type1->atomic.atype == type2->atomic.atype;
+               return type1->atomic.akind == type2->atomic.akind;
        case TYPE_ARRAY:
                return array_types_compatible(&type1->array, &type2->array);
-       case TYPE_POINTER:
-               return types_compatible(type1->pointer.points_to,
-                                       type2->pointer.points_to);
+
+       case TYPE_POINTER: {
+               const type_t *const to1 = skip_typeref(type1->pointer.points_to);
+               const type_t *const to2 = skip_typeref(type2->pointer.points_to);
+               return types_compatible(to1, to2);
+       }
+
        case TYPE_COMPOUND_STRUCT:
        case TYPE_COMPOUND_UNION:
        case TYPE_ENUM:
@@ -633,6 +851,14 @@ bool types_compatible(const type_t *type1, const type_t *type2)
                /* TODO: not implemented */
                break;
 
+       case TYPE_BITFIELD:
+               /* not sure if this makes sense or is even needed, implement it if you
+                * really need it! */
+               panic("type compatibility check for bitfield type");
+
+       case TYPE_ERROR:
+               /* Hmm, the error type should be compatible to all other types */
+               return true;
        case TYPE_INVALID:
                panic("invalid type found in compatible types");
        case TYPE_TYPEDEF:
@@ -644,6 +870,9 @@ bool types_compatible(const type_t *type1, const type_t *type2)
        return false;
 }
 
+/**
+ * Check if two pointer types are compatible.
+ */
 bool pointers_compatible(const type_t *type1, const type_t *type2)
 {
        assert(!is_typeref(type1));
@@ -651,16 +880,23 @@ bool pointers_compatible(const type_t *type1, const type_t *type2)
 
        assert(type1->kind == TYPE_POINTER);
        assert(type2->kind == TYPE_POINTER);
+       (void) type1;
+       (void) type2;
        /* TODO */
        return true;
 }
 
+/**
+ * Skip all typerefs and return the underlying type.
+ */
 type_t *skip_typeref(type_t *type)
 {
-       unsigned qualifiers = type->base.qualifiers;
+       unsigned qualifiers = TYPE_QUALIFIER_NONE;
 
-       while(1) {
+       while(true) {
                switch(type->kind) {
+               case TYPE_ERROR:
+                       return type;
                case TYPE_TYPEDEF: {
                        qualifiers |= type->base.qualifiers;
                        const typedef_type_t *typedef_type = &type->typedeft;
@@ -676,7 +912,7 @@ type_t *skip_typeref(type_t *type)
                        if(typeof_type->typeof_type != NULL) {
                                type = typeof_type->typeof_type;
                        } else {
-                               type = typeof_type->expression->base.datatype;
+                               type = typeof_type->expression->base.type;
                        }
                        continue;
                }
@@ -686,11 +922,23 @@ type_t *skip_typeref(type_t *type)
                break;
        }
 
-       return type;
-}
+       if (qualifiers != TYPE_QUALIFIER_NONE) {
+               type_t *const copy     = duplicate_type(type);
+               copy->base.qualifiers |= qualifiers;
 
+               type = typehash_insert(copy);
+               if (type != copy) {
+                       obstack_free(type_obst, copy);
+               }
+       }
 
+       return type;
+}
 
+/**
+ * Hash the given type and return the "singleton" version
+ * of it.
+ */
 static type_t *identify_new_type(type_t *type)
 {
        type_t *result = typehash_insert(type);
@@ -700,18 +948,33 @@ static type_t *identify_new_type(type_t *type)
        return result;
 }
 
-type_t *make_atomic_type(atomic_type_type_t atype, type_qualifiers_t qualifiers)
+/**
+ * Creates a new atomic type.
+ *
+ * @param akind       The kind of the atomic type.
+ * @param qualifiers  Type qualifiers for the new type.
+ */
+type_t *make_atomic_type(atomic_type_kind_t atype, type_qualifiers_t qualifiers)
 {
        type_t *type = obstack_alloc(type_obst, sizeof(atomic_type_t));
        memset(type, 0, sizeof(atomic_type_t));
 
        type->kind            = TYPE_ATOMIC;
        type->base.qualifiers = qualifiers;
-       type->atomic.atype    = atype;
+       type->base.alignment  = 0;
+       type->atomic.akind    = atype;
+
+       /* TODO: set the aligmnent depending on the atype here */
 
        return identify_new_type(type);
 }
 
+/**
+ * Creates a new pointer type.
+ *
+ * @param points_to   The points-to type for teh new type.
+ * @param qualifiers  Type qualifiers for the new type.
+ */
 type_t *make_pointer_type(type_t *points_to, type_qualifiers_t qualifiers)
 {
        type_t *type = obstack_alloc(type_obst, sizeof(pointer_type_t));
@@ -719,13 +982,33 @@ type_t *make_pointer_type(type_t *points_to, type_qualifiers_t qualifiers)
 
        type->kind              = TYPE_POINTER;
        type->base.qualifiers   = qualifiers;
+       type->base.alignment    = 0;
        type->pointer.points_to = points_to;
 
        return identify_new_type(type);
 }
 
+type_t *make_array_type(type_t *element_type, size_t size,
+                        type_qualifiers_t qualifiers)
+{
+       type_t *type = obstack_alloc(type_obst, sizeof(array_type_t));
+       memset(type, 0, sizeof(array_type_t));
+
+       type->kind                = TYPE_ARRAY;
+       type->base.qualifiers     = qualifiers;
+       type->base.alignment      = 0;
+       type->array.element_type  = element_type;
+       type->array.size          = size;
+       type->array.size_constant = true;
+
+       return identify_new_type(type);
+}
+
+/**
+ * Debug helper. Prints the given type to stdout.
+ */
 static __attribute__((unused))
-void dbg_type(type_t *type)
+void dbg_type(const type_t *type)
 {
        FILE *old_out = out;
        out = stderr;