From 99c1238f54a34acae7a4bde1c1e031fbf5d4d214 Mon Sep 17 00:00:00 2001
From: Alexandre Ferrando <alferpal@gmail.com>
Date: Sat, 26 Oct 2019 16:59:34 +0200
Subject: 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>
---
 src/tcp_wrap.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/tcp_wrap.h')

diff --git a/src/tcp_wrap.h b/src/tcp_wrap.h
index 0467a1c3f3..0099eedb4b 100644
--- a/src/tcp_wrap.h
+++ b/src/tcp_wrap.h
@@ -25,11 +25,12 @@
 #if defined(NODE_WANT_INTERNALS) && NODE_WANT_INTERNALS
 
 #include "async_wrap.h"
-#include "env.h"
 #include "connection_wrap.h"
 
 namespace node {
 
+class Environment;
+
 class TCPWrap : public ConnectionWrap<TCPWrap, uv_tcp_t> {
  public:
   enum SocketType {
-- 
cgit v1.2.3