[jira] [Closed] (FELIX-5251) Resolver impl is in need of extensive cleanup

["Thomas Watson (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/FELIX-5251?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Thomas Watson closed FELIX-5251.
--------------------------------
    Resolution: Fixed

Closing issue.

> Resolver impl is in need of extensive cleanup
> ---------------------------------------------
>
>                 Key: FELIX-5251
>                 URL: https://issues.apache.org/jira/browse/FELIX-5251
>             Project: Felix
>          Issue Type: Bug
>          Components: Resolver
>    Affects Versions: resolver-1.8.0
>         Environment: All
>            Reporter: Thomas Watson
>            Assignee: Thomas Watson
>
> The Resolver implementation has undergone a lot of changes since it was first contributed.  Over the coarse of development the code has deteriorated a bit.  This is not to blame the contributors to the codebase.  The codebase and overall problem is complicated and already hard enough to understand.
> I have been looking to implement some upcoming changes that are being proposed for the OSGi R7 specification.  Before I start that work I have been spending significant time scrubbing the resolver code and trying to make it a little more understandable.  Here are the main things I have done:
> 1) Create a common code path for dynamic resolution and regular resolution.  
> 2) Extract common code and greatly reduce the number of lines contained in complicated do/while loops.
> 3) Make proper use of the existing ResolveSession class to keep track of state during the resolution process.   This keeps the maintenance of state in one place.  The code had evolved into many separate options being passed around to methods, each maintaining some kind of state.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)