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

TurtleApp accepts argc by reference #10

Merged
merged 1 commit into from
Aug 7, 2013

Conversation

muratsevim
Copy link
Contributor

Otherwise running turtlesim_node with a command line argument, such as
rosrun turtlesim turtlesim_node __name:=sim
produces a segmentation fault with the following crash report on OSX

Thread 0 Crashed:: Dispatch queue: com.apple.main-thread
0   QtCore                          0x000000010a2c1047 QString::fromLocal8Bit(char const*, int) + 25
1   QtCore                          0x000000010a358fc2 QCoreApplication::arguments() + 98
2   QtGui                           0x0000000109793a34 -[QCocoaApplicationDelegate application:openFiles:] + 222
3   com.apple.AppKit                0x00007fff89907f16 -[NSApplication _doOpenFile:ok:tryTemp:] + 274
4   com.apple.AppKit                0x00007fff896501ef -[NSApplication finishLaunching] + 1678
5   com.apple.AppKit                0x00007fff8964f860 -[NSApplication run] + 125
6   QtGui                           0x000000010979ae11 QEventDispatcherMac::processEvents(QFlags) + 461
7   QtCore                          0x000000010a354d89 QEventLoop::processEvents(QFlags) + 79
8   QtCore                          0x000000010a354ee7 QEventLoop::exec(QFlags) + 341
9   QtCore                          0x000000010a357a6b QCoreApplication::exec() + 199
10  turtlesim_node                  0x000000010963e238 TurtleApp::exec() + 104
11  turtlesim_node                  0x000000010963de82 main + 66
12  turtlesim_node                  0x000000010963de34 start + 52

See discussion at http://qt-project.org/forums/viewthread/18703/

@dirk-thomas
Copy link
Member

Great catch. Thanks for also including the pointer to the qt-project discussion.

dirk-thomas added a commit that referenced this pull request Aug 7, 2013
TurtleApp accepts argc by reference
@dirk-thomas dirk-thomas merged commit 9b4891f into ros:hydro-devel Aug 7, 2013
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

Successfully merging this pull request may close these issues.

2 participants