diff options
author | Alexandre Ferrando <alferpal@gmail.com> | 2019-10-26 17:59:34 +0300 |
---|---|---|
committer | Anna Henningsen <anna@addaleax.net> | 2019-10-31 16:11:05 +0300 |
commit | 99c1238f54a34acae7a4bde1c1e031fbf5d4d214 (patch) | |
tree | e194c67fbb35922186bf33b7b391eb3b78e76754 /src/js_stream.h | |
parent | 7477f90b5c9e903b1680cae7e826b5bb8ed545f5 (diff) |
src: change env.h includes for forward declarations
Due to how the Environment class is used through the codebase,
there are a lot of includes referencing either env.h or env-inl.h.
This can cause that when any development touches those libraries,
a lot of files have to be recompiled.
This commit attempts to change those includes by forward declarations
when possible to mitigate the issue.
Refs: https://github.com/nodejs/node/issues/27531
PR-URL: https://github.com/nodejs/node/pull/30133
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: David Carlier <devnexen@gmail.com>
Reviewed-By: Franziska Hinkelmann <franziska.hinkelmann@gmail.com>
Diffstat (limited to 'src/js_stream.h')
-rw-r--r-- | src/js_stream.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/js_stream.h b/src/js_stream.h index 8198a6bb36a..460ac23bc98 100644 --- a/src/js_stream.h +++ b/src/js_stream.h @@ -4,12 +4,13 @@ #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS #include "async_wrap.h" -#include "env.h" #include "stream_base.h" #include "v8.h" namespace node { +class Environment; + class JSStream : public AsyncWrap, public StreamBase { public: static void Initialize(v8::Local<v8::Object> target, |