refactor(lookahead-iterator)!: implement Iterator

Refs: #21

Author: ObserverOfTime

src/main/java/io/github/treesitter/jtreesitter/LookaheadIterator.java

@@ -19,7 +19,7 @@
  * For {@index MISSING} nodes, a lookahead iterator created on the previous non-extra leaf node may be appropriate.
  */
 @NullMarked
-public final class LookaheadIterator implements AutoCloseable, Enumeration<LookaheadIterator.Symbol> {
+public final class LookaheadIterator implements AutoCloseable, Iterator<LookaheadIterator.Symbol> {
     private final Arena arena;
     private final MemorySegment self;
     private final short state;
@@ -80,8 +80,9 @@ public boolean reset(@Unsigned short state, Language language) {
         return ts_lookahead_iterator_reset(self, language.segment(), state);
     }
 
+    /** Check if the lookahead iterator has more symbols. */
     @Override
-    public boolean hasMoreElements() {
+    public boolean hasNext() {
         if (iterFirst) {
             iterFirst = false;
             hasNext = ts_lookahead_iterator_next(self);
@@ -96,8 +97,8 @@ public boolean hasMoreElements() {
      * @throws NoSuchElementException If there are no more symbols.
      */
     @Override
-    public Symbol nextElement() throws NoSuchElementException {
-        if (!hasMoreElements()) throw new NoSuchElementException();
+    public Symbol next() throws NoSuchElementException {
+        if (!hasNext()) throw new NoSuchElementException();
         hasNext = ts_lookahead_iterator_next(self);
         return new Symbol(getCurrentSymbol(), getCurrentSymbolName());
     }
@@ -127,6 +128,12 @@ public void close() throws RuntimeException {
         arena.close();
     }
 
+    /** @hidden */
+    @Override
+    public void remove() {
+        Iterator.super.remove();
+    }
+
     /** A class that pairs a symbol ID with its name. */
     public record Symbol(@Unsigned short id, String name) {}
 

src/test/java/io/github/treesitter/jtreesitter/LookaheadIteratorTest.java

@@ -45,28 +45,28 @@ void getCurrentSymbolName() {
     @Test
     @DisplayName("reset(state)")
     void resetState() {
-        assertDoesNotThrow(() -> lookahead.nextElement());
+        assertDoesNotThrow(() -> lookahead.next());
         assertTrue(lookahead.reset(state));
         assertEquals("ERROR", lookahead.getCurrentSymbolName());
     }
 
     @Test
     @DisplayName("reset(language)")
     void resetLanguage() {
-        assertDoesNotThrow(() -> lookahead.nextElement());
+        assertDoesNotThrow(() -> lookahead.next());
         assertTrue(lookahead.reset(state, language));
         assertEquals("ERROR", lookahead.getCurrentSymbolName());
     }
 
     @Test
-    void hasMoreElements() {
-        assertTrue(lookahead.hasMoreElements());
+    void hasNext() {
+        assertTrue(lookahead.hasNext());
         assertEquals("ERROR", lookahead.getCurrentSymbolName());
     }
 
     @Test
-    void nextElement() {
-        assertEquals("end", lookahead.nextElement().name());
+    void next() {
+        assertEquals("end", lookahead.next().name());
     }
 
     @Test