Review request: 8003935: Simplify the needed includes for using Thread::current()
stefan.karlsson at oracle.com
Tue Nov 27 05:20:13 PST 2012
Thanks David, Rickard and Coleen for the reviews.
On 11/23/2012 12:54 PM, Stefan Karlsson wrote:
> Today, whenever we use Thread::current() we have to all these lines:
> #include "runtime/thread.hpp"
> #ifdef TARGET_OS_FAMILY_linux
> # include "thread_linux.inline.hpp"
> #ifdef TARGET_OS_FAMILY_solaris
> # include "thread_solaris.inline.hpp"
> #ifdef TARGET_OS_FAMILY_windows
> # include "thread_windows.inline.hpp"
> #ifdef TARGET_OS_FAMILY_bsd
> # include "thread_bsd.inline.hpp"
> This patch hides this dispatching in a new file named
> thread.inline.hpp. Now we only have to include thread.inline.hpp.
> Some background to these includes: This type of dispatching was
> introduced into the source files when we removed the includeDB files.
> We discussed whether we should use "dispatch files" or not for this
> kind platform dependent includes. The decision was to not create
> dispatch files for all types of platform at that time, but instead
> manually create them when we thought it was warranted.
More information about the hotspot-dev