Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parsing switch in braced parameter #44

Merged
merged 6 commits into from
May 30, 2021
Merged

Parsing switch in braced parameter #44

merged 6 commits into from
May 30, 2021

Conversation

magicant
Copy link
Owner

@magicant magicant commented May 25, 2021

  • Define syntax
  • Define errors that may happen while parsing
  • Implement parser
  • Use parser
  • Ensure corner cases are tested (e.g. ${#-?}, ${#:-})
  • Support context-dependent results (text v. word)
    • Tilde expansion
    • Single- and double-quotes
    • Allow escaping }
  • Implement FromStr
  • Refactor

unit word ${x-word} "text" "${x-text}"
Tilde Special Special Literal Literal
Single quotes Special Special Literal Literal
Double quotes Special Special Special Special
\\ Special Special Special Special
\" Special Special Special Special
\} Special Special Literal Special
\{ Special Special Literal Literal

@magicant magicant self-assigned this May 25, 2021
@magicant
Copy link
Owner Author

Support context-dependent results (text v. word)

This part seems going to be rather complicated, so I work on it in a separate PR.

@magicant magicant marked this pull request as ready for review May 30, 2021 02:18
@magicant magicant merged commit e9d4155 into master May 30, 2021
@magicant magicant deleted the param_parser branch May 30, 2021 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant