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

cpp17 cleanup grammar #1228

Merged
merged 25 commits into from
Sep 24, 2017
Merged

cpp17 cleanup grammar #1228

merged 25 commits into from
Sep 24, 2017

Conversation

guwirth
Copy link
Collaborator

@guwirth guwirth commented Sep 8, 2017

  • expr-or-braced-init-list
  • simple-type-specifier
  • balanced-token
  • attribute ...
  • nodeclspec-function-declaration
  • postfix-expression
  • using-declaration
  • enum-head-name
  • defining-type-id, defining-type-specifier, defining-type-specifier-seq
  • trailing-return-type
  • remove trailingTypeSpecifierSeq
  • decl-specifier
  • tryBlock
  • functionTryBlock
  • literals
  • comment unqualifiedId
  • comment lambdaCapture
  • statement
  • condition
  • aliasDeclaration
  • comment typeSpecifierSeq, definingTypeSpecifierSeq
  • comment attributeUsingPrefix
  • attributeList

mark remaining differences in CxxGrammarImpl.java


This change is Reviewable

@guwirth guwirth added this to the 0.9.8 milestone Sep 8, 2017
@guwirth guwirth self-assigned this Sep 8, 2017
@guwirth
Copy link
Collaborator Author

guwirth commented Sep 24, 2017

@ALL still not all issues are solved. Remaining parts are marked with todo in grammar.
Please give us feedback to the changes.

@guwirth guwirth merged commit f7de90b into SonarOpenCommunity:master Sep 24, 2017
@guwirth guwirth deleted the cpp17/clean-up-grammar branch October 3, 2017 13:52
@guwirth guwirth restored the cpp17/clean-up-grammar branch October 3, 2017 13:52
@guwirth guwirth mentioned this pull request Oct 29, 2017
@guwirth guwirth deleted the cpp17/clean-up-grammar branch January 21, 2018 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant