I'm going to push with a minimal change from the reviewed version, wherein m_rememberedDisplayState
doesn't get modified until all displays in a given display changeset were processed. A temporary newRememberedDisplayState
variable takes care of deleting the matched items, which could theoretically happen for several displays. Practically speaking, I don't think it'll make a difference for most people and use cases, but perhaps it catches one or the other case that would have otherwise been missed.
↧
Jakob Petsovits commented on merge request !466 at Plasma / PowerDevil
↧