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/tty_wrap.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/tty_wrap.h')
-rw-r--r-- | src/tty_wrap.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/tty_wrap.h b/src/tty_wrap.h index df0c4b7c145..fdf07e4242c 100644 --- a/src/tty_wrap.h +++ b/src/tty_wrap.h @@ -24,12 +24,13 @@ #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS -#include "env.h" #include "uv.h" #include "stream_wrap.h" namespace node { +class Environment; + class TTYWrap : public LibuvStreamWrap { public: static void Initialize(v8::Local<v8::Object> target, |