Having second thoughts.  Where I suggested taking out this call

-      else if (isregexop (ch))
+      else if (ch == '*')    /* only wildcard currently implemented */
I should have just taken the extraneous '.' out of isregexop.  That would
make it slightly simpler to add the remaining shell wildcards ('?' and '[')
when desired.

'[' already works, if there's also a * to trigger expansion, with the small
bug that special chars within [] still get escaped or translated so the set
may include an unexpected '.' or '\'.

Other regex chars also work, which probably should be counted as a
bug.  I assume the eventual goal is to behave just like a common shell.


_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to