Mailinglist Archive: opensuse-bugs (13459 mails)
| < Previous | Next > |
[Bug 335582] New: Please, remove constructor from "Storage" module
- From: bugzilla_noreply@xxxxxxxxxx
- Date: Mon, 22 Oct 2007 03:13:32 -0600 (MDT)
- Message-id: <bug-335582-21960@xxxxxxxxxxxxxxxxxxxxxxxxx/>
https://bugzilla.novell.com/show_bug.cgi?id=335582
Summary: Please, remove constructor from "Storage" module
Product: openSUSE 11.0
Version: unspecified
Platform: Other
OS/Version: Other
Status: NEW
Severity: Enhancement
Priority: P5 - None
Component: YaST2
AssignedTo: fehr@xxxxxxxxxx
ReportedBy: locilka@xxxxxxxxxx
QAContact: jsrain@xxxxxxxxxx
CC: aschnell@xxxxxxxxxx, visnov@xxxxxxxxxx
Found By: ---
We've been discussing the Storage on our workshop. The statement was that
importing Storage doesn't call any constructors. However the truth is, that if
a module contains a function with the same name (as module), it is called on
import.
Storage has a Storage::Storage()
See:
--- storagetest.ycp ---
{
import "Storage";
}
--- storagetest.ycp ---
time yast2 storagetest.ycp
real 0m4.483s
user 0m1.564s
sys 0m0.276s
just to compare:
--- fileutilstest.ycp ---
{
import "FileUtils";
}
--- fileutilstest.ycp ---
time yast2 fileutilstest.ycp
real 0m0.238s
user 0m0.160s
sys 0m0.080s
Anyway, maybe it's rather worth a feature request in FATE.
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
Summary: Please, remove constructor from "Storage" module
Product: openSUSE 11.0
Version: unspecified
Platform: Other
OS/Version: Other
Status: NEW
Severity: Enhancement
Priority: P5 - None
Component: YaST2
AssignedTo: fehr@xxxxxxxxxx
ReportedBy: locilka@xxxxxxxxxx
QAContact: jsrain@xxxxxxxxxx
CC: aschnell@xxxxxxxxxx, visnov@xxxxxxxxxx
Found By: ---
We've been discussing the Storage on our workshop. The statement was that
importing Storage doesn't call any constructors. However the truth is, that if
a module contains a function with the same name (as module), it is called on
import.
Storage has a Storage::Storage()
See:
--- storagetest.ycp ---
{
import "Storage";
}
--- storagetest.ycp ---
time yast2 storagetest.ycp
real 0m4.483s
user 0m1.564s
sys 0m0.276s
just to compare:
--- fileutilstest.ycp ---
{
import "FileUtils";
}
--- fileutilstest.ycp ---
time yast2 fileutilstest.ycp
real 0m0.238s
user 0m0.160s
sys 0m0.080s
Anyway, maybe it's rather worth a feature request in FATE.
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
| < Previous | Next > |