No problem. I'll update the patch and add some no-op functions instead.
Thanks for the review :)
Chris
On 07/07/2015 01:18 PM, Jasper St. Pierre wrote:
Wacky. In any case, NULL is not a valid listener, which is sort of
terrible, but it is how it is. You need to create an empty function
that does nothing.
On Tue, Jul 7, 2015 at 10:17 AM, Christopher Michael
<[email protected]> wrote:
On 07/07/2015 01:15 PM, Jasper St. Pierre wrote:
Shouldn't missing fields in structs be auto-initialized to 0 / NULL? I
thought that was part of the C specification.
I thought so also however when compiling some other code which was also
creating a wl_output_listener, I uncovered the warnings about missing field
initializers. When I looked/referenced the existing screenshooting code I
noticed that they were missing from there also, so I just made a quick patch
to address that.
Cheers,
Chris
On Tue, Jul 7, 2015 at 8:52 AM, Christopher Michael
<[email protected]> wrote:
This patch adds missing placeholders for the wl_output listener
functions 'done' and 'scale. Currently these placeholders are being
set to NULL as the done and scale callbacks are not used in the
screenshot client.
Signed-off-by: Chris Michael <[email protected]>
---
clients/screenshot.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/clients/screenshot.c b/clients/screenshot.c
index f11e3ba..0d9b320 100644
--- a/clients/screenshot.c
+++ b/clients/screenshot.c
@@ -114,7 +114,9 @@ display_handle_mode(void *data,
static const struct wl_output_listener output_listener = {
display_handle_geometry,
- display_handle_mode
+ display_handle_mode,
+ NULL,
+ NULL,
};
static void
--
2.4.4
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel
_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel
