Re: [PR] Remove APR pool from Logger constructor in next ABI version [logging-log4cxx]

2024-07-15 Thread via GitHub
swebb2066 merged PR #392: URL: https://github.com/apache/logging-log4cxx/pull/392 -- 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: notifications-unsubscr

Re: [PR] Remove APR pool from Logger constructor in next ABI version [logging-log4cxx]

2024-07-13 Thread via GitHub
swebb2066 commented on PR #392: URL: https://github.com/apache/logging-log4cxx/pull/392#issuecomment-2227160668 > Would it make sense to create a new branch On reflection, I am now inclined to add new methods in 1.3 and generate deprecated warnings on the functions to be removed i

Re: [PR] Remove APR pool from Logger constructor in next ABI version [logging-log4cxx]

2024-07-13 Thread via GitHub
rm5248 commented on PR #392: URL: https://github.com/apache/logging-log4cxx/pull/392#issuecomment-2226909411 Would it make sense to create a new branch(e.g. `log4cxx-next` or something like that) to do this type of refactoring in? It might make it a bit cleaner rather than having a bunch o

[PR] Remove APR pool from Logger constructor in next ABI version [logging-log4cxx]

2024-07-12 Thread via GitHub
swebb2066 opened a new pull request, #392: URL: https://github.com/apache/logging-log4cxx/pull/392 No memory pool should live as long as a Log4cxx logger. -- 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 t