<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">

<html>

<head>

  <meta content="text/html;charset=iso-8859-1" http-equiv="Content-Type">

</head>

<body bgcolor="#ffffff" text="#000000">

<tt>Jean-Baptiste Kempf wrote:</tt>

<blockquote cite="mid:20090902164657.GA4086@videolan.org" type="cite">

  <pre wrap="">On Wed, Sep 02, 2009 at 07:30:44PM +0300, Rémi Denis-Courmont wrote :

  </pre>

  <blockquote type="cite">

    <pre wrap="">Le mercredi 2 septembre 2009 19:14:20 Nick Pope, vous avez écrit :

    </pre>

    <blockquote type="cite">

      <pre wrap="">Can you think of anything further to add regarding this?  And if it is

fine is someone able to commit it?

      </pre>

    </blockquote>

    <pre wrap="">It's fine with me, but JB seems to imply that this would break Windows 

compilation.

    </pre>

  </blockquote>

<pre wrap="">Sorry, that was not what I meant.

I just said that as this is useless on Windows, we could skip it, since

Qt folks don't seem to remove it on Win32</pre>

</blockquote>

<tt>I don't really think it is necessary.  I don't know what the

internal behaviour of Qt compiled for Win32 is regarding

setWindowRole().  I would presume that it just does a no-op as their

documentation states that setWindowRole() is for X11 only.  I am not

able to compile it for Win32 right now, I'm afraid, so I can't test - I

would think that it will just work fine, otherwise it would break

cross-platform compatibility as far as the Qt library is concerned.</tt><br>

</body>

</html>