Opened 12 years ago

Closed 12 years ago

#5184 closed Patches (invalid)

optional_test.cpp confuses libc++

Reported by: Chris Jefferson Owned by: Fernando Cacciola
Milestone: To Be Determined Component: optional
Version: Boost 1.45.0 Severity: Problem
Keywords: Cc:

Description

The advised way of specialising swap is just to write:

void swap(mytype&, mytype&)

rather than:

template<> void swap(mytype&, mytype&)

using template<> confuses libc++, which uses namespace versioning.

I don't believe these template<> add anything to the test, which still passes on g++ without them. The attached patch removes them from optional_test.cpp

Attachments (1)

optional_test.cpp.patch (1000 bytes ) - added by Chris Jefferson 12 years ago.
Patch to libs/optional/test/optional_test.cpp

Download all attachments as: .zip

Change History (2)

by Chris Jefferson, 12 years ago

Attachment: optional_test.cpp.patch added

Patch to libs/optional/test/optional_test.cpp

comment:1 by Chris Jefferson, 12 years ago

Resolution: invalid
Status: newclosed

Actually, this has turned out to be a compiler bug in clang which will get fixed, so there is no need for a patch.

Note: See TracTickets for help on using tickets.