[jbosstools-issues] [JBoss JIRA] (JBIDE-3355) org.jboss.tools.common.text.ext.hyperlink.AbstractHyperlink review

[Denis Golovin (JIRA)]

     [ https://issues.jboss.org/browse/JBIDE-3355?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Denis Golovin updated JBIDE-3355:
---------------------------------

    Fix Version/s: 4.0.0.Alpha2
                       (was: 4.0.0.Alpha1)


moved to 4.0.0.Alpha2
                
> org.jboss.tools.common.text.ext.hyperlink.AbstractHyperlink review
> ------------------------------------------------------------------
>
>                 Key: JBIDE-3355
>                 URL: https://issues.jboss.org/browse/JBIDE-3355
>             Project: Tools (JBoss Tools)
>          Issue Type: Sub-task
>          Components: Cleanup
>    Affects Versions: 3.0.0.cr1
>            Reporter: Denis Golovin
>            Assignee: Denis Golovin
>             Fix For: 4.0.0.Alpha2
>
>
> 1. Consider using another approach instead of using snippet below in all methods:
> 	protected IFile getFile() {
> 		StructuredModelWrapper smw = new StructuredModelWrapper();
> 		try {
> 			smw.init(getDocument());
> 			return smw.getFile();
> 		} finally {
> 			smw.dispose();
> 		}
> 	}
> 2. Review static methods, as example - public static IFile getFile(IStructuredModel structuredModel) looks to compex

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira