Server ABI needs bumping, as I think this will change the mangled name of the consume symbol:
9 - virtual void consume(MirEvent const& event) = 0; 10 + virtual void consume(MirEvent const* event) = 0;
Although you're only changing the interface, existing external callers of the function won't be able to find it any more without rebuilding.
So "Approved" pending a mirserver ABI bump.
« Back to merge proposal
Server ABI needs bumping, as I think this will change the mangled name of the consume symbol:
9 - virtual void consume(MirEvent const& event) = 0;
10 + virtual void consume(MirEvent const* event) = 0;
Although you're only changing the interface, existing external callers of the function won't be able to find it any more without rebuilding.
So "Approved" pending a mirserver ABI bump.