Fix highlight on unnamed class, struct and enum #22

rhysd · 2014-05-16T05:31:07Z

C++ では，特定のクラスから継承して匿名クラスを定義することができます．また，enum に underlying type を指定することができます．
これらの構文がラベルの構文に似ており，シンタックスハイライトはラベル部分のみのマッチで行われてしまうので誤ってラベルとしてハイライトされてしまっていました．

struct base{};

int main()
{
    //
    // Below 'enum', 'struct' and 'class' are highlighted as label
    //

    // Specify int as enum's underlying
    enum : int {} aaa;

    // unnamed derived struct
    struct : base {} bbb;

    // unnamed derived class
    class : base {} ccc;

    return 0;
}

解決策として，ラベルが struct, class, enum でないかをチェックするようにしました．

Problem: In C++, unnamed derived class and struct can be defined using ':'. enum also can be defined with specific underlying type using ':'. They are highlighted as labels wrongly. ``` struct base{}; int main() { // // Below 'enum', 'struct' and 'class' are highlighted as label // // Specify enum's underlying type enum : int {} aaa; // unnamed derived struct struct : base {} bbb; // unnamed derived class class : base {} ccc; return 0; } ``` Solution: Check that labels don't match to them.

osyo-manga · 2014-05-16T09:16:33Z

++

mattn · 2014-05-16T09:29:25Z

LGTM

Fix highlight on unnamed class, struct and enum

mattn · 2014-05-16T09:30:16Z

Thank you

rhysd · 2014-05-16T17:08:42Z

🐶 👍

mattn · 2014-05-18T13:14:06Z

先日パッチを送ったばかりで、たて続けに送る姿勢を見せるとBramに嫌がられそうなのでしばらく時間を下さい。
すみません。

rhysd · 2014-06-04T08:34:23Z

すみません，コメント気づいていませんでした．
パッチを送るタイミングについては @mattn さんにお任せします…！

mattn · 2014-06-04T08:38:32Z

🐕 👍

mattn added a commit that referenced this pull request May 16, 2014

Merge pull request #22 from rhysd/fix/unamed-class-and-enum

ed3fc2c

Fix highlight on unnamed class, struct and enum

mattn merged commit ed3fc2c into vim-jp:master May 16, 2014

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix highlight on unnamed class, struct and enum #22

Fix highlight on unnamed class, struct and enum #22

Fix highlight on unnamed class, struct and enum #22

Fix highlight on unnamed class, struct and enum #22

Conversation