#1535 closed Patches (fixed)
[result_of] remove unneeded headers
Reported by: | 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)
Change History (10)
by , 15 years ago
Attachment: | result_of_header.patch added |
---|
comment:1 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 by , 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 , 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 , 14 years ago
Milestone: | Boost 1.36.0 → Boost 1.38.0 |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
Version: | Boost Development Trunk → Boost Release Branch |
Any chance of this finding it's way into 1.38?
Thanks, Richard Webb
comment:5 by , 13 years ago
Component: | function → utility |
---|
comment:6 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
(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 , 12 years ago
Milestone: | Boost 1.38.0 → Boost 1.44.0 |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
Reopening, because this change has been reverted on the release branch.
comment:8 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
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.
remove unneeded
#include