Ticket UUID: | 6cd041bcc1abb879a744a19e4503149c4bdf36c3 | |||
Title: | Probably useless temporary widget creation | |||
Type: | Bug | Version: | 1.9.7 | |
Submitter: | oehhar | Created on: | 2013-10-17 06:44:29 | |
Subsystem: | bwidget 1.x | Assigned To: | oehhar | |
Priority: | 5 Medium | Severity: | Minor | |
Status: | Closed | Last Modified: | 2013-10-17 12:57:54 | |
Resolution: | Accepted | Closed By: | oehhar | |
Closed on: | 2013-10-17 12:57:54 | |||
Description: |
(text/x-fossil-wiki)
<b>Widget::tkinclude </b> temporary creates an instance of the target widget class. Wolfgang S. Kechel proposed to remove this by the attached patch. | |||
User Comments: |
oehhar added on 2013-10-17 12:57:54:
(text/x-fossil-wiki)
Accepted after sure-Email, commit [2ff16995c7] oehhar added on 2013-10-17 06:48:44: (text/x-fossil-wiki) I am sorry, I have a bad feeling removing those commands. I often got into the trap that the original BWidget code by Eric is well thought and apparently useless things have a reason. A widget creation causes many other actions in the background which are not clear to me in total. Sorry for this defensive response. Maybee someone may convince me to include the patch. Sorry, Harald |
Attachments:
- widget.patch [download] added by oehhar on 2013-10-17 06:45:19. [details]