servo/webrender

Multiwindow example panics when debug feature is enabled

Offen

#3.860 geöffnet am 17.02.2020

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)Rust (319 Forks)github user discovery
help wantedtype: bug

Repository-Metriken

Stars
 (3.355 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 20T 5h) (1 gemergte PR in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Untersuche die doppelte Initialisierung von thread profiler und füge eine Bedingung hinzu, um sie zu verhindern, wenn der Profiler bereits initialisiert ist.
Tech Stack
rust
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
85