Opened 10 years ago

Closed 9 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@…> 10 years ago.
Output from GCC
8197.patch​ (856 bytes ) - added by Andrii Sydorchuk 9 years ago.

Download all attachments as: .zip

Change History (12)

by Stepan Podoskin <stepik-777@…>, 10 years ago

Attachment: gcc-output.txt​ added

Output from GCC

comment:1 by Marshall Clow, 10 years ago

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 10 years ago by Marshall Clow (previous) (diff)

comment:2 by viboes, 10 years ago

Component: None → polygon
Owner: set to Lucanus Simonson

comment:3 by Andrii Sydorchuk, 10 years ago

Owner: changed from Lucanus Simonson to Andrii Sydorchuk

comment:4 by Andrii Sydorchuk, 10 years ago

Milestone: To Be Determined → Boost 1.54.0
Resolution: → fixed
Status: new → closed

comment:5 by Andrii Sydorchuk, 10 years ago

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

comment:6 by Andrii Sydorchuk, 10 years ago

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

comment:7 by Stepan Podoskin <stepik-777@…>, 9 years ago

Resolution: fixed
Status: closed → reopened

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

by Andrii Sydorchuk, 9 years ago

Attachment: 8197.patch​ added

comment:8 by Andrii Sydorchuk, 9 years ago

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 by Andrii Sydorchuk, 9 years ago

Milestone: Boost 1.54.0 → Boost 1.55.0
Status: reopened → new

comment:10 by Andrii Sydorchuk, 9 years ago

Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.