<br><br><div><span class="gmail_quote">On 12/3/06, <b class="gmail_sendername">Clément Stenac</b> <<a href="mailto:zorglub@videolan.org">zorglub@videolan.org</a>> wrote:</span><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
Hi,<br><br>>>  Also can you say a few words on when/how this memleak happened and<br>>> an approximation on how log it's been around?<br><br>It happened last week when I made most of input_thread_t private.<br>
<br>input_thread_t is now basically<br>{<br>  a few public things;<br>  input_thread_private_t* p;<br>}<br><br>... and I forgot to free it ...<br><br>So the memleak was a few hundreds of bytes at each input thread, since<br>
last week.</blockquote><div><br>Ok, nice to know. </div><br><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
>> And finally, shouldn't p_input->p get a proper name?<br>><br>> Probably (but *I* won't do it).<br><br>I actually didn't want a p_very_long_name as it appears extremeley<br>often, but it wouldn't be hard to change that.
</blockquote><div><br>I that there is likely a relation between the variables short name and it not being freed. <br>People would tend to think "this 'p'-variable can't possibly be important, <br>so I will not check it up." I'd say anything this important deserve a proper name, and p_private isn't that long. Functions using it a lot could maintain a shorthand anyway like most things do for foo->p_sys. Having core code use so short names will make the source base harder to maintain and harder to understand for new coders.
<br><br>Regards<br><br>Sigmund<br></div><br><blockquote class="gmail_quote" style="margin-top: 0; margin-right: 0; margin-bottom: 0; margin-left: 0; margin-left: 0.80ex; border-left-color: #cccccc; border-left-width: 1px; border-left-style: solid; padding-left: 1ex">
Clément<br><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>