servo/webrender

Multiwindow example panics when debug feature is enabled

Aperta

#3860 aperta il 17 feb 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Rust (319 fork)github user discovery
help wantedtype: bug

Metriche repository

Star
 (3355 stelle)
Metriche merge PR
 (Merge medio 20g 5h) (1 PR mergiata in 30 g)

Descrizione

I try cargo run --bin multiwindow --features=debug and get

thread 'main' panicked at 'assertion failed: profiler.borrow().is_none()', /Users/pavel/.cargo/registry/src/github.com-1ecc6299db9ec823/thread_profiler-0.1.3/src/lib.rs:95:17
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.
fish: 'cargo run --bin multiwindow --f…' terminated by signal SIGABRT (Abort)

As far as I understand the problem is in double initializaition of thread_profiler for compositor thread. What do you think about moving it under wr_has_been_initialized condition?

https://github.com/SergeevPavel/webrender/commit/7a2587acb45ba6552f86c9743ce31ceb616f364c

Guida contributor

Direzione di ricerca
Investiga la doppia inizializzazione di thread profiler e aggiungi una condizione per evitarla quando il profiler è già inizializzato.
Tech stack
rust
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85