[Valyrian-Code]

When v.surf.rst cannot fill some holes, r.fillnulls warns the user and is meant to list the unfilled holes so they can be examined. The list was built with a loop that never used the loop variable, so it only printed the first hole name with one leading comma per remaining hole.

For three unfilled holes the message printed , , hole_3 instead of hole_3, hole_7, hole_12.

This replaces the loop with ", ".join(failed_list) so every unfilled hole is reported, and fixes two typos in the adjacent warning (where to were, and a stray with).

-        outlist = failed_list[0]
-        for hole in failed_list[1:]:
-            outlist = ", " + outlist
-        gs.message(outlist)
+        gs.message(", ".join(failed_list))

The failure branch is only reached when v.surf.rst fails on specific holes, which is not deterministic to trigger, so this keeps the existing behavior otherwise and just fixes the reporting.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.