[vlc-devel] [PATCH] Avoid invalid index constructor.

Vicente Jiménez googuy at gmail.com
Fri Mar 20 17:46:35 CET 2009


As it's usually happens with untested patches, it's not correct!

I've corrected it and I think I'm just going to use it myself to clean
a bit of debug information.
I don't think this patch could be enough interesting or correct to be
commited in the master branch.

As Rémi says, a simpler patch could be remove the message, but we lost
an important message that alert us if we try to use an incorrect
parameter in a 'foreach' construction if we are hacking with http
interface pages.



On Thu, Mar 12, 2009 at 6:52 PM, Vicente Jiménez <googuy at gmail.com> wrote:
> This could be a better patch.
>
> WARNING! this is a quick patch. I supose it's correct but it's unstested.
>
> On Thu, Mar 12, 2009 at 2:01 PM, Rémi Denis-Courmont <rem at videolan.org> wrote:
>> Le jeudi 12 mars 2009 14:13:21 Vicente Jiménez, vous avez écrit :
>>> I've done this patch because I'm continuously requesting VLC's VLM
>>> information, and when VLC is put in debug mode, I get lots of very
>>> annoying invalid index constructor messages that doesn't let me read
>>> others debug messages.
>>
>> Then maybe the proper fix is to remove that message instead??
>>
>> --
>> Rémi Denis-Courmont
>> http://git.remlab.net/cgi-bin/gitweb.cgi?p=vlc-courmisch.git;a=summary
>> _______________________________________________
>> vlc-devel mailing list
>> To unsubscribe or modify your subscription options:
>> http://mailman.videolan.org/listinfo/vlc-devel
>>
>



More information about the vlc-devel mailing list