code cleanup

Author: marcalff

buildscripts/semantic-convention/templates/registry/common.j2

@@ -28,7 +28,6 @@
 {# ========================================================================== #}
 
 {{ template.set_file_name("schema_url.h") }}
-{% import 'common.j2' as c %}
 
 {# ========================================================================== #}
 

buildscripts/semantic-convention/templates/registry/schema_url-h.j2

@@ -26,12 +26,17 @@
  * buildscripts/semantic-convention/templates/registry/semantic_attributes-h.j2
  */
 {# ========================================================================== #}
+{#
+    DESIGN NOTES:
+    - Use the debug flag to dump the semantic convention data
+      in the generated output, to inspect it.
+#}
+{# ========================================================================== #}
 
 {% set debug = false %}
 
 {% set file_name = ctx.output + (ctx.root_namespace | snake_case) ~ "_attributes.h" %}
 {{ template.set_file_name(file_name) }}
-{% import 'common.j2' as c %}
 
 {% set attributes = ctx.attributes | list %}
 {% set enum_attributes = attributes | select("enum") | rejectattr("name", "in", ctx.excluded_attributes) | list %}
@@ -45,7 +50,7 @@ k{{ attribute.name | pascal_case }}
 {% endmacro %}
 
 {% macro attribute_type(attribute) %}
-{{ attribute.type | enum_type | map_text("cpp_types", "FIXME") }}
+{{ attribute.type | enum_type | map_text("cpp_types") }}
 {% endmacro %}
 
 {% macro enum_namespace_name(attribute) %}
@@ -56,10 +61,6 @@ k{{ attribute.name | pascal_case }}
 k{{ member.id | pascal_case }}
 {% endmacro %}
 
-{% macro stable_class_ref(const_name, separator) %}
-{{ctx.stable_package_name}}.{{ctx.root_namespace}}_attributes{{separator}}{{const_name}}
-{% endmacro %}
-
 {% set cpp_attribute_namespace = attribute_namespace(ctx) %}
 
 {# ========================================================================== #}

buildscripts/semantic-convention/templates/registry/semantic_attributes-h.j2

@@ -42,7 +42,6 @@
 
 {% set file_name = ctx.output + (ctx.root_namespace | snake_case) ~ "_metrics.h" %}
 {{ template.set_file_name(file_name) }}
-{% import 'common.j2' as c %}
 
 {% set metrics = ctx.metrics | list %}
 

buildscripts/semantic-convention/templates/registry/semantic_metrics-h.j2

@@ -1,6 +1,17 @@
 # Copyright The OpenTelemetry Authors
 # SPDX-License-Identifier: Apache-2.0
 
+# This file is the main file for weaver,
+# used to generate semantic conventions,
+# for opentelemetry-cpp.
+#
+# For doc on weaver:
+# https://github.com/open-telemetry/weaver
+#
+# For doc on the semantic conventions:
+# https://github.com/open-telemetry/semantic-conventions
+#
+
 params:
   # excluded namespaces will not be generated
   excluded_namespaces: [ios, aspnetcore, signalr, android, dotnet, jvm, kestrel, v8js, veightjs, go, nodejs]