RFR: 8073389: Remove the include of resourceArea.hpp from classFileParser.hpp
Stefan Karlsson
stefan.karlsson at oracle.com
Wed Feb 18 09:35:46 UTC 2015
Hi,
Please review this patch to get rid of the inclusion of resourceArea.hpp
from classFileParser.hpp.
From the RFE:
The inclusion of resourceArea.hpp in classFileParser.hpp is causing
cyclic dependencies when I'm changing unrelated code. The main reason
for this is that a lot of implementation is put inside the
resourceArea.hpp file instead of a .cpp file.
I've opted to go the easy route now and get rid of the the
resourceArea.hpp dependency from classFileParser.hpp, but eventually it
would be good to fix that file.
This patch has to add explicit includes of resourceArea.hpp to other
.hpp files, that used to get their include from classFileParser.hpp. I
could have gotten rid of those dependencies as well, but I chose to not
do that for this patch.
http://cr.openjdk.java.net/~stefank/8073389/webrev.01/
https://bugs.openjdk.java.net/browse/JDK-8073389
Thanks,
StefanK
More information about the hotspot-dev
mailing list