From 7f39aef9123130acd171897ba0ef83a8b94387d2 Mon Sep 17 00:00:00 2001
From: Tim Neutkens <tim@timneutkens.nl>
Date: Fri, 28 Sep 2018 14:32:26 +0200
Subject: [PATCH] Remove obsolete check in dev-server (#5320)

Remove `if(this.hotReloader)` as it's always guaranteed to be there.

https://github.com/zeit/next.js/pull/5317#discussion_r221233064
---
 server/next-dev-server.js | 18 +++++-------------
 1 file changed, 5 insertions(+), 13 deletions(-)

diff --git a/server/next-dev-server.js b/server/next-dev-server.js
index b22229d8..44139369 100644
--- a/server/next-dev-server.js
+++ b/server/next-dev-server.js
@@ -21,23 +21,17 @@ export default class DevServer extends Server {
 
   async prepare () {
     await super.prepare()
-    if (this.hotReloader) {
-      await this.hotReloader.start()
-    }
+    await this.hotReloader.start()
   }
 
   async close () {
-    if (this.hotReloader) {
-      await this.hotReloader.stop()
-    }
+    await this.hotReloader.stop()
   }
 
   async run (req, res, parsedUrl) {
-    if (this.hotReloader) {
-      const {finished} = await this.hotReloader.run(req, res, parsedUrl)
-      if (finished) {
-        return
-      }
+    const {finished} = await this.hotReloader.run(req, res, parsedUrl)
+    if (finished) {
+      return
     }
 
     return super.run(req, res, parsedUrl)
@@ -117,8 +111,6 @@ export default class DevServer extends Server {
   }
 
   async getCompilationError (page) {
-    if (!this.hotReloader) return
-
     const errors = await this.hotReloader.getCompilationErrors(page)
     if (errors.length === 0) return