node incorrect in definition of ALP

[pfps]

The comment in the definition below is not correct, as ALP can be started with any RDF term.

Definition: Function ALP

Let ppe be an algebraic property path expression.
Let reachableTerms(x:term, ppe) be the set of RDF terms
reached by repeated matches of ppe,
when starting at RDF term x.

ALP(x:term, ppe) =
Let V = empty set
ALP(x:term, ppe, V)
return is V

V is the set of nodes visited

ALP(x:term, ppe, V:set of RDF terms) =
if ( x in V ) return
add x to V
X = reachableTerms(x, ppe)
For n:term in X
ALP(n, ppe, V)
End

[hartig]

Well. I agree that the comment is incorrect with respect to the first of the two functions here, i.e., for ALP(x:term, ppe), but not so for the second function, ALP(x:term, ppe, V).

Having said that, I think that this comment doesn't add anything and, thus, I am fine with simply removing it. @pfps would that address your issue!

[pfps]

Yes

[afs]

The comment in the definition below is not correct, as ALP can be started with any RDF term.

V is the set of nodes visited - it starts as empty and grows. It is the result.
In the first function it is initialized; in the second function, it is accumulating.

[pfps]

The comment in the definition below is not correct, as ALP can be started with any RDF term.

V is the set of nodes visited - it starts as empty and grows. It is the result. In the first function it is initialized; in the second function, it is accumulating.

But the first element of V can be a term, not just a node.