On Fri, Mar 20, 2026 at 10:42 AM Louis Chauvet
<[email protected]> wrote:
>
> On Fri, 13 Mar 2026 07:19:56 -0600, Jim Cromie <[email protected]> wrote:
> > [...]
> > cases; perhaps a 3-X-3 of classmap-defns X classmap-users would get
> > too awkward and fiddly.
> >
> > This is an untested interface extension, so drop this patch if it
> > feels too speculative, knowing theres at least a notional solution
> > should the situation arise.
>
> I agree on the patch principle, the implementation seems good.
>
> Can you move this patch after the BROKEN removal? I think you should
> "stabilize" core features first so we can merge part of the series (and
> reduce patch count).
>

I'll do this.

> >
> >
> > diff --git a/lib/dynamic_debug.c b/lib/dynamic_debug.c
> > index e9fa0c8868db..2083a8546460 100644
> > --- a/lib/dynamic_debug.c
> > +++ b/lib/dynamic_debug.c
> > @@ -206,12 +206,17 @@ ddebug_find_valid_class(struct _ddebug_info const 
> > *di, const char *query_class,
> >       return NULL;
> >  }
> >
> > -static bool ddebug_class_in_range(const int class_id, const struct 
> > _ddebug_class_map *map)
> > +static bool ddebug_class_map_in_range(const int class_id, const struct 
> > _ddebug_class_map *map)
>
> (that the rename I mentionned earlier)

I dont know what you mean here.

>
> --
> Louis Chauvet <[email protected]>

Reply via email to