Fixed the tokenisation of linemarkers
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
#1479 (closed)
Bug fix in the tokenisation (lexing) of linemarkers. Related Issue(s):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
Merge request reports
Activity
assigned to @rinderkn
mentioned in merge request serokell/ligo/ligo!447 (merged)
added 1 commit
- a7e520a7 - [JsLIGO] Verbatim strings are now only delimited by {| and |}.
A new deploy preview is available on Netlify at https://2092-2efc5d0f--ligolang.netlify.app, you can view the logs at https://app.netlify.com/sites/ligolang/deploys/6356d8fb53f0c607fe522349
added 1 commit
- e449f065 - Reverted last changes on verbatim string in JSLIGO.
A new deploy preview is available on Netlify at https://2092-6ce177ce--ligolang.netlify.app, you can view the logs at https://app.netlify.com/sites/ligolang/deploys/6356f75b4fa5e127d4196712
- Resolved by Melwyn Saldanha
mentioned in commit 6ce177ce
A new deploy preview is available on Netlify at https://2092-e139b48a--ligolang.netlify.app, you can view the logs at https://app.netlify.com/sites/ligolang/deploys/635776a07886da09d38f6abb
A new deploy preview is available on Netlify at https://2092-6a86e699--ligolang.netlify.app, you can view the logs at https://app.netlify.com/sites/ligolang/deploys/635779ffb003ce0b7a12b630
mentioned in commit 4b15cb1d