zwoop commented on PR #9401: URL: https://github.com/apache/trafficserver/pull/9401#issuecomment-1430338163
> We could move `new_UDPPacket` and `UDPPacketInternal::free` into `UnixUDPNet.cc` and then remove the extern from the `P_UDPPacket.h` file. The other `new_` functions could just call `new_UDPPacket` instead of the allocator directly. Yeh, I was trying that last night, and yes, I think this is right path, but ran into other problems. I think we should land this first, and then continue from there. I also think fixing all this will likely fix the "make check" issues on macOS. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
