canonical/mir

WindowManager::add_display()/remove_display()

Open

#2148 aperta il 20 ago 2021

Vedi su GitHub
 (4 commenti) (0 reazioni) (1 assegnatario)C++ (137 fork)auto 404
Good first issue

Metriche repository

Star
 (775 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

In one of the implementations of this interface SystemCompositorWindowManager we have:

    void add_display(mir::geometry::Rectangle const& area) override
    __attribute__((deprecated("Mir doesn't reliably call this: it is ignored. Use add_display_for_testing() instead")));

And, in this implementation, the outputs are tracked by the ActiveOutputsListener interface.

But the other implementation SystemCompositorWindowManager uses this function to track the outputs.

There's clearly something wrong with the diagnostic, one of the implementations, or several of these.

Guida contributor

Direzione di ricerca
Investiga le due implementazioni dell'interfaccia WindowManager. Determina quale implementazione utilizza correttamente add display e aggiorna l'annotazione deprecata di conseguenza.
Tech stack
cpp
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.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
C++Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70