zwoop edited a comment on pull request #8052: URL: https://github.com/apache/trafficserver/pull/8052#issuecomment-880079084
I like this, sort of :-). My suggestion would be that rather than adding set of new TS*Fast*() APIs, we modify the existing APIs. Yes, this breaks compatibility, but has several benefits: 1. It's a hell of a lot more obvious when and how to use the new flavor of these APIs 2. It avoids additional APIs that needs to be chosen carefully As such, I suggest we modify the existing APIs like this: ``` tsapi TSMLoc TSMimeHdrFieldFind(TSMBuffer bufp, TSMLoc hdr, const char *name, int length, TSMLoc *locp); ``` where if lockp == nullptr, the old behavior is used, and if provided, it uses the locp with the understanding that the caller owns the storage. If we do this, the patch will also have to change all the core plugins to use the modified APIs properly. -- 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]
