[jira] [Resolved] (SANDBOX-344) No WeightedGraph in current method signatures

["Simone Tripodi (Resolved) (JIRA)" <ji...@apache.org>]

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

Simone Tripodi resolved SANDBOX-344.
------------------------------------

    Resolution: Fixed
      Assignee: Simone Tripodi

GREAT! Thanks a lot for contributing Claudio, you patch has been committed, see [r1212887|https://svn.apache.org/viewvc?view=revision&revision=1212887] :)
                
> No WeightedGraph in current method signatures
> ---------------------------------------------
>
>                 Key: SANDBOX-344
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-344
>             Project: Commons Sandbox
>          Issue Type: Improvement
>          Components: Graph
>            Reporter: Claudio Squarcella
>            Assignee: Simone Tripodi
>            Priority: Minor
>              Labels: graph, patch, weightedgraph
>         Attachments: NoWeightedGraphInMethodSignatures.patch
>
>
> Hello,
> following a discussion on the mailing list (mainly with Simone) I am providing a small patch to "get rid" of the interface {{WeightedGraph}} in all the method signatures for the algorithms currently implemented, basically replacing it with the equally expressive {{Graph<V extends Vertex, We extends WeightedEdge>}}.
> The main reason: keep it simple. So far none of the algorithms really needs the {{WeightedGraph}} interface (e.g. no need for additional methods that cannot be found in {{Graph}}). With that said I did NOT remove the {{WeightedGraph}} interface.
> Hoping to meet your agreement and waiting for comments,
> Ciao
> Claudio

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira