Skip to content

Commit 20e059a

Browse files
author
Petr Vesely
committed
[UR] Move validation of range() to spec parsing
1 parent e16682d commit 20e059a

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

scripts/parse_specs.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -326,6 +326,10 @@ def __validate_params(d, tags):
326326

327327
if item['type'].endswith("flag_t"):
328328
raise Exception(prefix+"'type' must not be '*_flag_t': %s"%item['type'])
329+
330+
if type_traits.is_pointer(item['type']) and "_handle_t" in item['type'] and "[in]" in item['desc']:
331+
if not param_traits.is_range(item):
332+
raise Exception(prefix+"handle type must include a range(start, end) as part of 'desc'")
329333

330334
ver = __validate_version(item, prefix=prefix, base_version=d_ver)
331335
if ver < max_ver:

scripts/templates/helper.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1148,9 +1148,6 @@ def get_loader_prologue(namespace, tags, obj, meta):
11481148
fty_name = re.sub(r"(\w+)_handle_t", r"\1_factory", tname)
11491149

11501150
if type_traits.is_pointer(item['type']):
1151-
if not param_traits.is_range(item):
1152-
print(item)
1153-
raise Exception("Pointer to a handle parameter requires a `[range(start, end)]` check as part of the description.")
11541151
range_start = param_traits.range_start(item)
11551152
range_end = param_traits.range_end(item)
11561153
prologue.append({

0 commit comments

Comments
 (0)