Modify

Opened 6 years ago

Closed 6 years ago

#5235 closed Patches (fixed)

uuid unneccasary test header upsets clang

Reported by: chrisj Owned by: atompkins
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 chrisj 6 years ago.
Corrected patch for uuid testsuite

Download all attachments as: .zip

Change History (4)

Changed 6 years ago by chrisj

Corrected patch for uuid testsuite

comment:1 Changed 6 years ago by chrisj

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

comment:2 Changed 6 years ago by atompkins

Committed to trunk.

comment:3 Changed 6 years ago by atompkins

  • Resolution set to fixed
  • Status changed from new to closed

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain atompkins.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.