#1518 CoffeeScript: don't use C states

Bug
closed-fixed
3
2013-10-15
2013-08-26
No

CoffeeScript lexer uses SCE_C_* styles but SCE_COFFEESCRIPT_* exist (and aren't all the same). Attached patch fixes usage of them so it only uses CoffeeScript specific ones.

This has also been highlighted in [#1420]

Also, CoffeeScript styles STRINGRAW, TRIPLEVERBATIM and HASHQUOTEDSTRING aren't used. Moreover HASHQUOTEDSTRING and COMMENTBLOCK both have value 22, which looks like a potential bug.

1 Attachments

Related

Bugs: #1420
Bugs: #1554

Discussion

  • Neil Hodgson

    Neil Hodgson - 2013-08-26
    • assigned_to: Neil Hodgson
     
  • Neil Hodgson

    Neil Hodgson - 2013-08-26

    This will be examined once 3.3.5 is released in about one week.

     
  • Neil Hodgson

    Neil Hodgson - 2013-09-02

    Committed as [c2ef64].

    Its unlikely SCE_COFFEESCRIPT_UUID is needed either as UUID was included for Microsoft COM and IDL.

     

    Related

    Commit: [c2ef64]

  • Neil Hodgson

    Neil Hodgson - 2013-09-02
    • status: open --> open-fixed
     
  • Neil Hodgson

    Neil Hodgson - 2013-10-15
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks