Opened 15 years ago

Closed 12 years ago

Last modified 12 years ago

#1535 closed Patches (fixed)

[result_of] remove unneeded headers

Reported by: Shunsuke Sogame <pstade.mb@…> Owned by: Douglas Gregor
Milestone: Boost 1.44.0 Component: utility
Version: Boost Release Branch Severity: Optimization
Keywords: result_of Cc:

Description

For some historical reason, utility/result_of.hpp includes unneeded headers. Here is a patch.

Attachments (1)

result_of_header.patch (677 bytes ) - added by Shunsuke Sogame <pstade.mb@…> 15 years ago.
remove unneeded #include

Download all attachments as: .zip

Change History (10)

by Shunsuke Sogame <pstade.mb@…>, 15 years ago

Attachment: result_of_header.patch added

remove unneeded #include

comment:1 by Douglas Gregor, 15 years ago

Resolution: fixed
Status: newclosed

(In [42234]) Reduce header dependencies, from Shunsuke Sogame. Fixes #1535

comment:2 by Eric Niebler, 15 years ago

(In [42236]) Merged revisions 42180-42235 via svnmerge from https://svn.boost.org/svn/boost/trunk

........

r42194 | anthonyw | 2007-12-19 23:46:00 -0800 (Wed, 19 Dec 2007) | 1 line

added missing parentheses

........

r42195 | anthonyw | 2007-12-20 00:37:02 -0800 (Thu, 20 Dec 2007) | 1 line

added hardware_concurrency support for apple, freebsd and sun platforms

........

r42196 | niels_dekker | 2007-12-20 01:09:44 -0800 (Thu, 20 Dec 2007) | 1 line

Removed "mark-expected-failures" from value_init_test, for both Borland and GCC, as value_init.hpp changeset [41942] provides a workaround for them.

........

r42202 | fmhess | 2007-12-20 06:39:53 -0800 (Thu, 20 Dec 2007) | 6 lines

Made putting the name of a typedef inside a "classname" element successfully produce a link to the typedef's reference documentation, or to the typedef's synopsis if that is all it has.

........

r42217 | vladimir_prus | 2007-12-20 13:18:47 -0800 (Thu, 20 Dec 2007) | 1 line

Explain a couple of gcc limitations with precompiled headers

........

r42220 | matias | 2007-12-20 13:56:39 -0800 (Thu, 20 Dec 2007) | 1 line

add explicit std::string initialization

........

r42221 | matias | 2007-12-20 13:59:18 -0800 (Thu, 20 Dec 2007) | 1 line

add license to each html page

........

r42222 | matias | 2007-12-20 14:15:11 -0800 (Thu, 20 Dec 2007) | 1 line

Time out: test_bimap_property_map @ gcc-3.4.6_linux_x86_64

........

r42223 | djowel | 2007-12-20 15:24:28 -0800 (Thu, 20 Dec 2007) | 1 line

Removing fusion variant adapter

........

r42224 | djowel | 2007-12-20 15:28:26 -0800 (Thu, 20 Dec 2007) | 1 line

Removing fusion variant adapter

........

r42227 | nmusatti | 2007-12-20 23:03:44 -0800 (Thu, 20 Dec 2007) | 1 line

Increased the latest supported version to 5.9.3, corresponding to the December 2007 update.

........

r42228 | anthonyw | 2007-12-21 02:54:59 -0800 (Fri, 21 Dec 2007) | 1 line

Updated thread ID test

........

r42229 | anthonyw | 2007-12-21 03:05:37 -0800 (Fri, 21 Dec 2007) | 1 line

Markup hardware_concurrency failure as expected on tru64 platforms, as support is not implemented

........

r42234 | dgregor | 2007-12-21 13:18:17 -0800 (Fri, 21 Dec 2007) | 1 line

Reduce header dependencies, from Shunsuke Sogame. Fixes #1535

........

comment:3 by Richard Webb <richard.webb@…>, 14 years ago

I've run into this issue while making a minimal set of Boost headers to build my app (causes it to need more headers than it should), and i notice that the change hasnt been merged to the release branch.

Can the change be merged for 1.38?

comment:4 by Richard Webb <richard.webb@…>, 14 years ago

Milestone: Boost 1.36.0Boost 1.38.0
Resolution: fixed
Status: closedreopened
Version: Boost Development TrunkBoost Release Branch

Any chance of this finding it's way into 1.38?

Thanks, Richard Webb

comment:5 by Daniel James, 13 years ago

Component: functionutility

comment:6 by Daniel James, 13 years ago

Resolution: fixed
Status: reopenedclosed

(In [60289]) Merge various result_of changes.

  • [42234] Reduce header dependencies, from Shunsuke Sogame. Fixes #1535
  • [45256] result_of implementation that makes use of C++0x decltype, from Daniel Walker. Fixes #862.
  • [48620] Fix result_of to work with const-qualified function pointers. Fixes #1310
  • [60052] Remove use of deprecated config macro in result_of.

comment:7 by rwebb <richard.webb@…>, 12 years ago

Milestone: Boost 1.38.0Boost 1.44.0
Resolution: fixed
Status: closedreopened

Reopening, because this change has been reverted on the release branch.

comment:8 by anonymous, 12 years ago

Resolution: fixed
Status: reopenedclosed

The contents of the four files in [60289]:

  • utility/detail/result_of_iterate.hpp
  • utility/result_of.hpp
  • utility/test/result_of_test.cpp
  • utility/utility.htm

appear to be the same as of revision 66797 (comparing the trunk, the release branch, and the 1.45.0 release)

I am closing this ticket. If someone thinks that this is incorrect, please reopen it.

comment:9 by Marshall Clow, 12 years ago

I (Marshall) was the one who closed this ticket.

Note: See TracTickets for help on using tickets.