From f3d0591abfb68c7576029f9b69905345e0569442 Mon Sep 17 00:00:00 2001
From: Anna Henningsen <anna@addaleax.net>
Date: Mon, 21 Jan 2019 15:35:17 +0100
Subject: [PATCH] report: use `uv_handle_type_name()` to get handle type

PR-URL: https://github.com/nodejs/node/pull/25610
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
---
 src/node_report_utils.cc | 65 +++++-----------------------------------
 1 file changed, 7 insertions(+), 58 deletions(-)

diff --git a/src/node_report_utils.cc b/src/node_report_utils.cc
index 309cf8166051fa..2cd92599b0a91e 100644
--- a/src/node_report_utils.cc
+++ b/src/node_report_utils.cc
@@ -105,65 +105,26 @@ void ReportPath(uv_handle_t* h, std::ostringstream& out) {
 
 // Utility function to walk libuv handles.
 void WalkHandle(uv_handle_t* h, void* arg) {
-  std::string type;
+  const char* type = uv_handle_type_name(h->type);
   std::ostringstream data;
   JSONWriter* writer = static_cast<JSONWriter*>(arg);
   uv_any_handle* handle = reinterpret_cast<uv_any_handle*>(h);
 
-  // List all the types so we get a compile warning if we've missed one,
-  // (using default: supresses the compiler warning).
   switch (h->type) {
-    case UV_UNKNOWN_HANDLE:
-      type = "unknown";
-      break;
-    case UV_ASYNC:
-      type = "async";
-      break;
-    case UV_CHECK:
-      type = "check";
-      break;
-    case UV_FS_EVENT: {
-      type = "fs_event";
-      ReportPath(h, data);
-      break;
-    }
-    case UV_FS_POLL: {
-      type = "fs_poll";
+    case UV_FS_EVENT:
+    case UV_FS_POLL:
       ReportPath(h, data);
       break;
-    }
-    case UV_HANDLE:
-      type = "handle";
-      break;
-    case UV_IDLE:
-      type = "idle";
-      break;
-    case UV_NAMED_PIPE:
-      type = "pipe";
-      break;
-    case UV_POLL:
-      type = "poll";
-      break;
-    case UV_PREPARE:
-      type = "prepare";
-      break;
-    case UV_PROCESS: {
-      type = "process";
+    case UV_PROCESS:
       data << "pid: " << handle->process.pid;
       break;
-    }
-    case UV_STREAM:
-      type = "stream";
-      break;
-    case UV_TCP: {
-      type = "tcp";
+    case UV_TCP:
+    case UV_UDP:
       ReportEndpoints(h, data);
       break;
-    }
     case UV_TIMER: {
       uint64_t due = handle->timer.timeout;
       uint64_t now = uv_now(handle->timer.loop);
-      type = "timer";
       data << "repeat: " << uv_timer_get_repeat(&(handle->timer));
       if (due > now) {
         data << ", timeout in: " << (due - now) << " ms";
@@ -174,22 +135,15 @@ void WalkHandle(uv_handle_t* h, void* arg) {
     }
     case UV_TTY: {
       int height, width, rc;
-      type = "tty";
       rc = uv_tty_get_winsize(&(handle->tty), &width, &height);
       if (rc == 0) {
         data << "width: " << width << ", height: " << height;
       }
       break;
     }
-    case UV_UDP: {
-      type = "udp";
-      ReportEndpoints(h, data);
-      break;
-    }
     case UV_SIGNAL: {
       // SIGWINCH is used by libuv so always appears.
       // See http://docs.libuv.org/en/v1.x/signal.html
-      type = "signal";
       data << "signum: " << handle->signal.signum
 #ifndef _WIN32
            << " (" << node::signo_string(handle->signal.signum) << ")"
@@ -197,12 +151,7 @@ void WalkHandle(uv_handle_t* h, void* arg) {
            << "";
       break;
     }
-    case UV_FILE:
-      type = "file";
-      break;
-    // We shouldn't see "max" type
-    case UV_HANDLE_TYPE_MAX:
-      type = "max";
+    default:
       break;
   }