Skip to content

Parser swallows leading whitespace after indented placeable #286

Open

Description

@RumovZ

In this fixture, the leading whitespace in front of "four" is preserved as expected:

key12 =
{ "." }
    four

However, with normalized indentation, it is not:

key12 =
    { "." }
        four

This is wrongly parsed to the same AST as this:

key12 =
    { "." }
    four

Activity

  1. added
    help wantedWe need help making decisions or writing PRs for this.
    good first issueWant to help? Those are great bugs to start with!
    on Nov 10, 2022
  2. gregtatum commented on Nov 10, 2022

    @gregtatum
    Member

    I'm tentatively marking this as a good first issue since it will probably be a relatively small patch, but you should probably like parsers if you want to take this one.

  3. RumovZ commented on Nov 10, 2022

    @RumovZ
    ContributorAuthor

    When this is fixed, crlf.ftl may be removed from the IGNORE_LIST in fluent-syntax/tests/serializer_fixtures.rs.

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

    buggood first issueWant to help? Those are great bugs to start with!help wantedWe need help making decisions or writing PRs for this.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions