Skip to content
Snippets Groups Projects

Fixed the tokenisation of linemarkers

Merged Christian Rinderknecht requested to merge rinderknecht@linemarker_fix into dev
All threads resolved!

type:fixed

Checklist:

  • Changes follow the existing coding style (use dune @fmt to check).
  • Tests for the changes have been added (for bug fixes / feature).
  • Documentation has been updated.
  • Changelog description has been added (if appropriate).
  • Examples in changed behaviour have been added (for breaking change / feature).

Motivation and Context

Bug fix in the tokenisation (lexing) of linemarkers. Related Issue(s): #1479 (closed)

Description

In the library [LexerLib], the local variable [hash_state] in function [Core.scan_linemarker] was computed after calling [state#sync]. In fact, the order must be reversed.

Changelog description

Changelog details: Fixed the tokenisation of linemarkers

Edited by Christian Rinderknecht

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Melwyn Saldanha approved this merge request

    approved this merge request

  • mentioned in commit 6ce177ce

  • Laurent Canis added 2 commits

    added 2 commits

    • 6ce177ce - Merge branch 'rinderknecht@linemarker_fix' into 'dev'
    • ab04e1ad - [Bot] add changelog entry

    Compare with previous version

  • Melwyn Saldanha resolved all threads

    resolved all threads

  • Melwyn Saldanha started a merge train

    started a merge train

  • Melwyn Saldanha mentioned in commit 4b15cb1d

    mentioned in commit 4b15cb1d

  • Please register or sign in to reply
    Loading