Thanks about the comments, I'm making a note of the next edits to be done...<br>Regarding the number of devices, right now it simply read the first one, but it's possible to read them all, it's just that i don't know if it's possible to have more than one device in the playlist window....<br>
<br><div class="gmail_quote">On Sat, Mar 29, 2008 at 3:05 AM, Rafaël Carré <<a href="mailto:funman@videolan.org">funman@videolan.org</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
-----BEGIN PGP SIGNED MESSAGE-----<br>
Hash: SHA1<br>
<br>
Le Thu, 27 Mar 2008 23:23:58 +0000,<br>
"Fabio Ritrovato" <<a href="mailto:exsephiroth87@gmail.com">exsephiroth87@gmail.com</a>> a écrit :<br>
<div><div></div><div class="Wj3C7c"><br>
> Ok, i've edited the patch with the suggestions from funman, and added<br>
> <a href="http://configure.ac" target="_blank">configure.ac</a> and <a href="http://modules.am" target="_blank">modules.am</a> (now with no random changes xD)<br>
> The only thing left as it was is the device addiction...<br>
> I thought about that, and i think that maybe the best way to manage<br>
> that is using HAL, so for now i left things as they were, and in<br>
> future maybe I'll add that thing...<br>
><br>
> The patch is a brand new one, it's not to be applied after the old<br>
> one...<br>
<br>
</div></div>For the <a href="http://configure.ac" target="_blank">configure.ac</a> diff you should write default enabled, and use<br>
MSG_ERR if the module was explicitely asked:<br>
if test "${enable_mtp}" = "yes"<br>
<br>
About the access, please remove file-cat option<br>
Also, use spaces instead of tabs<br>
<br>
Instead of using getenv(HOME) you should use vlc custom functions to<br>
retrieve the home dir (look in include/vlc_configuration.h)<br>
You should remove b_stdin completely<br>
If you copy file per file, you don't have to check if your target is a<br>
directory.<br>
You should also be sure that your file is a regular file<br>
+               msg_Err( p_access, "Error deleting file %s, errno:<br>
%d...", p_sys<br>
You can use %m in msg_Err expressions, which will be replaced by a<br>
human readable error message (strerror())<br>
<br>
In AddTrack, instead of malloc() / sprintf() you can use asprintf()<br>
<br>
Also what happens if someone has 2 MTP devices connected when he launch<br>
the mtp services discovery ?<br>
<br>
Thanks, and sorry for being late to comment your patch<br>
<br>
- --<br>
Rafaël Carré<br>
-----BEGIN PGP SIGNATURE-----<br>
Version: GnuPG v2.0.8 (FreeBSD)<br>
<br>
iEYEARECAAYFAkfto+MACgkQYWCeGMCv8Q8VWACeKrVbXt9+cpOTf2ZmXd5IIeZn<br>
eFoAnRvEdJAUnT4XBAx1cUTzVAU5p6Lo<br>
=TfPO<br>
-----END PGP SIGNATURE-----<br>
<div><div></div><div class="Wj3C7c">_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>