[ptr_container, crc, format] Open tickets - code patches available
Hi, there are open tickets: https://svn.boost.org/trac/boost/ticket/7430 (ptr_container: warning fixes) https://svn.boost.org/trac/boost/ticket/7433 (crc: runtime check issue) https://svn.boost.org/trac/boost/ticket/7435 (format: crash) that we have submitted 5 months ago. Along with the bug reports there are code patches for the reported issues, that have been tested and used in production code on different platforms. We would kindly ask the maintainers to look at these issues and to apply the patches to the library code, so they can be integrated in the next release. In case the maintainers do not have time to fix these issues within say the next 2 weeks, I (Joachim) would like to ask for permission to apply the patches. Fixing the issues would be very helpful for us, so we don't have to patch the code locally for every new version. Thanks a lot, Joachim & Franz -- Interval Container Library [Boost.Icl] http://www.joachim-faulhaber.de
On 12-02-2013 17:31, Joachim Faulhaber wrote:
Hi,
there are open tickets:
https://svn.boost.org/trac/boost/ticket/7430 (ptr_container: warning fixes)
In case the maintainers do not have time to fix these issues within say the next 2 weeks, I (Joachim) would like to ask for permission to apply the patches. Fixing the issues would be very helpful for us, so we don't have to patch the code locally for every new version.
permission granted. -Thorsten
AMDG On 02/12/2013 08:31 AM, Joachim Faulhaber wrote:
Hi,
there are open tickets:
https://svn.boost.org/trac/boost/ticket/7430 (ptr_container: warning fixes) https://svn.boost.org/trac/boost/ticket/7433 (crc: runtime check issue)
These look okay to me.
https://svn.boost.org/trac/boost/ticket/7435 (format: crash)
This patch is wrong. I'll add further explanation in the ticket.
that we have submitted 5 months ago. Along with the bug reports there are code patches for the reported issues, that have been tested and used in production code on different platforms.
We would kindly ask the maintainers to look at these issues and to apply the patches to the library code, so they can be integrated in the next release.
In case the maintainers do not have time to fix these issues within say the next 2 weeks, I (Joachim) would like to ask for permission to apply the patches. Fixing the issues would be very helpful for us, so we don't have to patch the code locally for every new version.
In Christ, Steven Watanabe
Hi Steven, Thorsten, list, thanks for the help with the open tickets so far ... 2013/2/14 Steven Watanabe <watanabesj@gmail.com>:
AMDG
On 02/12/2013 08:31 AM, Joachim Faulhaber wrote:
Hi,
there are open tickets:
https://svn.boost.org/trac/boost/ticket/7430 (ptr_container: warning fixes)
I have committed this fix to the trunk. Do I have to merge them into the release branch for the next release or will Thorsten take care of that?
https://svn.boost.org/trac/boost/ticket/7433 (crc: runtime check issue)
These look okay to me.
For crc, the code in the trunk is pretty different to the code on the release branch. But it seems the code on trunk is not merged to releases. So I would like to apply the patch to the release once the release branch is open again.
https://svn.boost.org/trac/boost/ticket/7435 (format: crash)
This patch is wrong. I'll add further explanation in the ticket.
I looked into this in more detail today. There is a better way to fix this bug. I'll add more information to the ticket. Regards, Joachim -- Interval Container Library [Boost.Icl] http://www.joachim-faulhaber.de
AMDG On 02/22/2013 07:51 AM, Joachim Faulhaber wrote:
https://svn.boost.org/trac/boost/ticket/7433 (crc: runtime check issue)
These look okay to me.
For crc, the code in the trunk is pretty different to the code on the release branch. But it seems the code on trunk is not merged to releases.
Oh. That would be a problem. It's not good for the trunk and release branches to be out of sync for too long.
So I would like to apply the patch to the release once the release branch is open again.
Unless you feel like doing it, I'll review and merge all the CRC changes this weekend. In Christ, Steven Watanabe
2013/2/22 Steven Watanabe <watanabesj@gmail.com>:
AMDG
On 02/22/2013 07:51 AM, Joachim Faulhaber wrote:
https://svn.boost.org/trac/boost/ticket/7433 (crc: runtime check issue)
These look okay to me.
For crc, the code in the trunk is pretty different to the code on the release branch. But it seems the code on trunk is not merged to releases.
Oh. That would be a problem. It's not good for the trunk and release branches to be out of sync for too long.
So I would like to apply the patch to the release once the release branch is open again.
Unless you feel like doing it,
not really
I'll review and merge all the CRC changes this weekend.
Great! -- Interval Container Library [Boost.Icl] http://www.joachim-faulhaber.de
AMDG On 02/22/2013 09:15 AM, Joachim Faulhaber wrote:
I'll review and merge all the CRC changes this weekend.
Great!
*Sigh* Trunk crc is broken. I'm going to have to fix it first. In Christ, Steven Watanabe
participants (3)
-
Joachim Faulhaber -
Steven Watanabe -
Thorsten Ottosen