| Summary: | [RFE] Eliminate unnecessary typecasts when using IEventListener interface | ||
|---|---|---|---|
| Product: | [Retired] oVirt | Reporter: | Vojtech Szocs <vszocs> |
| Component: | ovirt-engine-webadmin | Assignee: | Nobody's working on this, feel free to take it <nobody> |
| Status: | CLOSED WONTFIX | QA Contact: | bugs <bugs> |
| Severity: | unspecified | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | unspecified | CC: | abonas, acathrow, ecohen, iheim, mgoldboi, yeylon |
| Target Milestone: | --- | Keywords: | FutureFeature |
| Target Release: | --- | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | ux | ||
| Fixed In Version: | Doc Type: | Enhancement | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2014-06-16 15:27:26 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
in the meantime, patches to supress findbug warnings are submitted. When this RFE is resolved, the patches can be reverted. http://gerrit.ovirt.org/#/c/21797/ Closing old bugs. If this issue is still relevant/important in current version, please re-open the bug. |
The IEventListener interface is declared as following: public interface IEventListener { void eventRaised(Event ev, Object sender, EventArgs args); } For specific events, such as PropertyChangedEvent triggered by following code within a model class: getPropertyChangedEvent().raise(this, e); where "e" is PropertyChangedEventArgs, client code will have to do unnecessary typecast like this: ... public void eventRaised(Event ev, Object sender, EventArgs args) { PropertyChangedEventArgs actualArgs = (PropertyChangedEventArgs) args; } ... To eliminate unnecessary typecasts as shown above, the IEventListener interface should be improved (i.e. parametrized).