duke8253 commented on PR #10742: URL: https://github.com/apache/trafficserver/pull/10742#issuecomment-1804575303
> I don't like the double `vector` implementation. I think it would be reasonable to have the core methods return `TSAction` instead of `TSEvent`, because > > > > * The latter can be extracted from the former. > > * It avoids the double allocation. > > * It hides the pointer hacking in the Event system instead of the API implementation. That does sound better, I'll change that. Did it this way since the other ones had that part in the API. -- 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]
