c4milo / c4milo/node-webkit-agent

startProfiling and stopProfiling should agree on the default name

Aperta
#55 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
JavaScript
Stelle
1.1k
Fork
77
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

In [v8-profiler.js](https://github.com/c4milo/node-webkit-agent/blob/master/lib/v8-profiler.js) it seems like [startProfiling](https://github.com/c4milo/node-webkit-agent/blob/master/lib/v8-profiler.js#L212) sets the default name to be

```
name = 'org.nodejs.profiles.cpu.user-initiated.' + (cpuCache.length + 1);
```

Meanwhile, [stopProfiling](https://github.com/c4milo/node-webkit-agent/blob/master/lib/v8-profiler.js#L220) sets the default name to be ''.

It seems like those should be consistent. I'm still wrapping my head around how everything fits together though, so please correct me if I'm confused.

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Direzione di ricerca

Leggi lib/v8-profiler.js in startProfiling e stopProfiling, confrontando la gestione del nome predefinito. Determina il comportamento predefinito condiviso previsto, rendi coerenti le due funzioni e verifica che il profiling possa essere avviato e arrestato con il nome predefinito.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript
Ambito
devtools
Tipo di issue
Bug
Difficoltà
2/5
Tempo stimato
1-3 ore
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.