Skip to content

Parsing literal strings is deficient. #17

Description

@WardCunningham

The current grammar requires literal strings (non-terminal strn) to be quoted with double-quotes. The current implementation has several problems.

  • The ch('"') terminal adsorbs trailing whitespace which should not be ignored in a literal string.
  • Embedded double-quotes are not correctly recognized even when properly escaped.
  • Single-quotes can enclose literal strings too.
  • Places an unreasonable cap on the size of literal strings and fails when exceeded.

https://s3.amazonaws.com/artifacts.opencypher.org/M18/railroad/StringLiteral.html

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions