Skip to content

[BUG] Compilation fails on current master branch #173

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

Closed
userxfce opened this issue Dec 21, 2022 · 3 comments
Closed

[BUG] Compilation fails on current master branch #173

userxfce opened this issue Dec 21, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@userxfce
Copy link

command:

mkdir -p /temp/cppfront-git && cd /temp/cppfront-git && git clone https://github.com/hsutter/cppfront.git && cd cppfront/source && g++ cppfront.cpp -std=c++20 -o cppfront

results in:

Cloning into 'cppfront'...
cppfront.cpp:656:23: error: declaration of 'const cpp2::token* cpp2::cppfront::arg_info::token' changes meaning of 'token' [-fpermissive]
656 | token const* token = nullptr;
| ^~~~~
In file included from parse.h:21,
from sema.h:21,
from cppfront.cpp:18:
lex.h:191:7: note: 'token' declared here as 'class cpp2::token'
191 | class token
| ^~~~~

@userxfce userxfce added the bug Something isn't working label Dec 21, 2022
@filipsajdak
Copy link
Contributor

What compiler are you using?

@hsutter
Copy link
Owner

hsutter commented Dec 21, 2022

@userxfce Thanks! You must be compiling with gcc or Clang with -Wall -Wextra plus turning warnings into errors with -Werror, which should be fine but this week I added some code that triggers the above warning.

Fixed by the commit above, it now compiles cleanly again under -Wall -Wextra.

@userxfce
Copy link
Author

@filipsajdak fixed already, but for reference:

$ g++ --version
g++ (Ubuntu 11.3.0-1ubuntu1~22.04) 11.3.0
Copyright (C) 2021 Free Software Foundation, Inc.
This is free software; see the source for copying conditions. There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

@hsutter fix confirmed & thanks!

Azmah-Bad pushed a commit to Azmah-Bad/cppfront that referenced this issue Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants