Hi guys,<br><br>heres the diff:  (should i use svn?)<br><br>

<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

<meta name="Generator" content="Kate, the KDE Advanced Text Editor">

<pre><b></b><span style="color: rgb(128, 0, 0);">@@ -376,8 +376,13 @@</span>

     {

         p_file_generic_dialog->SetMessage( wxU(p_arg->psz_title) );

         p_file_generic_dialog->SetWildcard( wxU(p_arg->psz_extensions) );

<span style="color: rgb(221, 0, 0);">+#if wxCHECK_VERSION(2,7,0) // versions 2.7.0 and above</span>

<span style="color: rgb(221, 0, 0);">+        p_file_generic_dialog->SetWindowStyle( (p_arg->b_save ? wxSAVE : wxOPEN) |</span>

<span style="color: rgb(221, 0, 0);">+                                         (p_arg->b_multiple ? wxMULTIPLE:0) );</span>

<span style="color: rgb(221, 0, 0);">+#else // versions below 2.7.0</span>

         p_file_generic_dialog->SetStyle( (p_arg->b_save ? wxSAVE : wxOPEN) |

                                          (p_arg->b_multiple ? wxMULTIPLE:0) );

<span style="color: rgb(221, 0, 0);">+#endif</span>

     }

     if( p_file_generic_dialog &&

</pre> Hopefully thats close to your coding standards. I checked it against 2.7.0-1 as well as 2.6.3 & 2.8.0. Interestingly SetWindowStyle() seemed to work with all 3 wx versions, but ive done it this way just in case it didnt work with other previous versions.

<br><br>If you accept the diff, ill post it to that forum thread as well.<br><br>Jim.<br><br><div><span class="gmail_quote">On 17/03/07, <b class="gmail_sendername">Jean-Baptiste Kempf</b> <<a href="mailto:jb@videolan.org">

jb@videolan.org</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>On Sat, Mar 17, 2007, Rainer Weidenthaler wrote :<br>

>    But with version 2.8 they changed this to SetWindowStyle. So first<br>>    question: are you sure you are compiling vlc with 2.6.3? Probably<br>>    providing some more configure options will get the compiler to link the

>    correct version of wxwidgets (use ./configure --help | grep wx ) One another mail about wx... ok, if you provide a correct patch about that special issue to make 0.8.6 compile with wx 2.8.0, I think we can include it.

<br>But we will not support it, so patch with a good set of #ifdef to get<br>wx-2.6.3 and wx-2.8.0 should be accepted, IMHO if YOU provide it.<br><br><br><br>--<br>Jean-Baptiste Kempf<br><a href="http://www.jbkempf.com/">

http://www.jbkempf.com/</a><br><br>--<br>This is the vlc-devel mailing-list, see <a href="http://www.videolan.org/vlc/">http://www.videolan.org/vlc/</a><br>To unsubscribe, please read <a href="http://developers.videolan.org/lists.html">

http://developers.videolan.org/lists.html</a><br><br></blockquote></div><br>