Attention is currently required from: laforge.

Patch set 2:Code-Review +1

View Change

1 comment:

• File src/core/msgb.c:

  • Patch Set #1, Line 320: * and adjusts the pointers (incl l1h-l4h) accordingly. The cb part is not copied.

    I think the cb was not copied as a performance reason, since lots of code doesn't use cb at all. […]

    I wonder how much the performance impact of copying the cb buffer would be. The additional effort should be neglectable. While the function was specified not to copy the cb, it thereis nothing said about msg->lchan and msg->dst. Maybe we could just drop the memcpy line?

To view, visit change 36324. To unsubscribe, or for help writing mail filters, visit settings.