From 871f239188d5215e561d9816df21ad44116d324b Mon Sep 17 00:00:00 2001 From: "Alan T. DeKok" Date: Tue, 3 Sep 2024 11:24:59 -0400 Subject: [PATCH] add assertions to make intent clear which doesn't change anything, but does help us track the migration away from '&' --- src/lib/unlang/xlat_expr.c | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/src/lib/unlang/xlat_expr.c b/src/lib/unlang/xlat_expr.c index c6a0368961..bcf574e1cc 100644 --- a/src/lib/unlang/xlat_expr.c +++ b/src/lib/unlang/xlat_expr.c @@ -2670,6 +2670,17 @@ static fr_slen_t tokenize_field(xlat_exp_head_t *head, xlat_exp_t **out, fr_sbuf } else if (tmpl_contains_xlat(node->vpt)) { node->flags = tmpl_xlat(vpt)->flags; + } else if (tmpl_is_attr(node->vpt)) { + node->flags.pure = false; + +#ifndef NDEBUG + if (vpt->name[0] == '%') { + fr_assert(vpt->rules.attr.prefix == TMPL_ATTR_REF_PREFIX_NO); + } else { + fr_assert(vpt->rules.attr.prefix == TMPL_ATTR_REF_PREFIX_YES); + } +#endif + } else { node->flags.pure = false; } -- 2.47.2