Opened 13 years ago
Closed 13 years ago
#3893 closed Patches (fixed)
Fix compilation with Sun CC (without stlport)
Reported by: | Owned by: | Marshall Clow | |
---|---|---|---|
Milestone: | Boost 1.43.0 | Component: | array |
Version: | Boost 1.42.0 | Severity: | Problem |
Keywords: | suncc | Cc: |
Description
Apparently Sun CC with its (ancient) STL is not officially supported by Boost but I hope you can consider this patch which at least allows me to use boost/array.hpp (and, more importantly, many other headers which use it) in my code which can't be compiled with STLport.
The patch is basically trivial and simply adds a workaround similar to the existing one for MSVC just above for Sun CC:
-
boost/array.hpp
a b 78 70 reference, iterator, reference> > reverse_iterator; 79 71 typedef std::reverse_iterator<std::_Ptrit<value_type, difference_type, const_iterator, 80 72 const_reference, iterator, reference> > const_reverse_iterator; 73 #elif defined(__SUNPRO_CC) && !defined(_STLPORT_VERSION) 74 typedef std::reverse_iterator<iterator, std::random_access_iterator_tag, 75 value_type, reference, iterator, 76 difference_type> reverse_iterator; 77 typedef std::reverse_iterator<const_iterator, std::random_access_iterator_tag, 78 value_type, const_reference, const_iterator, 79 difference_type> const_reverse_iterator; 81 80 #else 82 81 // workaround for broken reverse_iterator implementations 83 82 typedef std::reverse_iterator<iterator,T> reverse_iterator;
TIA!
P.S. The window is apparently too narrow to show the patch without mangling it, hopefully you can still read (and apply it) however using the attached file.
Attachments (1)
Change History (6)
by , 13 years ago
Attachment: | boost-array.diff added |
---|
comment:1 by , 13 years ago
Component: | None → array |
---|---|
Owner: | set to |
follow-up: 3 comment:2 by , 13 years ago
Does this workaround need to be replicated around line 210 as well, or does the Sun CC not support partial template specialization?
comment:3 by , 13 years ago
Replying to marshall:
Does this workaround need to be replicated around line 210 as well, or does the Sun CC not support partial template specialization?
Yeah, it would need to be duplicated. Sun CC does support partial specialization.
comment:4 by , 13 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
Added in revision 60436. Will migrate to release before 1.43
comment:5 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Migrated to release branch in revision 60824
Patch against latest svn trunk