Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 14:04 +0100, Jonathan Wakely wrote: On 13/10/14 13:10 +0100, Jonathan Wakely wrote: On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 13:10 +0100, Jonathan Wakely wrote: On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the patch. Thanks for merging and improving my pat

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the patch. Thanks for merging and improving my patches! If you have the time maybe you could also

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 13/10/14 12:35 +0200, Rüdiger Sonderfeld wrote: On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: Fixed (the new function should be in namespace std!) and applied to trunk. Thanks for the patch. Thanks for merging and improving my patches! Sorry it's taken so long, I needed to get

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Rüdiger Sonderfeld
On Monday 13 October 2014 11:03:51 Jonathan Wakely wrote: > Fixed (the new function should be in namespace std!) and applied to > trunk. Thanks for the patch. Thanks for merging and improving my patches! If you have the time maybe you could also look at the put_time patch. Unlike get_time it do

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-10-13 Thread Jonathan Wakely
On 16/04/14 17:06 +0200, Rüdiger Sonderfeld wrote: C++11 [ptr.align]. This should probably not be inline. But for now this avoids any ABI changes. * libstdc++-v3/testsuite/20_util/align/1.cc: New file. * libstdc++-v3/include/std/memory (align): New function. Fixed (the new function should be

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-04-16 Thread Jonathan Wakely
On 16/04/14 16:19 +0100, Jonathan Wakely wrote: On 16/04/14 17:06 +0200, Rüdiger Sonderfeld wrote: C++11 [ptr.align]. This should probably not be inline. But for now this avoids any ABI changes. Adding new non-member functions is fine ABI purposes (adding new virtual functions is not). Act

Re: [PATCH 1/2] libstdc++: Add std::align.

2014-04-16 Thread Jonathan Wakely
On 16/04/14 17:06 +0200, Rüdiger Sonderfeld wrote: C++11 [ptr.align]. This should probably not be inline. But for now this avoids any ABI changes. Adding new non-member functions is fine ABI purposes (adding new virtual functions is not).

[PATCH 1/2] libstdc++: Add std::align.

2014-04-16 Thread Rüdiger Sonderfeld
C++11 [ptr.align]. This should probably not be inline. But for now this avoids any ABI changes. * libstdc++-v3/testsuite/20_util/align/1.cc: New file. * libstdc++-v3/include/std/memory (align): New function. --- libstdc++-v3/include/std/memory | 35 + libstdc++-v3/testsuit