<html dir="ltr"><head></head><body style="text-align: left; direction: ltr; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;"><div>Hi!</div><div><br></div><div>On Tue, 2018-08-14 at 14:49 +0200, Jean-Baptiste Kempf wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><pre>You are not allowed to include vlc_common.h without having config.h+boilerplate.</pre><pre><br></pre></blockquote><pre><div style="font-family: "Fira Sans";">config.h is included in both the .cpp files.</div><div style="font-family: "Fira Sans";">Here I have just moved back vlc_common.h to the header file since the header files directly depend on the defines in vlc_common.h</div><div style="font-family: "Fira Sans";">This is similar to whats done for chromecast and chromecast.h</div></pre><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><pre>On Tue, 14 Aug 2018, at 14:13, Shaleen Jain via vlc-devel wrote:</pre><pre>_______________________________________________</pre><pre>vlc-devel mailing list</pre><pre>To unsubscribe or modify your subscription options:</pre><pre><a href="https://mailman.videolan.org/listinfo/vlc-devel">https://mailman.videolan.org/listinfo/vlc-devel</a></pre><pre>Email had 1 attachment:</pre><pre>+ [PATCH 1/4] upnp: fix headers</pre><pre>  4k (message/rfc822)</pre><pre><br></pre><pre><br></pre><pre><br></pre></blockquote><div><span><pre>-- <br></pre><div style="width: 71ch;">Regards,</div><div style="width: 71ch;">Shaleen Jain</div></span></div></body></html>