Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLion support #92

Open
MikiGrit opened this issue Mar 12, 2015 · 5 comments
Open

CLion support #92

MikiGrit opened this issue Mar 12, 2015 · 5 comments

Comments

@MikiGrit
Copy link

Support for new C++ IDE by JetBrains (CLion) would be nice :). Because some highlights are unusual.

@jkaving
Copy link
Owner

jkaving commented Mar 12, 2015

Yes, that would be nice.
I don't use C++ at all so I doubt that I will add this support myself.
But pull requests are welcome.

I have added C and C++ as unsupported languages in README.md (fd64c84).

@AbigailBuccaneer
Copy link

In the latest CLion (RC, build 141.352.13) there are included color schemes called "Solarized Dark" and "Solarized Light", but they're completely wrong.

Given that they've already added them, it seems likely that this is now a problem for Jetbrains to deal with.

@jkaving
Copy link
Owner

jkaving commented Apr 16, 2015

@AbigailBuccaneer, thanks for noticing the bundled schemes in CLion.
I have reported an issue to JetBrains about this: https://youtrack.jetbrains.com/issue/CPP-3076

@AlexJF
Copy link

AlexJF commented May 30, 2015

Created 2 versions of the Dark Solarized theme for CLion:

https://gist.github.com/AlexJF/b4530f0df83a312e4bac

  • Java based: Tried to keep true to the Java version by jkaving (yellow keywords, orange numbers, green escapes).
  • Vim based: Keep true to the CPP vim version by Ethan Schoonover (green keywords, cyan numbers, red escapes).

Had a look at the contents but the C/C++ colors do not appear to be all prefixed by CPP (could only find CPP_KEYWORD) so it might conflict with existing entries if merged directly into the existing .ics in the repository. Most actually appear to be with an OC prefix but I guess this would clash with ObjectiveC?

@AbigailBuccaneer
Copy link

@AlexJF the parsed element types (if not the parsing itself) is mostly shared between ObjectiveC and C/C++ - if you have a peek at a C++ file in PsiViewer, you'll see that everything uses Language: ObjectiveC. This is because the C++ support is derived from AppCode's ObjectiveC support.

It's best to kill two birds with one stone and make a scheme for C/C++/ObjectiveC/ObjectiveC++.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants