lib: refactor to use more primordials in internal/histogram.js

Co-authored-by: Antoine du Hamel <[email protected]> PR-URL: #36455 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Rich Trott <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: 2 people

lib/internal/histogram.js

@@ -5,7 +5,7 @@ const{
 }=require('internal/util');
 
 const{ format }=require('util');
-const{Map, Symbol }=primordials;
+const{SafeMap, Symbol }=primordials;
 
 const{
 ERR_INVALID_ARG_TYPE,
@@ -19,11 +19,10 @@ const kHandle = Symbol('kHandle');
 // record various metrics. This Histogram class provides a
 // generally read-only view of the internal histogram.
 classHistogram{
- #handle =undefined;
- #map =newMap();
+ #map =newSafeMap();
 
 constructor(internal){
-this.#handle=internal;
+this[kHandle]=internal;
 }
 
 [kInspect](){
@@ -39,23 +38,23 @@ class Histogram{
 }
 
 getmin(){
-returnthis.#handle ? this.#handle.min() : undefined;
+returnthis[kHandle]?.min();
 }
 
 getmax(){
-returnthis.#handle ? this.#handle.max() : undefined;
+returnthis[kHandle]?.max();
 }
 
 getmean(){
-returnthis.#handle ? this.#handle.mean() : undefined;
+returnthis[kHandle]?.mean();
 }
 
 getexceeds(){
-returnthis.#handle ? this.#handle.exceeds() : undefined;
+returnthis[kHandle]?.exceeds();
 }
 
 getstddev(){
-returnthis.#handle ? this.#handle.stddev() : undefined;
+returnthis[kHandle]?.stddev();
 }
 
 percentile(percentile){
@@ -65,26 +64,22 @@ class Histogram{
 if(percentile<=0||percentile>100)
 thrownewERR_INVALID_ARG_VALUE.RangeError('percentile',percentile);
 
-returnthis.#handle ? this.#handle.percentile(percentile) : undefined;
+returnthis[kHandle]?.percentile(percentile);
 }
 
 getpercentiles(){
 this.#map.clear();
-if(this.#handle)
-this.#handle.percentiles(this.#map);
+this[kHandle]?.percentiles(this.#map);
 returnthis.#map;
 }
 
 reset(){
-if(this.#handle)
-this.#handle.reset();
+this[kHandle]?.reset();
 }
 
 [kDestroy](){
-this.#handle=undefined;
+this[kHandle]=undefined;
 }
-
-get[kHandle](){returnthis.#handle;}
 }
 
 module.exports={