-
Notifications
You must be signed in to change notification settings - Fork 10
Conversation
Builds probably failing due to missing symbols (relies on changes in esy-sdl2#6) |
src/sdl2_wrapper.cpp
Outdated
@@ -401,6 +401,8 @@ CAMLprim value Val_SDL_Event(SDL_Event *event) { | |||
|
|||
int tag, mouseButton; | |||
|
|||
printf("event type is %d while pan event type is %d and mousewheel is %d\n", event->type, SDL_PANEVENT, SDL_MOUSEWHEEL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should remove the printf
s for the final version
Change to pushing errors to stderr as suggested by @Et7f3 Co-Authored-By: Et7f3 <cadeaudeelie@gmail.com>
…o add_pan_events
I think if you bring in the latest master, this should be build now @szbergeron 👍 |
src/sdl2_wrapper.cpp
Outdated
@@ -551,6 +567,7 @@ CAMLprim value Val_SDL_Event(SDL_Event *event) { | |||
v = Val_SDL_WindowEvent(23, event->window.windowID); | |||
break; | |||
default: | |||
fprintf(stderr, "Unknown event in sdl2_wrapper, event enum code was %d\n", event->type); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should remove this as well - might cause issues in the case where we run on Windows w/o a console allocated (we could hook into the SDL_Log
mechanism though instead)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor change - we should remove the fprintf
. Otherwise, everything else looks good - it should build now w/ master
merged
Looks great, @szbergeron ! Thank you! |
WIP: plumbs support for smooth scrolling
Adds a new "MousePan" event type that matches up with revery-ui/esy-sdl2#6
Needs to be integrated with Revery proper before it will work