Hi,
I’m trying to use eventpoints instead of breakpoints since I require breakpoints that
are not visible to clients.
But I noticed that the eventpoints were not planted as expected, so I had to make
a modification in breakpoints.c. My question is if this modification is correct.
The modification first:
In run_bp_evaulation(), change check for number of clients from checking the list to
instead check the property client_cnt of BreakpointInfo:
if (*bp->id && list_is_empty(&bp->link_clients)) return;
to this:
if (*bp->id && bp->client_cnt == 0) return;
Rationale:
In create_breakpoint_ext(), the client count is set to 1. I assume that this is
done to get pass different checks in the code that requires at least one client.
It seems like BreakpointInfo.client_count should match the number of clients
in BreakpointInfo.link_clients, so the modification should still be correct.
Compare to breakpoints:
In add_breakpoint(), a BreakpointReference is added to the created breakpoint, and this
operation will make sure that BreakpointInfo.link_clients is not empty.
Best regards,
Claes