Modify

Opened 5 years ago

Closed 4 years ago

#8197 closed Bugs (fixed)

range/algorithm conflicts with polygon

Reported by: Stepan Podoskin <stepik-777@…> Owned by: Andrii Sydorchuk
Milestone: Boost 1.55.0 Component: polygon
Version: Boost 1.53.0 Severity: Problem
Keywords: Cc:

Description

This fails to compile on GCC 4.7.2 (MinGW) but it can be compiled with MSVC 2010.

#include <boost/range/algorithm.hpp>
#include <boost/polygon/polygon.hpp>

Reversing order of includes fixes the problem. (It seems that boost/polygon documentation doesn't mention what headers should be included to use it so I just included polygon.hpp). GCC output attached.

Attachments (2)

gcc-output.txt (3.8 KB) - added by Stepan Podoskin <stepik-777@…> 5 years ago.
Output from GCC
8197.patch (856 bytes) - added by Andrii Sydorchuk 4 years ago.

Download all attachments as: .zip

Change History (12)

Changed 5 years ago by Stepan Podoskin <stepik-777@…>

Attachment: gcc-output.txt added

Output from GCC

comment:1 Changed 5 years ago by Marshall Clow

Interesting. Compiling with clang (and libc++), I don't get this error.

Just to be clear: I do get the error with gcc 4.7.2

Last edited 5 years ago by Marshall Clow (previous) (diff)

comment:2 Changed 5 years ago by viboes

Component: Nonepolygon
Owner: set to Lucanus Simonson

comment:3 Changed 5 years ago by Andrii Sydorchuk

Owner: changed from Lucanus Simonson to Andrii Sydorchuk

comment:4 Changed 5 years ago by Andrii Sydorchuk

Milestone: To Be DeterminedBoost 1.54.0
Resolution: fixed
Status: newclosed

comment:5 Changed 5 years ago by Andrii Sydorchuk

The issue is fixed on trunk and will go into Boost 1.54 release.

comment:6 Changed 5 years ago by Andrii Sydorchuk

(In [83918]) [Polygon] Merging from trunk for the upcoming release: 1) Updated Voronoi documentation. 2) Fixed #8026, #8197, #8257.

comment:7 Changed 4 years ago by Stepan Podoskin <stepik-777@…>

Resolution: fixed
Status: closedreopened

I still have this problem with boost 1.54.0 and gcc-4.8.1.

Changed 4 years ago by Andrii Sydorchuk

Attachment: 8197.patch added

comment:8 Changed 4 years ago by Andrii Sydorchuk

You are right, the fix was not merged successfully from trunk to release branch. I've just merged the fix into the release branch, and it will go live with the next release. Meanwhile, please use the patch (8197.patch) attached to this bug report.

comment:9 Changed 4 years ago by Andrii Sydorchuk

Milestone: Boost 1.54.0Boost 1.55.0
Status: reopenednew

comment:10 Changed 4 years ago by Andrii Sydorchuk

Resolution: fixed
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Andrii Sydorchuk.
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.