Boost C++ Libraries: Ticket #1488: Lots of warnings in ptr_container header files

Fri, 30 Nov 2007 21:34:28 GMT

Hi,

it may be a very stupid response but I do not see any warning at all. I see only the template parameters but it is also hard to scroll in the webpage.

I suggest you post the warnings - and only these - again.

anonymous — Fri, 30 Nov 2007 21:37:32 GMT

Replying to Jens Seidel <jensseidel@users.sf.net>:

Hi,

it may be a very stupid response but I do not see any warning at all. I see only the template parameters but it is also hard to scroll in the webpage.

I suggest you post the warnings - and only these - again.

I don't really understand what you're trying to say. What's the problem? Please elaborate.

Sun, 02 Dec 2007 16:41:18 GMT

Replying to anonymous:

I don't really understand what you're trying to say. What's the problem? Please elaborate.

Isn't it obvious? Look at the webpage (not your mail copy) http://svn.boost.org/trac/boost/ticket/1488 and tell me a single warning!

Ah, forget it. I copied the message and included it in another editor and could now read "warning C4512: '[snip']: assignment operator could not be generated"

Previously I read only the first columns and lines similar to: 1> with 1> [

I'm sure I even scrolled to the right side but navigation in the code snippet is very, very ugly! I wish a normal file attachment had beed used instead of unreadable Wiki syntax.

anonymous — Sun, 02 Dec 2007 16:44:46 GMT

Replying to Jens Seidel <jensseidel@users.sf.net>:

Ah, forget it. I copied the message and included it in another editor and could now read

I'm sure I even scrolled to the right side but navigation in the code snippet is very, very ugly! I wish a normal file attachment had beed used instead of unreadable Wiki syntax.

I still have yet to see the problem. I can read everything just fine. At the bottom of the code block is a horizontal scroll bar. You can use that to view the parts that are clipped by the edge of the code block box.

Sun, 02 Dec 2007 16:55:58 GMT

Replying to anonymous:

Replying to Jens Seidel <jensseidel@users.sf.net>:

I'm sure I even scrolled to the right side but navigation in the code snippet is very, very ugly! I wish a normal file attachment had beed used instead of unreadable Wiki syntax.

I still have yet to see the problem. I can read everything just fine. At the bottom of the code block is a horizontal scroll bar. You can use that to view the parts that are clipped by the edge of the code block box.

Maybe it is a browser issue. I use Konqueror in Debian Sid.

I used the horizontal scrollbar at the bottom. But this way I see only the last lines of the bug report which do not include "warning C[0-9]+" but only the callstack and something as T=Object,.

Probably the solution is to check the first columns which are displayed very carefully, after this scroll a few columns to the right and restart reading at the top, the rescroll again and ...

OK, the problem is that the few "warning C[0-9]+" strings are just not easy to find in the large output. Just remember this for future reports or use maybe colors, ...

anonymous — Sun, 02 Dec 2007 17:05:03 GMT

Replying to Jens Seidel <jensseidel@users.sf.net>:

Maybe it is a browser issue. I use Konqueror in Debian Sid.

I used the horizontal scrollbar at the bottom. But this way I see only the last lines of the bug report which do not include "warning C[0-9]+" but only the callstack and something as T=Object,.

Probably the solution is to check the first columns which are displayed very carefully, after this scroll a few columns to the right and restart reading at the top, the rescroll again and ...

OK, the problem is that the few "warning C[0-9]+" strings are just not easy to find in the large output. Just remember this for future reports or use maybe colors, ...

Don't blame me, blame Visual Studio. I would make it more readable if I could. In any case, the majority of people will be able to read it just fine. I'll attach a file later on for the visually impaired.

Thorsten Ottosen — Mon, 03 Dec 2007 08:43:53 GMT

Please submit a patch that get's rid of these warnings

Thanks

-Thorsten

attachment set

Mon, 03 Dec 2007 12:34:41 GMT

attachment → reversible_ptr_container.patch

silence 4127 warnings from reversible_ptr_container.hpp

Mon, 03 Dec 2007 12:38:47 GMT

Attached is a patch that silences the 4127 warnings from reversible_ptr_container.hpp on VC8/VC9.

The patch turns off the warning for the whole file, as it comes from several places (e.g. the usage of allow_null_values). Is this ok, or would you prefer the warnings disabled more locally?

status changed

Thorsten Ottosen — Mon, 03 Dec 2007 15:13:55 GMT

status new → assigned

I have applied it to trunk

-Thorsten

anonymous — Mon, 03 Dec 2007 19:02:57 GMT

Thanks for updating the trunk. While most of the warnings are gone, there's still one more:

1>c:\it\personal\collision\sdk\boost\boost\ptr_container\detail\move.hpp(23) : warning C4512: 'boost::ptr_container_detail::move_ptrs::move_source<Ptr>' : assignment operator could not be generated
1>        with
1>        [
1>            Ptr=boost::ptr_container_detail::static_move_ptr<Object,boost::ptr_container_detail::clone_deleter<boost::ptr_container_detail::reversible_ptr_container<boost::ptr_container_detail::sequence_config<Object,std::vector<void *,std::allocator<void *>>>,boost::heap_clone_allocator>::null_clone_allocator<false>>>
1>        ]
1>        c:\it\personal\collision\sdk\boost\boost\ptr_container\ptr_sequence_adapter.hpp(182) : see reference to class template instantiation 'boost::ptr_container_detail::move_ptrs::move_source<Ptr>' being compiled
1>        with
1>        [
1>            Ptr=boost::ptr_container_detail::static_move_ptr<Object,boost::ptr_container_detail::clone_deleter<boost::ptr_container_detail::reversible_ptr_container<boost::ptr_container_detail::sequence_config<Object,std::vector<void *,std::allocator<void *>>>,boost::heap_clone_allocator>::null_clone_allocator<false>>>
1>        ]
1>        c:\it\personal\collision\sdk\boost\boost\ptr_container\ptr_sequence_adapter.hpp(179) : while compiling class template member function 'void boost::ptr_sequence_adapter<T,VoidPtrSeq,CloneAllocator>::push_back(Object *)'
1>        with
1>        [
1>            T=Object,
1>            VoidPtrSeq=std::vector<void *,std::allocator<void *>>,
1>            CloneAllocator=boost::heap_clone_allocator
1>        ]
1>        c:\it\personal\collision\sdk\boost\boost\ptr_container\ptr_vector.hpp(35) : see reference to class template instantiation 'boost::ptr_sequence_adapter<T,VoidPtrSeq,CloneAllocator>' being compiled
1>        with
1>        [
1>            T=Object,
1>            VoidPtrSeq=std::vector<void *,std::allocator<void *>>,
1>            CloneAllocator=boost::heap_clone_allocator
1>        ]
1>        c:\it\personal\collision\source\world.h(20) : see reference to class template instantiation 'boost::ptr_vector<T>' being compiled
1>        with
1>        [
1>            T=Object
1>        ]

attachment set

Mon, 03 Dec 2007 22:48:51 GMT

attachment → move.patch

patch to silence C4512 warning on VC8+

Mon, 03 Dec 2007 22:50:21 GMT

Patch to move.hpp to silence C4512 warning on VC8+ attached

anonymous — Mon, 03 Dec 2007 23:03:44 GMT

Replying to Richard Webb <richard.webb@boldonjames.com>:

Patch to move.hpp to silence C4512 warning on VC8+ attached

Thanks for your patch. I tried to apply the patch using TortoiseSVN however it is saying that the patch is out of date? I'm not sure what this means.

Tue, 04 Dec 2007 09:53:25 GMT

I'm not sure either, but i tried applying the patch locally before submitting it and it was ok then.

Thorsten Ottosen — Tue, 04 Dec 2007 14:42:35 GMT

I applied it to trunk, and can merge it to the release tomorrow.

-Thorsten

anonymous — Tue, 04 Dec 2007 16:42:29 GMT

Replying to nesotto:

I applied it to trunk, and can merge it to the release tomorrow.

-Thorsten

It seems like all the warnings have gone. Thanks a lot for this, you did a great job. Keep up the great work on BOOST guys. I love it!

PS: How do I resolve this ticket?

status changed; resolution set

Thorsten Ottosen — Wed, 05 Dec 2007 17:34:02 GMT

status assigned → closed
resolution → fixed