From 7a05587a82e370dc7cd416923024fbaa74d4b7c9 Mon Sep 17 00:00:00 2001
From: Darshan Sen <raisinten@gmail.com>
Date: Fri, 18 Mar 2022 20:21:53 +0530
Subject: [PATCH] src: get rid of SetVerbose(true) calls

Signed-off-by: Darshan Sen <raisinten@gmail.com>
---
 src/udp_wrap.cc | 35 +++++++++++++++++++++++------------
 1 file changed, 23 insertions(+), 12 deletions(-)

diff --git a/src/udp_wrap.cc b/src/udp_wrap.cc
index 0fb15ce0a0d4ad..127a1a6e5d8fe7 100644
--- a/src/udp_wrap.cc
+++ b/src/udp_wrap.cc
@@ -732,12 +732,17 @@ void UDPWrap::OnRecv(ssize_t nread,
 
   Local<Object> address;
   {
-    TryCatchScope try_catch(env);
-    try_catch.SetVerbose(true);
-    DCHECK(try_catch.IsVerbose());
-    if (!AddressToJS(env, addr).ToLocal(&address)) {
-      DCHECK(try_catch.HasCaught() && !try_catch.HasTerminated());
-      argv[2] = try_catch.Exception();
+    bool has_caught = false;
+    {
+      TryCatchScope try_catch(env);
+      if (!AddressToJS(env, addr).ToLocal(&address)) {
+        DCHECK(try_catch.HasCaught() && !try_catch.HasTerminated());
+        argv[2] = try_catch.Exception();
+        DCHECK(!argv[2].IsEmpty());
+        has_caught = true;
+      }
+    }
+    if (has_caught) {
       DCHECK(!argv[2].IsEmpty());
       MakeCallback(env->onerror_string(), arraysize(argv), argv);
       return;
@@ -746,17 +751,23 @@ void UDPWrap::OnRecv(ssize_t nread,
 
   Local<ArrayBuffer> ab = ArrayBuffer::New(isolate, std::move(bs));
   {
-    TryCatchScope try_catch(env);
-    try_catch.SetVerbose(true);
-    DCHECK(try_catch.IsVerbose());
-    if (!Buffer::New(env, ab, 0, ab->ByteLength()).ToLocal(&argv[2])) {
-      DCHECK(try_catch.HasCaught() && !try_catch.HasTerminated());
-      argv[2] = try_catch.Exception();
+    bool has_caught = false;
+    {
+      TryCatchScope try_catch(env);
+      if (!Buffer::New(env, ab, 0, ab->ByteLength()).ToLocal(&argv[2])) {
+        DCHECK(try_catch.HasCaught() && !try_catch.HasTerminated());
+        argv[2] = try_catch.Exception();
+        DCHECK(!argv[2].IsEmpty());
+        has_caught = true;
+      }
+    }
+    if (has_caught) {
       DCHECK(!argv[2].IsEmpty());
       MakeCallback(env->onerror_string(), arraysize(argv), argv);
       return;
     }
   }
+
   argv[3] = address;
   MakeCallback(env->onmessage_string(), arraysize(argv), argv);
 }