search

X-Profiler / xprofiler

🌀An addon for node.js, which supporting output performance log and real-time profiling through sampling.
Other
430 stars 57 forks source link

refactor: consolidate dump actions #210

Closed legendecas closed 2 years ago

legendecas commented 2 years ago
  1. 合并 DumpData，使用 std::unique_ptr 明确 owner 语义、避免需要手动释放；
  2. 使用 detached 线程，合理回收 profiling watchdog thread.
codecov[bot] commented 2 years ago

Codecov Report

Merging #210 (6d0aefd) into master (289770f) will not change coverage. The diff coverage is n/a.

:exclamation: Current head 6d0aefd differs from pull request most recent head bc73f8a. Consider uploading reports for the commit bc73f8a to get more accurate results

@@            Coverage Diff            @@
##            master      #210   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9         9           
  Lines          296       296           
=========================================
  Hits           296       296

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more