[WIP] make char*

Author: Petr Vesely

include/ur.py

@@ -281,7 +281,7 @@ def __str__(self):
 ## @brief Extension type.
 class ur_platform_extension_t(Structure):
     _fields_ = [
-        ("name", c_char * UR_MAX_EXTENSION_NAME_LENGTH),                ## [in] null-terminated extension name.
+        ("name", c_char_p),                                             ## [in] null-terminated extension name.
         ("version", c_ulong)                                            ## [in] version of the extension using ::UR_MAKE_VERSION.
     ]
 

include/ur_api.h

@@ -368,8 +368,8 @@ urTearDown(
 ///////////////////////////////////////////////////////////////////////////////
 /// @brief Extension type.
 typedef struct ur_platform_extension_t {
-    char name[UR_MAX_EXTENSION_NAME_LENGTH]; ///< [in] null-terminated extension name.
-    uint32_t version;                        ///< [in] version of the extension using ::UR_MAKE_VERSION.
+    char *name;       ///< [in] null-terminated extension name.
+    uint32_t version; ///< [in] version of the extension using ::UR_MAKE_VERSION.
 
 } ur_platform_extension_t;
 

scripts/core/EXT.rst

@@ -15,7 +15,7 @@ without affecting the core specification. Extensions may in future be promoted t
 in a later version of Unified Runtime, when agreed upon by the Working Group. All extensions are 
 optional and are not required to be implemented by any particular adapter, but are expected to 
 be widely available and possibly be required in a future version of the specification. Adapters 
-must report which extensions are supported through the ${x}PlatformGetExtensionProperties query.
+must report which extensions are supported through the ${X}_PLATFORM_INFO_EXTENSIONS platform info query.
 Each extension may also impose additional restrictions on when it can be used - i.e. a platform
 or device query.
 
@@ -29,7 +29,7 @@ Requirements
 ============
 
 - Extensions must use globally unique names for macros, enums, structures and functions
-- Extensions must have globally unique extension names reported from ${x}PlatformGetExtensionProperties
+- Extensions must have globally unique extension names reported from ${X}_PLATFORM_INFO_EXTENSIONS platform info query
 - All extensions must be defined in this specification
 - Extensions must not break backwards compatibility of the core APIs
 - Standard extension versions must be backwards compatible with prior versions

scripts/core/platform.yml

@@ -20,8 +20,9 @@ desc: "Extension type."
 name: $x_platform_extension_t
 class: $xPlatform
 members:
-    - type: char
-      name: name[$X_MAX_EXTENSION_NAME_LENGTH]
+    # TODO - this should be char[], but haven't got it to work with generator yet
+    - type: char*
+      name: name
       desc: "[in] null-terminated extension name."
     - type: uint32_t
       name: version

source/common/ur_params.hpp

@@ -941,9 +941,9 @@ inline std::ostream &operator<<(std::ostream &os,
                                 const struct ur_platform_extension_t params) {
     os << "(struct ur_platform_extension_t){";
 
-    os << ".name[UR_MAX_EXTENSION_NAME_LENGTH] = ";
+    os << ".name = ";
 
-    os << (params.name[UR_MAX_EXTENSION_NAME_LENGTH]);
+    ur_params::serializePtr(os, (params.name));
 
     os << ", ";
     os << ".version = ";