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

regression: ‘QMutex’ does not name a type #62

Closed
nickodell opened this issue Nov 9, 2017 · 2 comments
Closed

regression: ‘QMutex’ does not name a type #62

nickodell opened this issue Nov 9, 2017 · 2 comments

Comments

@nickodell
Copy link
Contributor

Revision 5f2aadf does not compile for me. It fails with this error:

[ 74%] Building CXX object gui/CMakeFiles/Studio.dir/Studio_automoc.cpp.o
In file included from /home/nick/ao/gui/src/interpreter.cpp:21:0:
/home/nick/ao/gui/include/gui/interpreter.hpp:90:5: error: ‘QMutex’ does not name a type
     QMutex mutex;
     ^~~~~~
In file included from /home/nick/ao/gui/src/window.cpp:30:0:
/home/nick/ao/gui/include/gui/interpreter.hpp:90:5: error: ‘QMutex’ does not name a type
     QMutex mutex;
     ^~~~~~
In file included from /home/nick/ao/build/gui/Studio_automoc.dir/moc_interpreter_JL3ZW3PCB4US6T.cpp:9:0,
                 from /home/nick/ao/build/gui/Studio_automoc.cpp:7:
/home/nick/ao/build/gui/Studio_automoc.dir/../../../gui/include/gui/interpreter.hpp:90:5: error: ‘QMutex’ does not name a type
     QMutex mutex;
     ^~~~~~
/home/nick/ao/gui/src/interpreter.cpp: In member function ‘void _Interpreter::eval()’:
/home/nick/ao/gui/src/interpreter.cpp:107:28: error: ‘mutex’ was not declared in this scope
         QMutexLocker lock(&mutex);
                            ^~~~~
/home/nick/ao/gui/src/interpreter.cpp: In member function ‘void Interpreter::onScriptChanged(QString)’:
/home/nick/ao/gui/src/interpreter.cpp:270:40: error: ‘class _Interpreter’ has no member named ‘mutex’
         QMutexLocker lock(&interpreter.mutex);
                                        ^~~~~
gui/CMakeFiles/Studio.dir/build.make:542: recipe for target 'gui/CMakeFiles/Studio.dir/src/window.cpp.o' failed

However, 7dac3c7 compiles and runs successfully.

@mkeeter
Copy link
Member

mkeeter commented Nov 10, 2017

Probably just a missing #include <QMutex>; see if 7d6ba9a fixes this issue.

@nickodell
Copy link
Contributor Author

Fix confirmed.

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

No branches or pull requests

2 participants