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

c-c++ layer requires semantic layer #1634

Closed
herbertjones opened this issue May 18, 2015 · 4 comments
Closed

c-c++ layer requires semantic layer #1634

herbertjones opened this issue May 18, 2015 · 4 comments

Comments

@herbertjones
Copy link
Contributor

Unless semantic is enabled in dotspacemacs-configuration-layers, extra c-c++ packages do not appear to be added nor are they initialized correctly when other layers have enabled the packages. There is no error or warning accompanied by this behavior. Unless the semantic layer is enabled, c++ files will have missing functionality such as:

  • yasnippet minor mode is not initialized
  • srefactor keybindings are missing and the package is not installed

ycmd, however, still works if that layer is enabled.

Observed in develop 5b72221.

@tuhdo
Copy link
Contributor

tuhdo commented May 18, 2015

About semantic, it's intended since some people like it, some don't. Also, it's not specific to C++ so we separate it in its own layer.

@syl20bnr
Copy link
Owner

@herbertjones If I understand well, only yasnippet has a wrong behavior right ?

@herbertjones
Copy link
Contributor Author

Sounds like it. Perhaps it would be nicer for some kind of warning for users updating so that they know that they only need to add the semantic layer. Anyone finding the problem documented here in this github issue might be enough if the documentation for the layer is updated.

@StreakyCobra
Copy link
Contributor

The c-c++ layer now contains notes about the semantic layer, and also about srefactor only available on emacs >= 24.4. yasnippet is working fine here even without the semantic layer.

All points being solved, I close this issue. Feel free to open a new issue in case of trouble ☺️

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