GeezerVison

You can probably remove this thing from lib.h:
/* filter properties */
unsigned char filtered_stack;

You can probably remove this thing from lib.h: /* filter properties */ unsigned char filtered_stack;

Thanks, updated

1. these in src/base/base_globals.h:81-83 probably redundant:

/* filters */
zend_long     filter_type_stack;
xdebug_llist *filters_stack;

For keeping BC, how about adding xdebug_set_filter() stub to src/lib/compat_stubs.c/h ?

Also for BC sakes, makes sense to keep the constants, but just move them to compat_stubs:
XDEBUG_FILTER_, XDEBUG_PATH_, XDEBUG_NAMESPACE_*

wdyt?

1. these in src/base/base_globals.h:81-83 probably redundant:

/* filters */
zend_long     filter_type_stack;
xdebug_llist *filters_stack;

Removed

For keeping BC, how about adding xdebug_set_filter() stub to src/lib/compat_stubs.c/h ?

Also for BC sakes, makes sense to keep the constants, but just move them to compat_stubs: XDEBUG_FILTER__, XDEBUG_PATH__, XDEBUG_NAMESPACE_*

wdyt?

Good shout, added back to compat_stubs

👍