ams-tschoening commented on pull request #60: URL: https://github.com/apache/logging-log4cxx/pull/60#issuecomment-814681675
> Changing the return value of the Transcoder::decode may make sense in the future - passing the string to modify into the function looks like an optimization that may no longer be relevant with newer C++ versions[...] Keep in mind that the current implementation really APPENDS to the given string instead of returning a new value only. That behaviour would need to be kept most likely anyway. -- 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. For queries about this service, please contact Infrastructure at: [email protected]
