Skip to content

Commit 9ad7886

Browse files
committed
Update comments in Semgrex - not sure that single root matches is still a limitation
1 parent b77103c commit 9ad7886

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/edu/stanford/nlp/semgraph/semgrex/SemgrexPattern.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,12 +190,12 @@
190190
* the potential match will not be accepted.
191191
* <br>
192192
* This is only legal on relations with only one link between the two endpoints.
193-
* Other relations (such as grandparent) will throw an exception.
193+
* Other relations (such as grandparent) will throw a parse exception.
194194
*
195195
* <h3>TODO</h3>
196196
* At present a Semgrex pattern will match only once at a root node, even if there is more than one way of satisfying
197197
* it under the root node. Probably its semantics should be changed, or at least the option should be given, to return
198-
* all matches, as is the case for Tregex.
198+
* all matches, as is the case for Tregex. (Is this still true? It seems to match multiple times from root.)
199199
*
200200
* @author Chloe Kiddon
201201
*/

0 commit comments

Comments
 (0)