astitcher commented on PR #451: URL: https://github.com/apache/qpid-proton/pull/451#issuecomment-6008285412
Since this was raised I've made some of the changes in this PR to main. I'd like some explanation of the reason for the change in the library initialization - it just seems to be a slightly more elaborate vesion of the same code perhaps with better error reporting (but maybe that error reporting should be in the common code anyway). Also I'm unclear why you need to add the crypto library to the link - I can see on Linux/MacOS that the crypto library is linked. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
