Sorry. Careless. Sorted.


On 28 February 2014 07:45, John Sichi <notifications@github.com> wrote:

> In jgrapht-core/src/main/java/org/jgrapht/alg/NaiveLcaFinder.java:
>
> > +     * 0 length from the nodes we are considering as LCA to their respective children which we wish to find the LCA for.
> > +     */
> > +    public NaiveLcaFinder(Graph<V, E> graph) {
> > +	this.graph = graph;
> > +    }
> > +
> > +    /**
> > +     * Return the LCA of a and b
> > +     *
> > +     * @param a
> > +     *            the first element to find LCA for
> > +     * @param b
> > +     *            the other element to find the LCA for
> > +     * @return the LCA of a and b, or null if there is no LCA.
> > +     */
> > +    public V findLca(V a, V b) {
>
> Good, but the findLcas method should return Set, not V, right?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/jgrapht/jgrapht/pull/73/files#r10156804>
> .
>