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

[v2] Sometimes the mouse movements stop to be reported. #8

Closed
matlo opened this issue Nov 27, 2013 · 7 comments
Closed

[v2] Sometimes the mouse movements stop to be reported. #8

matlo opened this issue Nov 27, 2013 · 7 comments
Labels

Comments

@matlo
Copy link
Owner

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on May 07, 2010 17:52:09

Sometimes the mouse movements stop to be reported.

Mouse buttons work.

As a work around, press escape, wait half a second, press up and enter.

That will restart the emuclient.

Original issue: http://code.google.com/p/diyps3controller/issues/detail?id=6

@matlo
Copy link
Owner Author

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on May 07, 2010 08:52:25

Summary: [v2] Sometimes the mouse movements stop to be reported.

@matlo
Copy link
Owner Author

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on May 07, 2010 10:58:41

This is due to an array overrun in the stack.

The event array was only sized for 8 elements, and it may happen that there are more
than 8 elements returned by SDL_PeepEvents.

Second argument for SDL_PeepEvents was wrong: it was sizeof(events), it has to be
sizeof(events)/sizeof(events[0]).

@matlo
Copy link
Owner Author

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on May 07, 2010 11:04:00

Event buffer will be increased so that it can store 32 events (to avoid any lag issue).

@matlo
Copy link
Owner Author

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on May 07, 2010 11:06:51

Labels: -Priority-High Priority-Critical

@matlo
Copy link
Owner Author

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on May 07, 2010 11:07:34

Status: Fixed

@matlo
Copy link
Owner Author

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on May 08, 2010 05:39:17

Issue 4 has been merged into this issue.

@matlo
Copy link
Owner Author

@matlo matlo commented Nov 27, 2013

From mathieu....@gmail.com on December 19, 2011 09:14:55

Status: Verified

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

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.