It's easier to understand the meaning of the zero return value of these
function using the enum. It makes it obvious.
Signed-off-by: Francis Deslauriers <francis.deslauriers@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
Change-Id: Id8f199d82b17228fdc882e3ccbab8b3535840984
uint64_t lttng_filter_false(void *filter_data,
const char *filter_stack_data)
{
uint64_t lttng_filter_false(void *filter_data,
const char *filter_stack_data)
{
+ return LTTNG_FILTER_DISCARD;
}
#ifdef INTERPRETER_USE_SWITCH
}
#ifdef INTERPRETER_USE_SWITCH
goto end;
OP(FILTER_OP_RETURN):
goto end;
OP(FILTER_OP_RETURN):
- /* LTTNG_FILTER_DISCARD or LTTNG_FILTER_RECORD_FLAG */
+ /* LTTNG_FILTER_DISCARD or LTTNG_FILTER_RECORD_FLAG */
/* Handle dynamic typing. */
switch (estack_ax_t) {
case REG_S64:
/* Handle dynamic typing. */
switch (estack_ax_t) {
case REG_S64:
goto end;
OP(FILTER_OP_RETURN_S64):
goto end;
OP(FILTER_OP_RETURN_S64):
- /* LTTNG_FILTER_DISCARD or LTTNG_FILTER_RECORD_FLAG */
+ /* LTTNG_FILTER_DISCARD or LTTNG_FILTER_RECORD_FLAG */
retval = !!estack_ax_v;
ret = 0;
goto end;
retval = !!estack_ax_v;
ret = 0;
goto end;
- /* return 0 (discard) on error */
+ /* Return _DISCARD on error. */
+ return LTTNG_FILTER_DISCARD;