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

segfault if newFigure is missing #60

Open
gchabert opened this issue Jul 13, 2014 · 3 comments
Open

segfault if newFigure is missing #60

gchabert opened this issue Jul 13, 2014 · 3 comments
Labels

Comments

@gchabert
Copy link

Hello surfers of the vibes,

When I do this:
vibes::beginDrawing ();
vibes::newFigure("bla");
It works fine.
But if I try to do this instead:
vibes::beginDrawing ("bla");
I have a seg fault.
I guess the second variant is not correct but...

Gilles

@nicolaje
Copy link
Member

Gilles,

I can't reproduce your issue on windows, linux or mac.

Both following example work fine for me:

#include "vibes.h"

int main()
{
    vibes::beginDrawing("bla");
    vibes::drawBox(-1,1,-1,1);
    vibes::endDrawing();
    return 1;
}
int main()
{
    vibes::beginDrawing();
    vibes::newFigure("bla");
    vibes::drawBox(-1,1,-1,1);
    vibes::endDrawing();
    return 1;
}

I am using either the vibes.h & vibes.cpp from the released archive:
https://github.com/ENSTABretagneRobotics/VIBES/releases

or the ones on the repository:
https://github.com/ENSTABretagneRobotics/VIBES/tree/master/client-api/C%2B%2B/src

Do you still encounter this problem?

@nicolaje nicolaje added the bug label Dec 13, 2014
@dvinc
Copy link
Contributor

dvinc commented Feb 12, 2015

vibes::beginDrawing("bla");

This command starts the VIBes client in file output mode, telling it to write all its commands to a file named "bla" (instead of communicating with the viewer). If you don't have write permission to the working directory, it may crash.

I keep it as a bug, since in this case the client should not crash, but gracefully discard all vibes commands.

@gchabert
Copy link
Author

I can install the last release and try again, if you want... but remind me how to compile VIBES from the sources (I'm still desperatly looking for the doc !)
Gilles

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

No branches or pull requests

3 participants