Modify

Opened 7 years ago

Closed 6 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 7 years ago.
Corrected patch for uuid testsuite

Download all attachments as: .zip

Change History (4)

Changed 7 years ago by Chris Jefferson

Attachment: uuid_test.patch added

Corrected patch for uuid testsuite

comment:1 Changed 7 years ago by Chris Jefferson

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

comment:2 Changed 7 years ago by Andy Tompkins

Committed to trunk.

comment:3 Changed 6 years ago by Andy Tompkins

Resolution: fixed
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Andy Tompkins.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.