<div dir="ltr">Hi,<div class="gmail_extra"><br><div class="gmail_quote">On Fri, May 31, 2013 at 8:14 AM, Martin Minarik <span dir="ltr"><<a href="mailto:minarik11@student.fiit.stuba.sk" target="_blank">minarik11@student.fiit.stuba.sk</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

diff --git a/src/extensions/weston/weston-wfits.cpp b/src/extensions/weston/weston-wfits.cpp<br>
index a54e0fe..96cf617 100644<br>
--- a/src/extensions/weston/weston-wfits.cpp<br>
+++ b/src/extensions/weston/weston-wfits.cpp<br>
@@ -42,6 +42,10 @@ extern "C" {<br>
 #undef private<br>
 }<br>
<br>
+extern "C" {<br>
+#include <weston/evdev.h><br>
+}<br></blockquote><div><br></div><div style>Header files that could be potentially exposed to C++ really should be C++ friendly.  If evdev.h is going to be exported as part of the Weston SDK it should contain the appropriate extern "C" block.  Something like the following should work:</div>
<div style><br></div><div style><div>diff --git a/src/evdev.h b/src/evdev.h</div><div>index eb5c868..46a0798 100644</div><div>--- a/src/evdev.h</div><div>+++ b/src/evdev.h</div><div>@@ -26,6 +26,10 @@</div><div> #include <linux/input.h></div>
<div> #include <wayland-util.h></div><div> </div><div>+#ifdef  __cplusplus</div><div>+extern "C" {</div><div>+#endif</div><div>+</div><div> #define MAX_SLOTS 16</div><div> </div><div> enum evdev_event_type {</div>
<div>@@ -124,4 +128,9 @@ void</div><div> evdev_notify_keyboard_focus(struct weston_seat *seat,</div><div>                            struct wl_list *evdev_devices);</div><div> </div><div>+#ifdef  __cplusplus</div><div>+}</div>
<div>+#endif</div><div>+</div><div> #endif /* EVDEV_H */</div><div><br></div><div><br></div><div style>You then won't need to wrap the #include <weston/evdev.h> with an extern "C" block.</div><div style>
<br></div><div style>-Ossama</div></div></div></div></div>