Opened 13 years ago

Closed 13 years ago

#3524 closed Patches (fixed)

Unneeded Phoenix 2 include in Graph?

Reported by: François Barel <frabar666@…> Owned by: Andrew Sutton
Milestone: Boost 1.41.0 Component: graph
Version: Boost Development Trunk Severity: Regression
Keywords: graph include phoenix Cc:

Description

In r53226, a include of Phoenix 2 (boost/spirit/home/phoenix.hpp) was added to depth_first_search.hpp.

It's not immediately obvious to me why this is needed? The only other Phoenix references I could locate in Boost.Graph are in detail/read_graphviz_spirit.hpp -- and they are for Phoenix 1, not 2.

This new include is causing me trouble: to make it short, the result is I can no longer use both Boost.Graph and Spirit Classic in the same compilation unit (which worked fine before this change).

If this include is justified I'll find a workaround in my code -- but if it's not really needed, can it please be removed (trivial patch attached)?

Thanks

Attachments (1)

graph_no_phoenix2.diff (445 bytes ) - added by François Barel <frabar666@…> 13 years ago.
Remove Phoenix 2 include from Graph

Download all attachments as: .zip

Change History (2)

by François Barel <frabar666@…>, 13 years ago

Attachment: graph_no_phoenix2.diff added

Remove Phoenix 2 include from Graph

comment:1 by Jeremiah Willcock, 13 years ago

Resolution: fixed
Status: newclosed

(In [56868]) Removed unnecessary include of Phoenix 2; fixes #3524

Note: See TracTickets for help on using tickets.