X-Git-Url: https://git.cweiske.de/enigma2.git/blobdiff_plain/ba02fb4aced5868d047a5bffbd2ed87583daee4d..31688e1b8f028059a700a92a8276c97928abd260:/components.py diff --git a/components.py b/components.py index e66e43b1..20827180 100644 --- a/components.py +++ b/components.py @@ -1,5 +1,6 @@ from enigma import * import time +import sys # some helper classes first: class HTMLComponent: @@ -20,12 +21,72 @@ class HTMLSkin: return res class GUISkin: - data = { } + def __init__(self): + self.data = { } + def createGUIScreen(self, parent): for (name, val) in self.items(): self.data[name] = { } val.GUIcreate(self.data[name], parent, None) + + def deleteGUIScreen(self): + for (name, val) in self.items(): + w = self.data[name]["instance"] + val.GUIdelete(self.data[name]) + del self.data[name] + + # note: you'll probably run into this assert. if this happens, don't panic! + # yes, it's evil. I told you that programming in python is just fun, and + # suddently, you have to care about things you don't even know. + # + # but calm down, the solution is easy, at least on paper: + # + # Each Component, which is a GUIComponent, owns references to each + # instantiated eWidget (namely in screen.data[name]["instance"], in case + # you care.) + # on deleteGUIscreen, all eWidget *must* (!) be deleted (otherwise, + # well, problems appear. I don't want to go into details too much, + # but this would be a memory leak anyway.) + # The assert beyond checks for that. It asserts that the corresponding + # eWidget is about to be removed (i.e., that the refcount becomes 0 after + # running deleteGUIscreen). + # (You might wonder why the refcount is checked for 2 and not for 1 or 0 - + # one reference is still hold by the local variable 'w', another one is + # hold be the function argument to sys.getrefcount itself. So only if it's + # 2 at this point, the object will be destroyed after leaving deleteGUIscreen.) + # + # Now, how to fix this problem? You're holding a reference somewhere. (References + # can only be hold from Python, as eWidget itself isn't related to the c++ + # way of having refcounted objects. So it must be in python.) + # + # It could be possible that you're calling deleteGUIscreen trough a call of + # a PSignal. For example, you could try to call session.close() in response + # to a Button::click. This will fail. (It wouldn't work anyway, as you would + # remove a dialog while running it. It never worked - enigma1 just set a + # per-mainloop variable on eWidget::close() to leave the exec()...) + # That's why Session supports delayed closes. Just call Session.close() and + # it will work. + # + # Another reason is that you just stored the data["instance"] somewhere. or + # added it into a notifier list and didn't removed it. + # + # If you can't help yourself, just ask me. I'll be glad to help you out. + # Sorry for not keeping this code foolproof. I really wanted to archive + # that, but here I failed miserably. All I could do was to add this assert. + assert sys.getrefcount(w) == 2, "too many refs hold to " + str(w) + + def close(self): + self.deleteGUIScreen() + del self.data + +# note: components can be used in multiple screens, so we have kind of +# two contexts: first the per-component one (self), then the per-screen (i.e.: +# per eWidget one), called "priv". In "priv", for example, the instance +# of the eWidget is stored. + +# GUI components have a "notifier list" of associated eWidgets to one component +# (as said - one component instance can be used at multiple screens) class GUIComponent: """ GUI component """ @@ -35,11 +96,21 @@ class GUIComponent: def GUIcreate(self, priv, parent, skindata): i = self.GUIcreateInstance(self, parent, skindata) priv["instance"] = i - print str(self) + " notifier list before " + str(self.notifier) self.notifier.append(i) - print str(self) + " notifier list now " + str(self.notifier) - if self.notifierAdded: + try: self.notifierAdded(i) + except: + pass + + # GUIdelete must delete *all* references to the current component! + def GUIdelete(self, priv): + g = priv["instance"] + self.notifier.remove(g) + self.GUIdeleteInstance(g) + del priv["instance"] + + def GUIdeleteInstance(self, priv): + pass class VariableText: """VariableText can be used for components which have a variable text, based on any widget with setText call""" @@ -53,7 +124,6 @@ class VariableText: def setText(self, text): if self.message != text: self.message = text - print self.notifier for x in self.notifier: x.setText(self.message) @@ -85,6 +155,10 @@ class Clock(HTMLComponent, GUIComponent, VariableText): VariableText.__init__(self) GUIComponent.__init__(self) self.doClock() + + self.clockTimer = eTimer() + self.clockTimer.timeout.get().append(self.doClock) + self.clockTimer.start(1000) # "funktionalitaet" def doClock(self): @@ -100,17 +174,17 @@ class Clock(HTMLComponent, GUIComponent, VariableText): return self.getText() class Button(HTMLComponent, GUIComponent, VariableText): - onClick = {} - def __init__(self, text=""): GUIComponent.__init__(self) VariableText.__init__(self) self.setText(text) + self.onClick = [ ] - def click(self): + def push(self): for x in self.onClick: x() - + return 0 + # html: def produceHTML(self): return "\n" @@ -118,8 +192,11 @@ class Button(HTMLComponent, GUIComponent, VariableText): # GUI: def GUIcreateInstance(self, priv, parent, skindata): g = eButton(parent) -# g.clicked = [ self.click ] + g.selected.get().append(self.push) return g + + def GUIdeleteInstance(self, g): + g.selected.get().remove(self.push) class Header(HTMLComponent, GUIComponent, VariableText): @@ -147,3 +224,22 @@ class VolumeBar(HTMLComponent, GUIComponent, VariableValue): g.setRange(0, 100) return g +class MenuList(HTMLComponent, GUIComponent): + def __init__(self): + GUIComponent.__init__(self) + + def getCurrent(self): + return self.l.getCurrentSelection() + + def GUIcreateInstance(self, priv, parent, skindata): + g = eListbox(parent) + # BIG BIG HACK. :( we have to ensure that the eListboxPythonStringContent doesn't get destroyed. + # we really have to take a look at the GC stuff + self.l = eListboxPythonStringContent() + self.l.setList(["Test Object 1", "Item #2", "Item #3", "nun kommt eine Zahl:", 15, "Bla fasel", "lulabla"]) + g.setContent(self.l) + return g + + def GUIdeleteInstance(self, g): + g.setContent(None) + del self.l