cxxrtl: use `static inline` instead of `inline` in the C API.
authorwhitequark <whitequark@whitequark.org>
Sun, 20 Dec 2020 14:48:16 +0000 (14:48 +0000)
committerwhitequark <whitequark@whitequark.org>
Sun, 20 Dec 2020 14:48:16 +0000 (14:48 +0000)
In C, non-static inline functions require an implementation elsewhere
(even though the body is right there in the header). It is basically
never desirable to use those as opposed to static inline ones.

backends/cxxrtl/cxxrtl_capi.h

index 7d9c60ac5907365a706c1b38ba77c13c6203e328..2df2b7287f44719b5377b208fa243a6df882cfa1 100644 (file)
@@ -272,7 +272,7 @@ struct cxxrtl_object *cxxrtl_get_parts(cxxrtl_handle handle, const char *name, s
 // This function is a shortcut for the most common use of `cxxrtl_get_parts`. It asserts that,
 // if the object exists, it consists of a single part. If assertions are disabled, it returns NULL
 // for multi-part objects.
-inline struct cxxrtl_object *cxxrtl_get(cxxrtl_handle handle, const char *name) {
+static inline struct cxxrtl_object *cxxrtl_get(cxxrtl_handle handle, const char *name) {
        size_t parts = 0;
        struct cxxrtl_object *object = cxxrtl_get_parts(handle, name, &parts);
        assert(object == NULL || parts == 1);