Opened 12 years ago

Closed 11 years ago

#5235 closed Patches (fixed)

uuid unneccasary test header upsets clang

Reported by: Chris Jefferson Owned by: Andy Tompkins
Milestone: To Be Determined Component: uuid
Version: Boost Development Trunk Severity: Problem
Keywords: Cc:

Description

uuid has a test header called "lightweight_test_ex.hpp", which defines test_ne_impl. I assume this was an extension to the boost lightweight_test, but now it is in lightweight_test.hpp.

This makes the header unnecessary, but further clang (correctly) rejects the fact that test_ne_impl is defined twice.

This patch removes the header, and references to it. This fixes clang, and doesn't break g++ (I didn't test any other compilers, but I can't see why the would have a problem).

Attachments (1)

uuid_test.patch (3.8 KB ) - added by Chris Jefferson 12 years ago.
Corrected patch for uuid testsuite

Download all attachments as: .zip

Change History (4)

by Chris Jefferson, 12 years ago

Attachment: uuid_test.patch added

Corrected patch for uuid testsuite

comment:1 by Chris Jefferson, 12 years ago

First patch deleted wrong file, sorry. This one is correct.

comment:2 by Andy Tompkins, 12 years ago

Committed to trunk.

comment:3 by Andy Tompkins, 11 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.