Modify

Ticket #5235 (closed Patches: fixed)

Opened 3 years ago

Last modified 3 years ago

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

uuid_test.patch Download (3.8 KB) - added by chrisj 3 years ago.
Corrected patch for uuid testsuite

Change History

Changed 3 years ago by chrisj

Corrected patch for uuid testsuite

comment:1 Changed 3 years ago by chrisj

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

comment:2 Changed 3 years ago by atompkins

Committed to trunk.

comment:3 Changed 3 years ago by atompkins

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

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
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.