freak82 opened a new issue, #11374: URL: https://github.com/apache/trafficserver/issues/11374
Hi there, There is [this `read` function](https://github.com/apache/trafficserver/blob/master/plugins/multiplexer/dispatch.cc#L133) in `plugins/multiplexer/dispatch.cc`: ``` uint64_t read(const TSIOBuffer &b, std::string &o, const int64_t l = 0) { TSIOBufferReader reader = TSIOBufferReaderAlloc(b); const uint64_t length = read(reader, o); TSIOBufferReaderFree(reader); return length; } ``` and it internally calls an [overload](https://github.com/apache/trafficserver/blob/master/plugins/multiplexer/dispatch.cc#L105) of the same function ``` uint64_t read(const TSIOBufferReader &r, std::string &o, int64_t l = 0) ``` I think the first function should also forward the `l` argument to the overload version but it doesn't. Currently this doesn't seem to break anything because of the current usages of the `read` functions. So, does this need to be fixed? -- 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]
